From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hannes Reinecke Subject: Re: [Qemu-devel] [PATCH 0/4] scsi fixes Date: Tue, 19 Jul 2011 10:10:47 +0200 Message-ID: <4E253C07.90008@suse.de> References: <1310389346-30790-1-git-send-email-hare@suse.de> <4E1C4E2D.50501@redhat.com> <4E2524CF.3080309@suse.de> <4E2534A1.4000701@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Paolo Bonzini , Alexander Graf , qemu-devel@nongnu.org, kvm@vger.kernel.org, Stefan Haynoczi To: Kevin Wolf Return-path: Received: from cantor2.suse.de ([195.135.220.15]:44329 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752431Ab1GSIKv (ORCPT ); Tue, 19 Jul 2011 04:10:51 -0400 In-Reply-To: <4E2534A1.4000701@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 07/19/2011 09:39 AM, Kevin Wolf wrote: > Am 19.07.2011 08:31, schrieb Hannes Reinecke: >> On 07/12/2011 03:37 PM, Kevin Wolf wrote: >>> Am 11.07.2011 15:02, schrieb Hannes Reinecke: >>>> Hi all, >>>> >>>> these are some fixes I found during debugging my megasas HBA emula= tion. >>>> This time I've sent them as a separate patchset for inclusion. >>>> All of them have been acked, so please apply. >>>> >>>> Hannes Reinecke (4): >>>> iov: Update parameter usage in iov_(to|from)_buf() >>>> scsi: Add 'hba_private' to SCSIRequest >>>> scsi-disk: Fixup debugging statement >>>> scsi-disk: Mask out serial number EVPD >>> >>> Thanks, applied all to the block branch. >>> >> Any chance to have them pulled into the main tree? >> My megasas emulation relies on them, and it feels a bit >> stupid to send a patch relying on some fixes not in mainline. >> At the same time it's really stupid to resend the entire >> patchset again ... > > I'm hoping to send a pull request today, now that the VMDK patches lo= ok > good finally. > > Anyway, I don't think that not having the patches in master yet shoul= d > stop you from going forward with the next patches. They will go throu= gh > the block tree anyway, so basing them on that tree is fine (and you > wouldn't be the first one to do that). Just state in PATCH 0/n for th= e > reviewers that it depends on the other patches. > Well, the remaining patch is 'just' the megasas emulation itself. And I want to make reviewing that as easy as possible, so that it's=20 not again being held off by complains about missing patches. Cheers, Hannes --=20 Dr. Hannes Reinecke zSeries & Storage hare@suse.de +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg GF: J. Hawn, J. Guild, F. Imend=F6rffer, HRB 16746 (AG N=FCrnberg)