Re: [PATCH 09/18] virtio: use avail_event index

2011-05-19 Thread Michael S. Tsirkin
On Wed, May 18, 2011 at 09:49:42AM +0930, Rusty Russell wrote: > On Tue, 17 May 2011 09:10:31 +0300, "Michael S. Tsirkin" > wrote: > > Well one can imagine a driver doing: > > > > while (virtqueue_get_buf()) { > > virtqueue_add_buf() > > } > > virtqueue_kick() > > > > wh

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-17 Thread Michael S. Tsirkin
On Wed, May 18, 2011 at 09:49:42AM +0930, Rusty Russell wrote: > On Tue, 17 May 2011 09:10:31 +0300, "Michael S. Tsirkin" > wrote: > > Well one can imagine a driver doing: > > > > while (virtqueue_get_buf()) { > > virtqueue_add_buf() > > } > > virtqueue_kick() > > > > wh

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-17 Thread Rusty Russell
On Tue, 17 May 2011 09:10:31 +0300, "Michael S. Tsirkin" wrote: > Well one can imagine a driver doing: > > while (virtqueue_get_buf()) { > virtqueue_add_buf() > } > virtqueue_kick() > > which looks sensible (batch kicks) but might > process any number of bufs bet

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-17 Thread Tom Lendacky
On Monday, May 16, 2011 02:12:21 AM Rusty Russell wrote: > On Sun, 15 May 2011 16:55:41 +0300, "Michael S. Tsirkin" wrote: > > On Mon, May 09, 2011 at 02:03:26PM +0930, Rusty Russell wrote: > > > On Wed, 4 May 2011 23:51:47 +0300, "Michael S. Tsirkin" wrote: > > > > Use the new avail_event fea

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-16 Thread Michael S. Tsirkin
On Mon, May 16, 2011 at 04:42:21PM +0930, Rusty Russell wrote: > On Sun, 15 May 2011 16:55:41 +0300, "Michael S. Tsirkin" > wrote: > > On Mon, May 09, 2011 at 02:03:26PM +0930, Rusty Russell wrote: > > > On Wed, 4 May 2011 23:51:47 +0300, "Michael S. Tsirkin" > > > wrote: > > > > Use the new av

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-16 Thread Rusty Russell
On Sun, 15 May 2011 16:55:41 +0300, "Michael S. Tsirkin" wrote: > On Mon, May 09, 2011 at 02:03:26PM +0930, Rusty Russell wrote: > > On Wed, 4 May 2011 23:51:47 +0300, "Michael S. Tsirkin" > > wrote: > > > Use the new avail_event feature to reduce the number > > > of exits from the guest. > >

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-15 Thread Michael S. Tsirkin
On Mon, May 09, 2011 at 02:03:26PM +0930, Rusty Russell wrote: > On Wed, 4 May 2011 23:51:47 +0300, "Michael S. Tsirkin" > wrote: > > Use the new avail_event feature to reduce the number > > of exits from the guest. > > Figures here would be nice :) You mean ASCII art in comments? > > @@ -228,

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-08 Thread Rusty Russell
On Wed, 4 May 2011 23:51:47 +0300, "Michael S. Tsirkin" wrote: > Use the new avail_event feature to reduce the number > of exits from the guest. Figures here would be nice :) > @@ -228,6 +237,12 @@ add_head: >* new available array entries. */ > virtio_wmb(); > vq->vring.avail

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-05 Thread Tom Lendacky
On Wednesday, May 04, 2011 03:51:47 PM Michael S. Tsirkin wrote: > Use the new avail_event feature to reduce the number > of exits from the guest. > > Signed-off-by: Michael S. Tsirkin > --- > drivers/virtio/virtio_ring.c | 39 > ++- 1 files changed, 38 inse

Re: [PATCH 09/18] virtio: use avail_event index

2011-05-05 Thread Michael S. Tsirkin
On Wed, May 04, 2011 at 04:58:18PM -0500, Tom Lendacky wrote: > > On Wednesday, May 04, 2011 03:51:47 PM Michael S. Tsirkin wrote: > > Use the new avail_event feature to reduce the number > > of exits from the guest. > > > > Signed-off-by: Michael S. Tsirkin > > --- > > drivers/virtio/virtio_ri

[PATCH 09/18] virtio: use avail_event index

2011-05-04 Thread Michael S. Tsirkin
Use the new avail_event feature to reduce the number of exits from the guest. Signed-off-by: Michael S. Tsirkin --- drivers/virtio/virtio_ring.c | 39 ++- 1 files changed, 38 insertions(+), 1 deletions(-) diff --git a/drivers/virtio/virtio_ring.c b/drivers/