Skip to content

Commit

Permalink
lib/netlink-protocol.h: Do not include headers for windows kernel build.
Browse files Browse the repository at this point in the history
Added saurabh's fix to not to include some header files
in lib/netlink-protocol.h not needed by windows driver.

This is a temporary fix to make sure that windows driver can compile.

We are working in parallel on adding netlink support in windows driver
(openvswitch/ovs-issues#18). With netlink support
changes we will get rid of the dependency on lib/netlink-protocol.h.

Testing:
1. Verified make distcheck on linux.
2. Verified that windows driver can compile now.
3. Verified that ping works over vxlan tunnel.

Signed-off-by: Ankur Sharma <ankursharma@vmware.com>
Co-authored-by: Saurabh Shah <ssaurabh@vmware.com>
Tested-by: Ankur Sharma <ankursharma@vmware.com>
Reported-by: Alin Serdean <aserdean@cloudbasesolutions.com>
Reported-at: openvswitch/ovs-issues#21
Signed-off-by: Ben Pfaff <blp@nicira.com>
  • Loading branch information
2 people authored and blp committed Aug 12, 2014
1 parent 07ba3db commit 99f368f
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions lib/netlink-protocol.h
Expand Up @@ -29,9 +29,15 @@
* on other platforms it directly defines the structures and macros itself.
*/

/* This file is included by windows driver as well (as of now).
* It does not have access to following header files,
* hence do not include them for windows driver.
*/
#ifndef OVS_WIN_DP
#include <stdint.h>
#include <sys/socket.h>
#include "util.h"
#endif

#ifdef HAVE_NETLINK
#include <linux/netlink.h>
Expand Down

0 comments on commit 99f368f

Please sign in to comment.