[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/6] Add new vhost-user netdev backend
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 6/6] Add new vhost-user netdev backend |
Date: |
Tue, 10 Dec 2013 15:29:35 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 |
On 12/09/2013 05:25 PM, Antonios Motakis wrote:
> Add a new QEMU netdev backend that is intended to invoke vhost_net
> with the vhost-user backend. Also decouple virtio-net from the tap
> backend.
>
> Signed-off-by: Antonios Motakis <address@hidden>
> Signed-off-by: Nikolay Nikolaev <address@hidden>
> ---
> +++ b/include/net/vhost-user.h
> @@ -0,0 +1,17 @@
> +/*
> + * vhost-user.h
> + *
> + * Copyright (c) 2013 Virtual Open Systems Sarl.
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2. See
> + * the COPYING file in the top-level directory.
Can you please use GPLv2+ (that is, add the "or later" clause)? Yes, we
already have GPLv2-only files, but I'd like to avoid adding even more of
them.
> +++ b/net/vhost-user.c
> @@ -0,0 +1,95 @@
> +/*
> + * vhost-user.c
> + *
> + * Copyright (c) 2013 Virtual Open Systems Sarl.
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2. See
> + * the COPYING file in the top-level directory.
Same question applies to all new files added throughout this series.
> +++ b/qapi-schema.json
> @@ -3009,11 +3009,24 @@
> 'hubid': 'int32' } }
>
> ##
> +# @NetdevVhostUserOptions
> +#
> +# Vhost-user network backend
> +#
> +# @file: control socket path
What does it mean when 'file' is not present? Is there a default value?
Normally, we mark '#optional' in the docs for an optional argument.
> +#
> +# Since 2.0
> +##
> +{ 'type': 'NetdevVhostUserOptions',
> + 'data': {
> + '*file': 'str' } }
Or is file always present, in which case this should be 'file' instead
of '*file'?
> +
> +##
> # @NetClientOptions
> #
> # A discriminated record of network device traits.
> #
> -# Since 1.2
> +# Since 2.0
Wrong. NetClientOptions has existed since 1.2; but some of the branches
of the union are newer. The way we have documented that elsewhere looks
more like:
# A discriminated record of network device traits.
# @vde: traits for VDE
# @dump: traits when using the device to dump all traffic
# @bridge: traits for a bridge device
# @hubport: traits for a hub port
# @vhost-user: traits for a vhost-user (since 2.0)
#
# Since 1.2
> ##
> { 'union': 'NetClientOptions',
> 'data': {
> @@ -3025,7 +3038,8 @@
> 'vde': 'NetdevVdeOptions',
> 'dump': 'NetdevDumpOptions',
> 'bridge': 'NetdevBridgeOptions',
> - 'hubport': 'NetdevHubPortOptions' } }
> + 'hubport': 'NetdevHubPortOptions',
> + 'vhost-user': 'NetdevVhostUserOptions' } }
>
> ##
> # @NetLegacy
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCHv2 0/6] Vhost and vhost-net support for userspace based backends, Antonios Motakis, 2013/12/09
- [Qemu-devel] [PATCH 1/6] Add -mem-share option, Antonios Motakis, 2013/12/09
- [Qemu-devel] [PATCH 2/6] Decouple vhost from kernel interface, Antonios Motakis, 2013/12/09
- [Qemu-devel] [PATCH 3/6] Add vhost-user skeleton, Antonios Motakis, 2013/12/09
- [Qemu-devel] [PATCH 4/6] Add domain socket communication for vhost-user backend, Antonios Motakis, 2013/12/09
- [Qemu-devel] [PATCH 5/6] Add vhost-user calls implementation, Antonios Motakis, 2013/12/09
- [Qemu-devel] [PATCH 6/6] Add new vhost-user netdev backend, Antonios Motakis, 2013/12/09
- Re: [Qemu-devel] [PATCH 6/6] Add new vhost-user netdev backend,
Eric Blake <=