From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49735) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZzAEV-0004C5-RG for qemu-devel@nongnu.org; Wed, 18 Nov 2015 16:25:40 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZzAES-0000C5-JY for qemu-devel@nongnu.org; Wed, 18 Nov 2015 16:25:39 -0500 Received: from mx1.redhat.com ([209.132.183.28]:42094) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZzAES-0000C0-DD for qemu-devel@nongnu.org; Wed, 18 Nov 2015 16:25:36 -0500 Date: Wed, 18 Nov 2015 23:25:32 +0200 From: "Michael S. Tsirkin" Message-ID: <20151118231639-mutt-send-email-mst@redhat.com> References: <00f401d12108$09faee70$1df0cb50$@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <00f401d12108$09faee70$1df0cb50$@samsung.com> Subject: Re: [Qemu-devel] [PATCH v2] vhost: Fix aborting if KVM does not support eventfds List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Pavel Fedin Cc: 'Christian Borntraeger' , qemu-devel@nongnu.org On Tue, Nov 17, 2015 at 10:17:37AM +0300, Pavel Fedin wrote: > If you happen to have a stock kernel of old version, like 3.x, On ARM > and you > attempt to enable vhost by setting vhost=on, qemu aborts with error: > kvm_mem_ioeventfd_add: error adding ioeventfd: Function not implemented > > This patch adds capability check, so that vhost gets disabled instead. A > warning is displayed, explaining the reason: > > 2015-11-13T08:43:51.146802Z qemu-system-aarch64: KVM does not support eventfd binding > 2015-11-13T08:43:51.146915Z qemu-system-aarch64: unable to start vhost net: 38: falling back on userspace virtio > > This problem can be observed with libvirt, which checks for /dev/vhost-net > availability and just inserts "vhost=on" automatically in this case. > > Signed-off-by: Pavel Fedin > --- > v1 => v2: > - Removed "MMIO" from warning message, because it applies not only to MMIO > - Add note about warning to the commit message > --- > hw/virtio/vhost.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c > index 1794f0d..50b8171 100644 > --- a/hw/virtio/vhost.c > +++ b/hw/virtio/vhost.c > @@ -24,6 +24,7 @@ > #include "hw/virtio/virtio-bus.h" > #include "hw/virtio/virtio-access.h" > #include "migration/migration.h" > +#include "sysemu/kvm.h" > > static struct vhost_log *vhost_log; > static struct vhost_log *vhost_log_shm; > @@ -1083,6 +1084,11 @@ int vhost_dev_enable_notifiers(struct vhost_dev *hdev, VirtIODevice *vdev) > r = -ENOSYS; > goto fail; > } > + if (!kvm_eventfds_enabled()) { > + error_report("KVM does not support eventfd binding"); > + r = -ENOSYS; > + goto fail; > + } > > for (i = 0; i < hdev->nvqs; ++i) { > r = k->set_host_notifier(qbus->parent, hdev->vq_index + i, true); That's a wrong thing for vhost to do - we don't want it poking at kvm because it does not work with either kvm or eventfd. It uses host notifiers, there should be an API to check for that. Or better yet, actually implement host notifiers in userspace - should not be that hard to do, just do write into eventfd. > -- > 1.9.5.msysgit.0 >