From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:40274) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Stvd5-0007Zc-0D for qemu-devel@nongnu.org; Wed, 25 Jul 2012 03:03:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Stvd0-0005TF-4O for qemu-devel@nongnu.org; Wed, 25 Jul 2012 03:03:30 -0400 Received: from mail-we0-f173.google.com ([74.125.82.173]:47567) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Stvcz-0005T6-UA for qemu-devel@nongnu.org; Wed, 25 Jul 2012 03:03:26 -0400 Received: by weyz53 with SMTP id z53so280143wey.4 for ; Wed, 25 Jul 2012 00:03:25 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <500F9A37.9080203@redhat.com> Date: Wed, 25 Jul 2012 09:03:19 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1343169246-17636-1-git-send-email-nab@linux-iscsi.org> <1343169246-17636-8-git-send-email-nab@linux-iscsi.org> <500F99B4.4050209@redhat.com> In-Reply-To: <500F99B4.4050209@redhat.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC 7/9] virtio-scsi: Start/stop vhost List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jens Axboe , Stefan Hajnoczi , kvm-devel , "Michael S. Tsirkin" , qemu-devel , "Nicholas A. Bellinger" , lf-virt , Anthony Liguori , target-devel , Hannes Reinecke , Zhi Yong Wu , Christoph Hellwig Il 25/07/2012 09:01, Paolo Bonzini ha scritto: >> > From: Stefan Hajnoczi >> > >> > This patch starts and stops vhost as the virtio device transitions >> > through its status phases. Vhost can only be started once the guest >> > reports its driver has successfully initialized, which means the >> > virtqueues have been set up by the guest. >> > >> > (v2: Squash virtio-scsi: use the vhost-scsi host device from stefan) >> > >> > Signed-off-by: Stefan Hajnoczi >> > Signed-off-by: Zhi Yong Wu >> > Cc: Michael S. Tsirkin >> > Cc: Paolo Bonzini >> > Signed-off-by: Nicholas Bellinger > Hmm, this is not what the patch does... :) Oops, the above comment was meant for patch 5. Which is a one-liner that can be squashed here. Anyway there is some problem with the ordering of the patches, because this patch includes vhost-scsi.h (introduced in patch 9) and patch 5 uses VHostSCSI (defined by vhost-scsi.h). Paolo