From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NG8Se-0008R4-NB for qemu-devel@nongnu.org; Thu, 03 Dec 2009 04:58:56 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1NG8SZ-0008JX-06 for qemu-devel@nongnu.org; Thu, 03 Dec 2009 04:58:55 -0500 Received: from [199.232.76.173] (port=51665 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NG8SY-0008J9-MP for qemu-devel@nongnu.org; Thu, 03 Dec 2009 04:58:50 -0500 Received: from mx1.redhat.com ([209.132.183.28]:61282) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NG8SY-0003mM-5z for qemu-devel@nongnu.org; Thu, 03 Dec 2009 04:58:50 -0500 Message-ID: <4B178BCE.7030003@redhat.com> Date: Thu, 03 Dec 2009 11:58:38 +0200 From: Avi Kivity MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH] virtio spec: add virtio-blk max sectors feature List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avishay Traeger1 Cc: Rusty Russell , qemu-devel@nongnu.org, kvm@vger.kernel.org, virtualization On 12/03/2009 10:42 AM, Avishay Traeger1 wrote: > I previously submitted a patch to have the guest virtio-blk driver get the > value for the maximum I/O size from the host bdrv, rather than assume that > there is no limit. Avi requested that I first patch the virtio spec > (http://ozlabs.org/~rusty/virtio-spec/). Below is that patch. > > Please CC me on replies, as I am not subscribed. > > Copying Rusty and virtualization@. > Thank you, > Avishay > > Signed-off-by: Avishay Traeger > > --- virtio-spec-0.8.2.lyx 2009-11-30 16:34:16.000000000 +0200 > +++ virtio-spec-0.8.2-mod.lyx 2009-12-01 16:04:36.000000000 +0200 > @@ -3999,6 +3999,20 @@ > \end_inset > > . > +\change_inserted 0 1259672174 > + > +\end_layout > + > +\begin_layout Description > + > +\change_inserted 0 1259672252 > +VIRTIO_BLK_F_SECTOR_MAX > +\begin_inset space ~ > +\end_inset > + > +(10) Maximum sectors in an I/O. > +\change_unchanged > + > \end_layout > > \end_deeper > @@ -4068,6 +4082,17 @@ > \begin_layout Plain Layout > > u32 blk_size; > +\change_inserted 0 1259672274 > + > +\end_layout > + > +\begin_layout Plain Layout > + > +\change_inserted 0 1259672305 > + > + u32 sectors_max; > +\change_unchanged > + > \end_layout > > \begin_layout Plain Layout > @@ -4107,6 +4132,18 @@ > \begin_layout Enumerate > If the VIRTIO_BLK_F_RO feature is set by the device, any write requests > will fail. > +\change_inserted 0 1259672345 > + > +\end_layout > + > +\begin_layout Enumerate > + > +\change_inserted 0 1259672454 > +If the VIRTIO_BLK_F_SECTOR_MAX feature is negotiated, the sectors_max > field > + should be read to determine the maximum I/O size for the driver to use. > + No requests should be submitted which go beyond this limit. > +\change_unchanged > + > \end_layout > > \begin_layout Section* > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- error compiling committee.c: too many arguments to function