[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 07/16] net: use qtailq for vlan and client lists
From: |
Mark McLoughlin |
Subject: |
[Qemu-devel] [PATCH 07/16] net: use qtailq for vlan and client lists |
Date: |
Thu, 8 Oct 2009 19:58:23 +0100 |
Signed-off-by: Mark McLoughlin <address@hidden>
---
net.c | 116 +++++++++++++++++++++++++++++++-------------------------------
net.h | 6 ++--
savevm.c | 2 +-
3 files changed, 62 insertions(+), 62 deletions(-)
diff --git a/net.c b/net.c
index 87dcf06..35547a5 100644
--- a/net.c
+++ b/net.c
@@ -113,10 +113,8 @@
#include "qemu-config.h"
#include "slirp/libslirp.h"
-#include "qemu-queue.h"
-
-static VLANState *first_vlan;
+static QTAILQ_HEAD(, VLANState) vlans;
/***********************************************************/
/* network device redirectors */
@@ -287,12 +285,14 @@ static char *assign_name(VLANClientState *vc1, const char
*model)
char buf[256];
int id = 0;
- for (vlan = first_vlan; vlan; vlan = vlan->next) {
+ QTAILQ_FOREACH(vlan, &vlans, next) {
VLANClientState *vc;
- for (vc = vlan->first_client; vc; vc = vc->next)
- if (vc != vc1 && strcmp(vc->model, model) == 0)
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
+ if (vc != vc1 && strcmp(vc->model, model) == 0) {
id++;
+ }
+ }
}
snprintf(buf, sizeof(buf), "%s.%d", model, id);
@@ -309,8 +309,10 @@ VLANClientState *qemu_new_vlan_client(VLANState *vlan,
NetCleanup *cleanup,
void *opaque)
{
- VLANClientState *vc, **pvc;
+ VLANClientState *vc;
+
vc = qemu_mallocz(sizeof(VLANClientState));
+
vc->model = qemu_strdup(model);
if (name)
vc->name = qemu_strdup(name);
@@ -321,43 +323,35 @@ VLANClientState *qemu_new_vlan_client(VLANState *vlan,
vc->receive_iov = receive_iov;
vc->cleanup = cleanup;
vc->opaque = opaque;
+
vc->vlan = vlan;
+ QTAILQ_INSERT_TAIL(&vc->vlan->clients, vc, next);
- vc->next = NULL;
- pvc = &vlan->first_client;
- while (*pvc != NULL)
- pvc = &(*pvc)->next;
- *pvc = vc;
return vc;
}
void qemu_del_vlan_client(VLANClientState *vc)
{
- VLANClientState **pvc = &vc->vlan->first_client;
+ QTAILQ_REMOVE(&vc->vlan->clients, vc, next);
- while (*pvc != NULL)
- if (*pvc == vc) {
- *pvc = vc->next;
- if (vc->cleanup) {
- vc->cleanup(vc);
- }
- qemu_free(vc->name);
- qemu_free(vc->model);
- qemu_free(vc);
- break;
- } else
- pvc = &(*pvc)->next;
+ if (vc->cleanup) {
+ vc->cleanup(vc);
+ }
+
+ qemu_free(vc->name);
+ qemu_free(vc->model);
+ qemu_free(vc);
}
VLANClientState *qemu_find_vlan_client(VLANState *vlan, void *opaque)
{
- VLANClientState **pvc = &vlan->first_client;
+ VLANClientState *vc;
- while (*pvc != NULL)
- if ((*pvc)->opaque == opaque)
- return *pvc;
- else
- pvc = &(*pvc)->next;
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
+ if (vc->opaque == opaque) {
+ return vc;
+ }
+ }
return NULL;
}
@@ -375,7 +369,7 @@ qemu_find_vlan_client_by_name(Monitor *mon, int vlan_id,
return NULL;
}
- for (vc = vlan->first_client; vc != NULL; vc = vc->next) {
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
if (!strcmp(vc->name, client_str)) {
break;
}
@@ -393,7 +387,7 @@ int qemu_can_send_packet(VLANClientState *sender)
VLANState *vlan = sender->vlan;
VLANClientState *vc;
- for (vc = vlan->first_client; vc != NULL; vc = vc->next) {
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
if (vc == sender) {
continue;
}
@@ -414,7 +408,7 @@ qemu_deliver_packet(VLANClientState *sender, const uint8_t
*buf, int size)
sender->vlan->delivering = 1;
- for (vc = sender->vlan->first_client; vc != NULL; vc = vc->next) {
+ QTAILQ_FOREACH(vc, &sender->vlan->clients, next) {
ssize_t len;
if (vc == sender) {
@@ -557,7 +551,7 @@ static int qemu_deliver_packet_iov(VLANClientState *sender,
sender->vlan->delivering = 1;
- for (vc = sender->vlan->first_client; vc != NULL; vc = vc->next) {
+ QTAILQ_FOREACH(vc, &sender->vlan->clients, next) {
ssize_t len;
if (vc == sender) {
@@ -2305,22 +2299,25 @@ static int net_dump_init(VLANState *vlan, const char
*device,
/* find or alloc a new VLAN */
VLANState *qemu_find_vlan(int id, int allocate)
{
- VLANState **pvlan, *vlan;
- for(vlan = first_vlan; vlan != NULL; vlan = vlan->next) {
- if (vlan->id == id)
+ VLANState *vlan;
+
+ QTAILQ_FOREACH(vlan, &vlans, next) {
+ if (vlan->id == id) {
return vlan;
+ }
}
+
if (!allocate) {
return NULL;
}
+
vlan = qemu_mallocz(sizeof(VLANState));
vlan->id = id;
+ QTAILQ_INIT(&vlan->clients);
QTAILQ_INIT(&vlan->send_queue);
- vlan->next = NULL;
- pvlan = &first_vlan;
- while (*pvlan != NULL)
- pvlan = &(*pvlan)->next;
- *pvlan = vlan;
+
+ QTAILQ_INSERT_TAIL(&vlans, vlan, next);
+
return vlan;
}
@@ -3118,12 +3115,15 @@ void net_set_boot_mask(int net_boot_mask)
void do_info_network(Monitor *mon)
{
VLANState *vlan;
- VLANClientState *vc;
- for(vlan = first_vlan; vlan != NULL; vlan = vlan->next) {
+ QTAILQ_FOREACH(vlan, &vlans, next) {
+ VLANClientState *vc;
+
monitor_printf(mon, "VLAN %d devices:\n", vlan->id);
- for(vc = vlan->first_client; vc != NULL; vc = vc->next)
+
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
monitor_printf(mon, " %s: %s\n", vc->name, vc->info_str);
+ }
}
}
@@ -3134,10 +3134,13 @@ void do_set_link(Monitor *mon, const QDict *qdict)
const char *name = qdict_get_str(qdict, "name");
const char *up_or_down = qdict_get_str(qdict, "up_or_down");
- for (vlan = first_vlan; vlan != NULL; vlan = vlan->next)
- for (vc = vlan->first_client; vc != NULL; vc = vc->next)
- if (strcmp(vc->name, name) == 0)
+ QTAILQ_FOREACH(vlan, &vlans, next) {
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
+ if (strcmp(vc->name, name) == 0) {
goto done;
+ }
+ }
+ }
done:
if (!vc) {
@@ -3161,16 +3164,11 @@ void net_cleanup(void)
{
VLANState *vlan;
- /* close network clients */
- for(vlan = first_vlan; vlan != NULL; vlan = vlan->next) {
- VLANClientState *vc = vlan->first_client;
-
- while (vc) {
- VLANClientState *next = vc->next;
+ QTAILQ_FOREACH(vlan, &vlans, next) {
+ VLANClientState *vc, *next_vc;
+ QTAILQ_FOREACH_SAFE(vc, &vlan->clients, next, next_vc) {
qemu_del_vlan_client(vc);
-
- vc = next;
}
}
}
@@ -3179,7 +3177,7 @@ static void net_check_clients(void)
{
VLANState *vlan;
- for(vlan = first_vlan; vlan != NULL; vlan = vlan->next) {
+ QTAILQ_FOREACH(vlan, &vlans, next) {
if (vlan->nb_guest_devs == 0 && vlan->nb_host_devs == 0)
continue;
if (vlan->nb_guest_devs == 0)
@@ -3206,6 +3204,8 @@ int net_init_clients(void)
#endif
}
+ QTAILQ_INIT(&vlans);
+
if (qemu_opts_foreach(&qemu_net_opts, net_init_client, NULL, 1) == -1) {
return -1;
}
diff --git a/net.h b/net.h
index 164ea3d..79e9b47 100644
--- a/net.h
+++ b/net.h
@@ -26,7 +26,7 @@ struct VLANClientState {
LinkStatusChanged *link_status_changed;
int link_down;
void *opaque;
- struct VLANClientState *next;
+ QTAILQ_ENTRY(VLANClientState) next;
struct VLANState *vlan;
char *model;
char *name;
@@ -47,8 +47,8 @@ struct VLANPacket {
struct VLANState {
int id;
- VLANClientState *first_client;
- struct VLANState *next;
+ QTAILQ_HEAD(, VLANClientState) clients;
+ QTAILQ_ENTRY(VLANState) next;
unsigned int nb_guest_devs, nb_host_devs;
QTAILQ_HEAD(send_queue, VLANPacket) send_queue;
int delivering;
diff --git a/savevm.c b/savevm.c
index 7a363b6..27a7686 100644
--- a/savevm.c
+++ b/savevm.c
@@ -131,7 +131,7 @@ static void qemu_announce_self_once(void *opaque)
continue;
len = announce_self_create(buf, nd_table[i].macaddr);
vlan = nd_table[i].vlan;
- for(vc = vlan->first_client; vc != NULL; vc = vc->next) {
+ QTAILQ_FOREACH(vc, &vlan->clients, next) {
vc->receive(vc, buf, len);
}
}
--
1.6.2.5
- [Qemu-devel] [PATCH 00/16] Add a -netdev option, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 04/16] net: remove unused qemu_handler_true(), Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 09/16] net: add QemuOptsList arg to net_client_parse(), Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 10/16] net: add -netdev option, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 07/16] net: use qtailq for vlan and client lists,
Mark McLoughlin <=
- [Qemu-devel] [PATCH 01/16] slirp: fix !CONFIG_SLIRP compilation, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 03/16] net: pass monitor handle to client init functions, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 15/16] net: refactor packet queueing code, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 06/16] net: remove id field from NICInfo, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 05/16] net: handle id= parameter for -net, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 08/16] net: allow clients not associated with a vlan, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 13/16] net: add -net nic,netdev= option, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 14/16] net: allow NICs to be connected to netdevs, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 11/16] net: handle -netdevice options, Mark McLoughlin, 2009/10/08
- [Qemu-devel] [PATCH 12/16] net: maintain a list of vlan-less clients, Mark McLoughlin, 2009/10/08