[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/16] net: Use hubs for the vlan feature
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 02/16] net: Use hubs for the vlan feature |
Date: |
Mon, 23 Jul 2012 15:55:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.6esrpre) Gecko/20120714 Thunderbird/10.0.6 |
On 07/20/12 14:01, Stefan Hajnoczi wrote:
> diff --git a/net.c b/net.c
> index e7a8d81..f2752c7 100644
> --- a/net.c
> +++ b/net.c
> @@ -25,6 +25,7 @@
>
> #include "config-host.h"
>
> +#include "net/hub.h"
> #include "net/tap.h"
> #include "net/socket.h"
> #include "net/dump.h"
This includes "net/hub.h" for the second time in "net.c" (still true at
the end of the series). Causes no problems, of course.
> @@ -876,17 +879,17 @@ static int net_client_init1(const void *object, int
> is_netdev, Error **errp)
> }
>
> if (net_client_init_fun[opts->kind]) {
> - VLANState *vlan = NULL;
> + VLANClientState *peer = NULL;
>
> /* Do not add to a vlan if it's a -netdev or a nic with a netdev=
> * parameter. */
> if (!is_netdev &&
> (opts->kind != NET_CLIENT_OPTIONS_KIND_NIC ||
> !opts->nic->has_netdev)) {
> - vlan = qemu_find_vlan(u.net->has_vlan ? u.net->vlan : 0, true);
> + peer = net_hub_add_port(u.net->has_vlan ? u.net->vlan : 0, NULL);
> }
"u.net->vlan" is int32_t ('int32' in the schema
<http://lists.nongnu.org/archive/html/qemu-devel/2012-06/msg01675.html>);
net_hub_add_port()
expects unsigned.
> diff --git a/net/dump.h b/net/dump.h
> index 0fa2dd7..a1beb0d 100644
> --- a/net/dump.h
> +++ b/net/dump.h
> @@ -28,6 +28,6 @@
> #include "qapi-types.h"
>
> int net_init_dump(const NetClientOptions *opts, const char *name,
> - VLANState *vlan);
> + VLANClientState *vlan);
>
> #endif /* QEMU_NET_DUMP_H */
Should be called "peer", but it will be renamed later on. Same for
"net/slirp.h".
Laszlo
- Re: [Qemu-devel] [PATCH 08/16] net: Remove VLANState, (continued)
- [Qemu-devel] [PATCH 14/16] net: cleanup deliver/deliver_iov func pointers, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 07/16] net: Remove vlan code from net.c, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 06/16] net: Convert qdev_prop_vlan to peer with hub, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 11/16] net: Rename vc local variables to nc, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 16/16] hub: add the support for hub own flow control, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 02/16] net: Use hubs for the vlan feature, Stefan Hajnoczi, 2012/07/20
- Re: [Qemu-devel] [PATCH 02/16] net: Use hubs for the vlan feature,
Laszlo Ersek <=
- [Qemu-devel] [PATCH 09/16] net: Rename non_vlan_clients to net_clients, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 13/16] net: Make "info network" output more readable info, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 12/16] net: Rename qemu_del_vlan_client() to qemu_del_net_client(), Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 04/16] hub: Check that hubs are configured correctly, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 01/16] net: Add a hub net client, Stefan Hajnoczi, 2012/07/20