Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2017-01-05 Thread Stefan Hajnoczi
On Thu, Nov 17, 2016 at 07:38:45PM +0200, Michael S. Tsirkin wrote: > On Thu, Nov 17, 2016 at 01:27:49PM +, Stefan Hajnoczi wrote: > > On Thu, Nov 17, 2016 at 12:17:57AM +0200, Michael S. Tsirkin wrote: > > > On Wed, Nov 16, 2016 at 09:53:06PM +, Stefan Hajnoczi wrote: > > > > Disabling not

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-18 Thread Stefan Hajnoczi
On Fri, Nov 18, 2016 at 04:21:33PM +0200, Michael S. Tsirkin wrote: > On Fri, Nov 18, 2016 at 10:58:47AM +, Stefan Hajnoczi wrote: > > On Thu, Nov 17, 2016 at 07:38:45PM +0200, Michael S. Tsirkin wrote: > > > On Thu, Nov 17, 2016 at 01:27:49PM +, Stefan Hajnoczi wrote: > > > > On Thu, Nov 1

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-18 Thread Michael S. Tsirkin
On Fri, Nov 18, 2016 at 10:58:47AM +, Stefan Hajnoczi wrote: > On Thu, Nov 17, 2016 at 07:38:45PM +0200, Michael S. Tsirkin wrote: > > On Thu, Nov 17, 2016 at 01:27:49PM +, Stefan Hajnoczi wrote: > > > On Thu, Nov 17, 2016 at 12:17:57AM +0200, Michael S. Tsirkin wrote: > > > > On Wed, Nov 1

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-18 Thread Christian Borntraeger
On 11/18/2016 12:02 PM, Stefan Hajnoczi wrote: > On Thu, Nov 17, 2016 at 12:01:30PM +0100, Christian Borntraeger wrote: >> On 11/16/2016 10:53 PM, Stefan Hajnoczi wrote: >>> Disabling notifications during virtqueue processing reduces the number of >>> exits. The virtio-net device already uses virt

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-18 Thread Stefan Hajnoczi
On Thu, Nov 17, 2016 at 12:01:30PM +0100, Christian Borntraeger wrote: > On 11/16/2016 10:53 PM, Stefan Hajnoczi wrote: > > Disabling notifications during virtqueue processing reduces the number of > > exits. The virtio-net device already uses virtio_queue_set_notifications() > > but > > virtio-b

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-18 Thread Stefan Hajnoczi
On Thu, Nov 17, 2016 at 07:38:45PM +0200, Michael S. Tsirkin wrote: > On Thu, Nov 17, 2016 at 01:27:49PM +, Stefan Hajnoczi wrote: > > On Thu, Nov 17, 2016 at 12:17:57AM +0200, Michael S. Tsirkin wrote: > > > On Wed, Nov 16, 2016 at 09:53:06PM +, Stefan Hajnoczi wrote: > > > > Disabling not

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-17 Thread Michael S. Tsirkin
On Thu, Nov 17, 2016 at 01:27:49PM +, Stefan Hajnoczi wrote: > On Thu, Nov 17, 2016 at 12:17:57AM +0200, Michael S. Tsirkin wrote: > > On Wed, Nov 16, 2016 at 09:53:06PM +, Stefan Hajnoczi wrote: > > > Disabling notifications during virtqueue processing reduces the number of > > > exits. T

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-17 Thread Stefan Hajnoczi
On Thu, Nov 17, 2016 at 12:17:57AM +0200, Michael S. Tsirkin wrote: > On Wed, Nov 16, 2016 at 09:53:06PM +, Stefan Hajnoczi wrote: > > Disabling notifications during virtqueue processing reduces the number of > > exits. The virtio-net device already uses virtio_queue_set_notifications() > > b

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-17 Thread Christian Borntraeger
On 11/16/2016 10:53 PM, Stefan Hajnoczi wrote: > Disabling notifications during virtqueue processing reduces the number of > exits. The virtio-net device already uses virtio_queue_set_notifications() > but > virtio-blk and virtio-scsi do not. > > The following benchmark shows a 15% reduction in

Re: [Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-16 Thread Michael S. Tsirkin
On Wed, Nov 16, 2016 at 09:53:06PM +, Stefan Hajnoczi wrote: > Disabling notifications during virtqueue processing reduces the number of > exits. The virtio-net device already uses virtio_queue_set_notifications() > but > virtio-blk and virtio-scsi do not. > > The following benchmark shows a

[Qemu-devel] [PATCH 0/3] virtio: disable notifications in blk and scsi

2016-11-16 Thread Stefan Hajnoczi
Disabling notifications during virtqueue processing reduces the number of exits. The virtio-net device already uses virtio_queue_set_notifications() but virtio-blk and virtio-scsi do not. The following benchmark shows a 15% reduction in virtio-blk-pci MMIO exits: (host)$ qemu-system-x86_64 \