Patchwork [net-next] bridge: use __u16 in if_bridge.h

login
register
mail settings
Submitter Amerigo Wang
Date Feb. 14, 2013, 5:32 a.m.
Message ID <1360819951-19484-1-git-send-email-amwang@redhat.com>
Download mbox | patch
Permalink /patch/220354/
State Accepted
Delegated to: David Miller
Headers show

Comments

Amerigo Wang - Feb. 14, 2013, 5:32 a.m.
From: Cong Wang <amwang@redhat.com>

We should use "__u16" instead of "u16" in the user-space visable
header.

Cc: Vlad Yasevich <vyasevic@redhat.com>
Cc: Stephen Hemminger <stephen@networkplumber.org>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Cong Wang <amwang@redhat.com>

---
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller - Feb. 14, 2013, 5:54 a.m.
From: Cong Wang <amwang@redhat.com>
Date: Thu, 14 Feb 2013 13:32:31 +0800

> From: Cong Wang <amwang@redhat.com>
> 
> We should use "__u16" instead of "u16" in the user-space visable
> header.
> 
> Cc: Vlad Yasevich <vyasevic@redhat.com>
> Cc: Stephen Hemminger <stephen@networkplumber.org>
> Cc: David S. Miller <davem@davemloft.net>
> Signed-off-by: Cong Wang <amwang@redhat.com>

Seems obvious enough, applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/include/uapi/linux/if_bridge.h b/include/uapi/linux/if_bridge.h
index f1bf8d3..2d70d79 100644
--- a/include/uapi/linux/if_bridge.h
+++ b/include/uapi/linux/if_bridge.h
@@ -124,8 +124,8 @@  enum {
 #define BRIDGE_VLAN_INFO_UNTAGGED	(1<<2)	/* VLAN egresses untagged */
 
 struct bridge_vlan_info {
-	u16 flags;
-	u16 vid;
+	__u16 flags;
+	__u16 vid;
 };
 
 /* Bridge multicast database attributes