From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=52853 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PLBGD-0002oE-9z for qemu-devel@nongnu.org; Wed, 24 Nov 2010 04:03:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PLBG8-00036y-Ax for qemu-devel@nongnu.org; Wed, 24 Nov 2010 04:03:29 -0500 Received: from mx1.redhat.com ([209.132.183.28]:20083) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PLBG8-00036q-1Q for qemu-devel@nongnu.org; Wed, 24 Nov 2010 04:03:24 -0500 Message-ID: <4CECD50F.9060501@redhat.com> Date: Wed, 24 Nov 2010 10:04:15 +0100 From: Kevin Wolf MIME-Version: 1.0 References: <1290586723-8724-1-git-send-email-nab@linux-iscsi.org> <4CECD36E.50401@suse.de> In-Reply-To: <4CECD36E.50401@suse.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH] scsi-generic: bugfixes for 'SCSIRequest' conversion List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Hannes Reinecke Cc: Stefan Hajnoczi , qemu-devel , "Nicholas A. Bellinger" , Gerd Hoffmann , Paolo Bonzini , linux-iscsi-target-dev@googlegroups.com Am 24.11.2010 09:57, schrieb Hannes Reinecke: > On 11/24/2010 09:18 AM, Nicholas A. Bellinger wrote: >> From: Nicholas Bellinger >> >> This patch adds a handful of bugfixes for scsi-generic >> that where added into: >> >> commit a4194b3f79a85e111f000788ddec05d465748851 >> Author: Hannes Reinecke >> Date: Mon Nov 22 15:39:33 2010 -0800 >> >> scsi: Use 'SCSIRequest' directly >> >> this includes: >> >> *) Fix incorrect errno usage in switch() statement within >> scsi_command_complete() >> >> *) Remove bogus scsi_command_complete() for residual case >> within scsi_read_complete() >> >> *) Remove incorrect '-' sign from return in scsi_send_command() >> >> Tested with .37-rc2 TCM_Loop FILEIO backstores on KVM host into >> Debian Lenny v2.6.26 KVM guest with an xfs filesystem mount. >> >> Signed-off-by: Nicholas A. Bellinger nab@linux-iscsi.org> Hannes, can you fold the necessary parts of this into the next version of your series, so that we don't break things first and fix them only later? Kevin