[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