On 01/11/2010 11:22 AM, Michael S. Tsirkin wrote:
Looks like order got mixed up: vhost_net header is added by a follow-up patch. Will be fixed in the next revision.Signed-off-by: Michael S. Tsirkin<m...@redhat.com> --- net.c | 8 ++++++++ net/tap.c | 29 +++++++++++++++++++++++++++++ qemu-options.hx | 4 +++- 3 files changed, 40 insertions(+), 1 deletions(-) diff --git a/net.c b/net.c index 6ef93e6..b942d03 100644 --- a/net.c +++ b/net.c @@ -976,6 +976,14 @@ static struct { .name = "vnet_hdr", .type = QEMU_OPT_BOOL, .help = "enable the IFF_VNET_HDR flag on the tap interface" + }, { + .name = "vhost", + .type = QEMU_OPT_BOOL, + .help = "enable vhost-net network accelerator", + }, { + .name = "vhostfd", + .type = QEMU_OPT_STRING, + .help = "file descriptor of an already opened vhost net device", },
Semantically, I think making vhost it's own backend makes more sense from a user perspective.
I don't think it's a significant code change. Regards, Anthony Liguori
