diff mbox

[trivial] print meaningful error message in case of --disable-vhost-net

Message ID 20110609210519.5BA095288@gandalf.tls.msk.ru
State New
Headers show

Commit Message

Michael Tokarev June 9, 2011, 8:55 p.m. UTC
When qemu gets compiled without support of vhost-net, any attempt
to use it fails with a very clear error message:

 qemu-system-x86_64: -netdev ...,vhost=on: vhost-net requested but could not be initialized

there's absolutely no reason given _why_ it coult not be
initialized, and even strace'ing the process in question
does not reveal any errors.  So print a message telling
what's going on.

Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
---
 hw/vhost_net.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Stefan Hajnoczi June 10, 2011, 8:32 a.m. UTC | #1
On Thu, Jun 9, 2011 at 9:55 PM, Michael Tokarev <mjt@tls.msk.ru> wrote:
> When qemu gets compiled without support of vhost-net, any attempt
> to use it fails with a very clear error message:
>
>  qemu-system-x86_64: -netdev ...,vhost=on: vhost-net requested but could not be initialized
>
> there's absolutely no reason given _why_ it coult not be
> initialized, and even strace'ing the process in question
> does not reveal any errors.  So print a message telling
> what's going on.
>
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
>  hw/vhost_net.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)

This is useful.  CCed Michael Tsirkin.

Stefan
diff mbox

Patch

diff --git a/hw/vhost_net.c b/hw/vhost_net.c
index 420e05f..33ba402 100644
--- a/hw/vhost_net.c
+++ b/hw/vhost_net.c
@@ -15,6 +15,7 @@ 
 
 #include "virtio-net.h"
 #include "vhost_net.h"
+#include "qemu-error.h"
 
 #include "config.h"
 
@@ -197,6 +198,7 @@  void vhost_net_cleanup(struct vhost_net *net)
 struct vhost_net *vhost_net_init(VLANClientState *backend, int devfd,
                                  bool force)
 {
+    error_report("vhost-net support is not compiled in");
     return NULL;
 }