WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-users

RE: [Xen-users] More verbose commit messages?

To: "Matt Ayres" <matta@xxxxxxxxxxxx>, <xen-users@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-users] More verbose commit messages?
From: "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>
Date: Wed, 18 May 2005 08:22:59 +0100
Cc: ian.pratt@xxxxxxxxxxxx
Delivery-date: Wed, 18 May 2005 07:22:31 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-users-request@lists.xensource.com?subject=help>
List-id: Xen user discussion <xen-users.lists.xensource.com>
List-post: <mailto:xen-users@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-users>, <mailto:xen-users-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-users>, <mailto:xen-users-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-users-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcVbaHZ8pZ8OGJTzSxmkvgDrdtNCOgAEGgnA
Thread-topic: [Xen-users] More verbose commit messages?
 

> Just as a suggestion, perhaps publish more of what a commit 
> change does (ie. FreeBSD style) or even reference a bug# or 
> mailing list thread.

Fair comment. From a developers point of view, I think the checkin
comments are generally quite descriptive, though it would be helpful if
they were concluded with a sentence that described the user impact,
citeing a bug where appropriate.

[ I do wish that checkin comments would be prefixed with the
architecture they effect if they're arch specific, or if they're guest
related, which guest they effect. However, it's probably futile trying
to get x86_32 and linux 2.6 developers to do this consistently :-) ]

> http://xen.bkbits.net:8080/xen-2.0-testing.bk/cset@xxxxxx?
> nav=index.html|ChangeSet@-7d
> 
> and while it does look like something important and maybe I 
> should consider upgrading in order to fix a serious bug I 
> really have no idea... it might just be a cleanup of the 
> coding style.  I'm not C hacker so I have no idea.

"Fix multi-page I/O accesses in the blkback driver in cases where we
receive partial-completion callbacks" is actually a pretty good
description of what that changset does. I'm not sure anyone has ever
filed this as a bug, but its potentially serious and was found as part
of investigating an issue with CDROM accesses not working.

Ian

_______________________________________________
Xen-users mailing list
Xen-users@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-users

<Prev in Thread] Current Thread [Next in Thread>