qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 0/3] virtio: don't poll masked vectors with irqf


From: Asias He
Subject: Re: [Qemu-devel] [PATCH 0/3] virtio: don't poll masked vectors with irqfd
Date: Wed, 19 Dec 2012 16:53:22 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0

On 12/18/2012 08:39 PM, Michael S. Tsirkin wrote:
> At the moment when vector is masked virtio will poll it
> in userspace, even if it is handled by irqfd.
> This is done in order to update pending bits, but
> it's not really required until someone reads the pending bits.
> On the other hand this read results in extra io thread wakeups.
> 
> As we only implement the pending bits as a compatibility
> feature (read - real drivers don't use it), we can defer
> the irqfd poll until the read actually happens.
> 
> This does not seem to affect vhost-net speed
> in simple benchmarks but could help block: both
> vhost-blk and dataplane when using irqfd,
> and I also think this is cleaner than enabling/disabling
> notifiers all the time.
> 
> This will also be the basis for future optimizations.
> 
> Michael S. Tsirkin (3):
>   msi: add API to get notified about pending bit poll
>   msix: expose access to masked/pending state
>   virtio-pci: don't poll masked vectors
> 
>  hw/pci/msix.c   | 19 +++++++++++++++----
>  hw/pci/msix.h   |  6 +++++-
>  hw/pci/pci.h    |  4 ++++
>  hw/vfio_pci.c   |  2 +-
>  hw/virtio-pci.c | 53 +++++++++++++++++++++++++++++++++++++++++------------
>  5 files changed, 66 insertions(+), 18 deletions(-)

The performance boost is significant here. It is close to the
result of dropping msix_fire_vector_notifier() hack.

-- 
Asias



reply via email to

[Prev in Thread] Current Thread [Next in Thread]