[ovs-dev] [PATCH] netlink-protocol: Define missing symbols.
Ethan Jackson
ethan at nicira.com
Fri Sep 2 18:59:27 UTC 2011
I'm fine with that. Wasn't sure what's the stylistically correct way
to implement a terrible hack. I'll resend it.
Ethan
On Fri, Sep 2, 2011 at 11:56, Jesse Gross <jesse at nicira.com> wrote:
> On Thu, Sep 1, 2011 at 7:53 PM, Ethan Jackson <ethan at nicira.com> wrote:
>> OVS fails to build with xenddk-56100build3926 because it has an
>> outdated genetlink header.
>> ---
>> lib/netlink-protocol.h | 11 +++++++++++
>> 1 files changed, 11 insertions(+), 0 deletions(-)
>>
>> diff --git a/lib/netlink-protocol.h b/lib/netlink-protocol.h
>> index 1b5fa71..6a3b92d 100644
>> --- a/lib/netlink-protocol.h
>> +++ b/lib/netlink-protocol.h
>> @@ -36,6 +36,17 @@
>> #ifdef HAVE_NETLINK
>> #include <linux/netlink.h>
>> #include <linux/genetlink.h>
>> +
>> +/* Some Xenddks have an outdated genetlink header file which doesn't have some
>> + * symbols it should. */
>> +#ifndef CTRL_ATTR_MCAST_GROUP_MAX
>> +#define CTRL_ATTR_MCAST_GRP_NAME 1
>> +#define CTRL_ATTR_MCAST_GRP_ID 2
>> +#define CTRL_ATTR_MCAST_GROUPS 7
>> +#undef CTRL_ATTR_MAX
>> +#define CTRL_ATTR_MAX 7
>> +#endif /* CTRL_ATTR_MCAST_GROUP_MAX */
>
> I would probably just drop in the entire CTRL_ATTR_MCAST_* enum and
> define but this should work as well.
>
More information about the dev
mailing list