Re: [libvirt] [PATCH 4/7] Ensure GET_VLAN_VID_CMD is always defined
On 03/07/13 17:41, Daniel P. Berrange wrote:
From: "Daniel P. Berrange" <berrange(a)redhat.com>
The RHEL4 vintage header files do not define GET_VLAN_VID_CMD.
Conditionally define it in our source, since the kernel can
raise a runtime error if it isn't supported
Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com>
---
src/util/virnetdev.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c
index 296871c..4cdbb0c 100644
--- a/src/util/virnetdev.c
+++ b/src/util/virnetdev.c
@@ -673,6 +673,9 @@ int virNetDevGetIndex(const char *ifname ATTRIBUTE_UNUSED,
#if defined(SIOCGIFVLAN) && defined(HAVE_STRUCT_IFREQ)
+# ifndef GET_VLAN_VID_CMD
+# define GET_VLAN_VID_CMD 9
+# endif
int virNetDevGetVLanID(const char *ifname, int *vlanid)
{
struct vlan_ioctl_args vlanargs = {
As you are attempting just fixing of the client it might be
feasible to do:
#if defined(SIOCGIFVLAN) && defined(HAVE_STRUCT_IFREQ) &&
defined(GET_VLAN_VID_CMD)
and let the compiler compile the fallback function that
probably won't be used anyways.
Additionally if somebody might want to fix the qemu driver
to work too, lack of vlan support won't be on the top of the
priority list.
Peter