[libvirt] [PATCHv2 2/4] util: add virNetDevVlanType

Laine Stump laine at laine.org
Tue Aug 14 07:15:22 UTC 2012


To allow for the possibility of vlan "trunks", which have more than
one vlan tag associated with them, we need a vlan struct. Since it
will be used by multiple files in src/util, src/conf, src/network, and
src/qemu, it must be defined in src/util. Unfortunately there isn't
currently a common file for simple netdev data definitions, so I
created a new file.
---
 src/Makefile.am          |  1 +
 src/libvirt_private.syms |  6 ++++
 src/util/virnetdevvlan.c | 88 ++++++++++++++++++++++++++++++++++++++++++++++++
 src/util/virnetdevvlan.h | 38 +++++++++++++++++++++
 4 files changed, 133 insertions(+)
 create mode 100644 src/util/virnetdevvlan.c
 create mode 100644 src/util/virnetdevvlan.h

diff --git a/src/Makefile.am b/src/Makefile.am
index 8a93a32..840bb82 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -104,6 +104,7 @@ UTIL_SOURCES =							\
 		util/virnetdevopenvswitch.h util/virnetdevopenvswitch.c \
 		util/virnetdevtap.h util/virnetdevtap.c		\
 		util/virnetdevveth.h util/virnetdevveth.c \
+		util/virnetdevvlan.h util/virnetdevvlan.c \
 		util/virnetdevvportprofile.h util/virnetdevvportprofile.c \
 		util/virnetlink.c util/virnetlink.h		\
 		util/virrandom.h util/virrandom.c		\
diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 89fb1f4..d20f9e3 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -1425,6 +1425,12 @@ virNetDevVethCreate;
 virNetDevVethDelete;
 
 
+# virnetdevvlan.h
+virNetDevVlanClear;
+virNetDevVlanCopy;
+virNetDevVlanEqual;
+virNetDevVlanFree;
+
 # virnetdevvportprofile.h
 virNetDevVPortProfileAssociate;
 virNetDevVPortProfileCheckComplete;
diff --git a/src/util/virnetdevvlan.c b/src/util/virnetdevvlan.c
new file mode 100644
index 0000000..d66902f
--- /dev/null
+++ b/src/util/virnetdevvlan.c
@@ -0,0 +1,88 @@
+/*
+ * Copyright (C) 2009-2012 Red Hat, Inc.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library;  If not, see
+ * <http://www.gnu.org/licenses/>.
+ *
+ * Authors:
+ *      Laine Stump <laine at redhat.com>
+ */
+
+#include <config.h>
+
+#include "internal.h"
+#include "virterror_internal.h"
+#include "virnetdevvlan.h"
+#include "memory.h"
+
+#define VIR_FROM_THIS VIR_FROM_NONE
+
+void
+virNetDevVlanClear(virNetDevVlanPtr vlan)
+{
+    VIR_FREE(vlan->tag);
+    vlan->nTags = 0;
+}
+
+void
+virNetDevVlanFree(virNetDevVlanPtr vlan)
+{
+    if (vlan)
+        virNetDevVlanClear(vlan);
+    VIR_FREE(vlan);
+}
+
+int
+virNetDevVlanEqual(const virNetDevVlanPtr a, const virNetDevVlanPtr b)
+{
+    int ii;
+
+    if (!(a || b))
+        return true;
+    if (!a || !b)
+        return false;
+
+    if (a->trunk != b->trunk ||
+        a->nTags != b->nTags) {
+        return false;
+    }
+
+    for (ii = 0; ii < a->nTags; ii++) {
+        if (a->tag[ii] != b->tag[ii])
+            return false;
+    }
+    return true;
+}
+
+/*
+ * virNetDevVlanCopy - copy from src into (already existing) dst.
+ *                     If src is NULL, dst will have nTags set to 0.
+ *                     dst is assumed to be empty on entry.
+ */
+int
+virNetDevVlanCopy(virNetDevVlanPtr dst, const virNetDevVlanPtr src)
+{
+    if (!src || src->nTags == 0)
+        return 0;
+
+    if (VIR_ALLOC_N(dst->tag, src->nTags) < 0) {
+        virReportOOMError();
+        return -1;
+    }
+
+    dst->trunk = src->trunk;
+    dst->nTags = src->nTags;
+    memmove(dst->tag, src->tag, src->nTags * sizeof(*src->tag));
+    return 0;
+}
diff --git a/src/util/virnetdevvlan.h b/src/util/virnetdevvlan.h
new file mode 100644
index 0000000..7a046f9
--- /dev/null
+++ b/src/util/virnetdevvlan.h
@@ -0,0 +1,38 @@
+/*
+ * Copyright (C) 2009-2012 Red Hat, Inc.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library;  If not, see
+ * <http://www.gnu.org/licenses/>.
+ *
+ * Authors:
+ *      Laine Stump <laine at redhat.com>
+ */
+
+#ifndef __VIR_NETDEV_VLAN_H__
+# define __VIR_NETDEV_VLAN_H__
+
+typedef struct _virNetDevVlan virNetDevVlan;
+typedef virNetDevVlan *virNetDevVlanPtr;
+struct _virNetDevVlan {
+    bool trunk;        /* true if this is a trunk */
+    int nTags;          /* number of tags in array */
+    unsigned int *tag; /* pointer to array of tags */
+};
+
+void virNetDevVlanClear(virNetDevVlanPtr vlan);
+void virNetDevVlanFree(virNetDevVlanPtr vlan);
+int virNetDevVlanEqual(const virNetDevVlanPtr a, const virNetDevVlanPtr b);
+int virNetDevVlanCopy(virNetDevVlanPtr dst, const virNetDevVlanPtr src);
+
+#endif /* __VIR_NETDEV_VLAN_H__ */
-- 
1.7.11.2




More information about the libvir-list mailing list