|
From: | Gerd Hoffmann |
Subject: | Re: [Qemu-devel] [PATCH 09/15] scsi: move scsi request parsing into generic code. |
Date: | Tue, 17 Nov 2009 13:51:29 +0100 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20091014 Fedora/3.0-2.8.b4.fc11 Lightning/1.0pre Thunderbird/3.0b4 |
On 11/17/09 13:27, Paul Brook wrote:
On Tuesday 17 November 2009, Christoph Hellwig wrote:The subject is a bit confusing - it's not the full request parsing but just some helpers.This is a good example of a patch with an insufficient commit message. Given the way GIT treats the first line of the commit mesaage, my advice is to make both the subject and the body of the commit message independent. On a more technical note, why aren't you also using this function in scsi- disc.c? Surely that's the whole point of moving it into common code.
Same as with the command move: next patch series will rework scsi-disk to put the new fields and functions into use.
cheers, Gerd
[Prev in Thread] | Current Thread | [Next in Thread] |