Message ID | 1331283625-26439-10-git-send-email-zwu.kernel@gmail.com |
---|---|
State | New |
Headers | show |
On 03/09/2012 05:00 PM, zwu.kernel@gmail.com wrote: > From: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> > > There is no longer a distinction between vlan clients and non-vlan > clients in the net core. The net core only knows about point-to-point > clients which are connected to a peer. It's time to rename the global > list of net clients since it no longer refers to vlans at all. > > Signed-off-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> > Signed-off-by: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com> > --- > net.c | 20 ++++++++++---------- > 1 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/net.c b/net.c > index e09b7a7..a510e27 100644 > --- a/net.c > +++ b/net.c > @@ -44,7 +44,7 @@ > # define CONFIG_NET_BRIDGE > #endif > > -static QTAILQ_HEAD(, VLANClientState) non_vlan_clients; > +static QTAILQ_HEAD(, VLANClientState) net_clients; > > int default_net = 1; > > @@ -165,7 +165,7 @@ static char *assign_name(VLANClientState *vc1, const char *model) > char buf[256]; > int id = 0; > > - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { > + QTAILQ_FOREACH(vc,&net_clients, next) { > if (vc == vc1) { > continue; > } > @@ -216,7 +216,7 @@ VLANClientState *qemu_new_net_client(NetClientInfo *info, > vc->peer = peer; > peer->peer = vc; > } > - QTAILQ_INSERT_TAIL(&non_vlan_clients, vc, next); > + QTAILQ_INSERT_TAIL(&net_clients, vc, next); > > vc->send_queue = qemu_new_net_queue(qemu_deliver_packet, > qemu_deliver_packet_iov, > @@ -248,7 +248,7 @@ NICState *qemu_new_nic(NetClientInfo *info, > > static void qemu_cleanup_vlan_client(VLANClientState *vc) > { > - QTAILQ_REMOVE(&non_vlan_clients, vc, next); > + QTAILQ_REMOVE(&net_clients, vc, next); > > if (vc->info->cleanup) { > vc->info->cleanup(vc); > @@ -302,7 +302,7 @@ void qemu_foreach_nic(qemu_nic_foreach func, void *opaque) > { > VLANClientState *nc; > > - QTAILQ_FOREACH(nc,&non_vlan_clients, next) { > + QTAILQ_FOREACH(nc,&net_clients, next) { > if (nc->info->type == NET_CLIENT_TYPE_NIC) { > func(DO_UPCAST(NICState, nc, nc), opaque); > } > @@ -467,7 +467,7 @@ VLANClientState *qemu_find_netdev(const char *id) > { > VLANClientState *vc; > > - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { > + QTAILQ_FOREACH(vc,&net_clients, next) { > if (vc->info->type == NET_CLIENT_TYPE_NIC) > continue; > if (!strcmp(vc->name, id)) { > @@ -1080,7 +1080,7 @@ void do_info_network(Monitor *mon) > net_client_type type; > > monitor_printf(mon, "Devices not on any VLAN:\n"); ^^^^^^^^^^^^ I think we should remove the line above. The output of "info network" (qemu) info network Devices not on any VLAN: e1000.0: type=nic,model=e1000,macaddr=52:54:00:12:34:56 \ hub0port1: type=(null), hub 0 port 1 peer e1000.0 port 0 peer bridge.0 The original output: (qemu) info network VLAN 0 devices: bridge.0: type=tap,helper=./qemu-bridge-helper,br=virbr0 e1000.0: type=nic,model=e1000,macaddr=52:54:00:12:34:56 Devices not on any VLAN: > - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { > + QTAILQ_FOREACH(vc,&net_clients, next) { > peer = vc->peer; > type = vc->info->type; > if (!peer || type == NET_CLIENT_TYPE_NIC) { > @@ -1133,7 +1133,7 @@ void net_cleanup(void) > { > VLANClientState *vc, *next_vc; > > - QTAILQ_FOREACH_SAFE(vc,&non_vlan_clients, next, next_vc) { > + QTAILQ_FOREACH_SAFE(vc,&net_clients, next, next_vc) { > qemu_del_vlan_client(vc); > } > } > @@ -1157,7 +1157,7 @@ void net_check_clients(void) > > net_hub_check_clients(); > > - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { > + QTAILQ_FOREACH(vc,&net_clients, next) { > if (!vc->peer) { > fprintf(stderr, "Warning: %s %s has no peer\n", > vc->info->type == NET_CLIENT_TYPE_NIC ? "nic" : "netdev", > @@ -1204,7 +1204,7 @@ int net_init_clients(void) > #endif > } > > - QTAILQ_INIT(&non_vlan_clients); > + QTAILQ_INIT(&net_clients); > > if (qemu_opts_foreach(qemu_find_opts("netdev"), net_init_netdev, NULL, 1) == -1) > return -1;
On Tue, Mar 13, 2012 at 5:23 PM, Mark Wu <wudxw@linux.vnet.ibm.com> wrote: > On 03/09/2012 05:00 PM, zwu.kernel@gmail.com wrote: >> >> From: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> >> >> There is no longer a distinction between vlan clients and non-vlan >> clients in the net core. The net core only knows about point-to-point >> clients which are connected to a peer. It's time to rename the global >> list of net clients since it no longer refers to vlans at all. >> >> Signed-off-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> >> Signed-off-by: Zhi Yong Wu<wuzhy@linux.vnet.ibm.com> >> --- >> net.c | 20 ++++++++++---------- >> 1 files changed, 10 insertions(+), 10 deletions(-) >> >> diff --git a/net.c b/net.c >> index e09b7a7..a510e27 100644 >> --- a/net.c >> +++ b/net.c >> @@ -44,7 +44,7 @@ >> # define CONFIG_NET_BRIDGE >> #endif >> >> -static QTAILQ_HEAD(, VLANClientState) non_vlan_clients; >> +static QTAILQ_HEAD(, VLANClientState) net_clients; >> >> int default_net = 1; >> >> @@ -165,7 +165,7 @@ static char *assign_name(VLANClientState *vc1, const >> char *model) >> char buf[256]; >> int id = 0; >> >> - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { >> + QTAILQ_FOREACH(vc,&net_clients, next) { >> >> if (vc == vc1) { >> continue; >> } >> @@ -216,7 +216,7 @@ VLANClientState *qemu_new_net_client(NetClientInfo >> *info, >> vc->peer = peer; >> peer->peer = vc; >> } >> - QTAILQ_INSERT_TAIL(&non_vlan_clients, vc, next); >> + QTAILQ_INSERT_TAIL(&net_clients, vc, next); >> >> vc->send_queue = qemu_new_net_queue(qemu_deliver_packet, >> qemu_deliver_packet_iov, >> @@ -248,7 +248,7 @@ NICState *qemu_new_nic(NetClientInfo *info, >> >> static void qemu_cleanup_vlan_client(VLANClientState *vc) >> { >> - QTAILQ_REMOVE(&non_vlan_clients, vc, next); >> + QTAILQ_REMOVE(&net_clients, vc, next); >> >> if (vc->info->cleanup) { >> vc->info->cleanup(vc); >> @@ -302,7 +302,7 @@ void qemu_foreach_nic(qemu_nic_foreach func, void >> *opaque) >> { >> VLANClientState *nc; >> >> - QTAILQ_FOREACH(nc,&non_vlan_clients, next) { >> + QTAILQ_FOREACH(nc,&net_clients, next) { >> >> if (nc->info->type == NET_CLIENT_TYPE_NIC) { >> func(DO_UPCAST(NICState, nc, nc), opaque); >> } >> @@ -467,7 +467,7 @@ VLANClientState *qemu_find_netdev(const char *id) >> { >> VLANClientState *vc; >> >> - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { >> + QTAILQ_FOREACH(vc,&net_clients, next) { >> >> if (vc->info->type == NET_CLIENT_TYPE_NIC) >> continue; >> if (!strcmp(vc->name, id)) { >> @@ -1080,7 +1080,7 @@ void do_info_network(Monitor *mon) >> net_client_type type; >> >> monitor_printf(mon, "Devices not on any VLAN:\n"); > > ^^^^^^^^^^^^ > I think we should remove the line above. Yeah, actually we have noticed this. > > The output of "info network" > (qemu) info network > > Devices not on any VLAN: > e1000.0: type=nic,model=e1000,macaddr=52:54:00:12:34:56 > \ hub0port1: type=(null), > hub 0 > port 1 peer e1000.0 > port 0 peer bridge.0 > > The original output: > (qemu) info network > VLAN 0 devices: > bridge.0: type=tap,helper=./qemu-bridge-helper,br=virbr0 > e1000.0: type=nic,model=e1000,macaddr=52:54:00:12:34:56 > > Devices not on any VLAN: > >> - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { >> + QTAILQ_FOREACH(vc,&net_clients, next) { >> >> peer = vc->peer; >> type = vc->info->type; >> if (!peer || type == NET_CLIENT_TYPE_NIC) { >> @@ -1133,7 +1133,7 @@ void net_cleanup(void) >> { >> VLANClientState *vc, *next_vc; >> >> - QTAILQ_FOREACH_SAFE(vc,&non_vlan_clients, next, next_vc) { >> + QTAILQ_FOREACH_SAFE(vc,&net_clients, next, next_vc) { >> >> qemu_del_vlan_client(vc); >> } >> } >> @@ -1157,7 +1157,7 @@ void net_check_clients(void) >> >> net_hub_check_clients(); >> >> - QTAILQ_FOREACH(vc,&non_vlan_clients, next) { >> + QTAILQ_FOREACH(vc,&net_clients, next) { >> >> if (!vc->peer) { >> fprintf(stderr, "Warning: %s %s has no peer\n", >> vc->info->type == NET_CLIENT_TYPE_NIC ? "nic" : >> "netdev", >> @@ -1204,7 +1204,7 @@ int net_init_clients(void) >> #endif >> } >> >> - QTAILQ_INIT(&non_vlan_clients); >> + QTAILQ_INIT(&net_clients); >> >> if (qemu_opts_foreach(qemu_find_opts("netdev"), net_init_netdev, >> NULL, 1) == -1) >> return -1; > > >
diff --git a/net.c b/net.c index e09b7a7..a510e27 100644 --- a/net.c +++ b/net.c @@ -44,7 +44,7 @@ # define CONFIG_NET_BRIDGE #endif -static QTAILQ_HEAD(, VLANClientState) non_vlan_clients; +static QTAILQ_HEAD(, VLANClientState) net_clients; int default_net = 1; @@ -165,7 +165,7 @@ static char *assign_name(VLANClientState *vc1, const char *model) char buf[256]; int id = 0; - QTAILQ_FOREACH(vc, &non_vlan_clients, next) { + QTAILQ_FOREACH(vc, &net_clients, next) { if (vc == vc1) { continue; } @@ -216,7 +216,7 @@ VLANClientState *qemu_new_net_client(NetClientInfo *info, vc->peer = peer; peer->peer = vc; } - QTAILQ_INSERT_TAIL(&non_vlan_clients, vc, next); + QTAILQ_INSERT_TAIL(&net_clients, vc, next); vc->send_queue = qemu_new_net_queue(qemu_deliver_packet, qemu_deliver_packet_iov, @@ -248,7 +248,7 @@ NICState *qemu_new_nic(NetClientInfo *info, static void qemu_cleanup_vlan_client(VLANClientState *vc) { - QTAILQ_REMOVE(&non_vlan_clients, vc, next); + QTAILQ_REMOVE(&net_clients, vc, next); if (vc->info->cleanup) { vc->info->cleanup(vc); @@ -302,7 +302,7 @@ void qemu_foreach_nic(qemu_nic_foreach func, void *opaque) { VLANClientState *nc; - QTAILQ_FOREACH(nc, &non_vlan_clients, next) { + QTAILQ_FOREACH(nc, &net_clients, next) { if (nc->info->type == NET_CLIENT_TYPE_NIC) { func(DO_UPCAST(NICState, nc, nc), opaque); } @@ -467,7 +467,7 @@ VLANClientState *qemu_find_netdev(const char *id) { VLANClientState *vc; - QTAILQ_FOREACH(vc, &non_vlan_clients, next) { + QTAILQ_FOREACH(vc, &net_clients, next) { if (vc->info->type == NET_CLIENT_TYPE_NIC) continue; if (!strcmp(vc->name, id)) { @@ -1080,7 +1080,7 @@ void do_info_network(Monitor *mon) net_client_type type; monitor_printf(mon, "Devices not on any VLAN:\n"); - QTAILQ_FOREACH(vc, &non_vlan_clients, next) { + QTAILQ_FOREACH(vc, &net_clients, next) { peer = vc->peer; type = vc->info->type; if (!peer || type == NET_CLIENT_TYPE_NIC) { @@ -1133,7 +1133,7 @@ void net_cleanup(void) { VLANClientState *vc, *next_vc; - QTAILQ_FOREACH_SAFE(vc, &non_vlan_clients, next, next_vc) { + QTAILQ_FOREACH_SAFE(vc, &net_clients, next, next_vc) { qemu_del_vlan_client(vc); } } @@ -1157,7 +1157,7 @@ void net_check_clients(void) net_hub_check_clients(); - QTAILQ_FOREACH(vc, &non_vlan_clients, next) { + QTAILQ_FOREACH(vc, &net_clients, next) { if (!vc->peer) { fprintf(stderr, "Warning: %s %s has no peer\n", vc->info->type == NET_CLIENT_TYPE_NIC ? "nic" : "netdev", @@ -1204,7 +1204,7 @@ int net_init_clients(void) #endif } - QTAILQ_INIT(&non_vlan_clients); + QTAILQ_INIT(&net_clients); if (qemu_opts_foreach(qemu_find_opts("netdev"), net_init_netdev, NULL, 1) == -1) return -1;