flow_dissector: Don't use bit fields.
authorDavid S. Miller <davem@davemloft.net>
Tue, 1 Sep 2015 23:46:08 +0000 (16:46 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Sep 2015 23:46:08 +0000 (16:46 -0700)
Just have a flags member instead.

   In file included from include/linux/linkage.h:4:0,
                    from include/linux/kernel.h:6,
                    from net/core/flow_dissector.c:1:
   In function 'flow_keys_hash_start',
       inlined from 'flow_hash_from_keys' at net/core/flow_dissector.c:553:34:
>> include/linux/compiler.h:447:38: error: call to '__compiletime_assert_459' declared with attribute error: BUILD_BUG_ON failed: FLOW_KEYS_HASH_OFFSET % sizeof(u32)

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/flow_dissector.h
net/core/flow_dissector.c

index bddd1089dbce243866310ee2fe758883f8081c5a..8c8548cf5888c4535a809494fad454e755ff6749 100644 (file)
 struct flow_dissector_key_control {
        u16     thoff;
        u16     addr_type;
-       u32     is_fragment:1;
-       u32     first_frag:1;
-       u32     encapsulation:1;
+       u32     flags;
 };
 
+#define FLOW_DIS_IS_FRAGMENT   BIT(0)
+#define FLOW_DIS_FIRST_FRAG    BIT(1)
+#define FLOW_DIS_ENCAPSULATION BIT(2)
+
 /**
  * struct flow_dissector_key_basic:
  * @thoff: Transport header offset
index b563339436d0cb083e51ff015980bdacc23f7228..8d32020303c60b4ed7233eccb38564e39dce37bc 100644 (file)
@@ -189,12 +189,12 @@ ip:
                key_control->addr_type = FLOW_DISSECTOR_KEY_IPV4_ADDRS;
 
                if (ip_is_fragment(iph)) {
-                       key_control->is_fragment = 1;
+                       key_control->flags |= FLOW_DIS_IS_FRAGMENT;
 
                        if (iph->frag_off & htons(IP_OFFSET)) {
                                goto out_good;
                        } else {
-                               key_control->first_frag = 1;
+                               key_control->flags |= FLOW_DIS_FIRST_FRAG;
                                if (!(flags & FLOW_DISSECTOR_F_PARSE_1ST_FRAG))
                                        goto out_good;
                        }
@@ -398,7 +398,7 @@ ip_proto_again:
                        nhoff += sizeof(*eth);
                }
 
-               key_control->encapsulation = 1;
+               key_control->flags |= FLOW_DIS_ENCAPSULATION;
                if (flags & FLOW_DISSECTOR_F_STOP_AT_ENCAP)
                        goto out_good;
 
@@ -434,12 +434,12 @@ ip_proto_again:
                if (!fh)
                        goto out_bad;
 
-               key_control->is_fragment = 1;
+               key_control->flags |= FLOW_DIS_IS_FRAGMENT;
 
                nhoff += sizeof(_fh);
 
                if (!(fh->frag_off & htons(IP6_OFFSET))) {
-                       key_control->first_frag = 1;
+                       key_control->flags |= FLOW_DIS_FIRST_FRAG;
                        if (flags & FLOW_DISSECTOR_F_PARSE_1ST_FRAG) {
                                ip_proto = fh->nexthdr;
                                goto ip_proto_again;
@@ -450,7 +450,7 @@ ip_proto_again:
        case IPPROTO_IPIP:
                proto = htons(ETH_P_IP);
 
-               key_control->encapsulation = 1;
+               key_control->flags |= FLOW_DIS_ENCAPSULATION;
                if (flags & FLOW_DISSECTOR_F_STOP_AT_ENCAP)
                        goto out_good;
 
@@ -458,7 +458,7 @@ ip_proto_again:
        case IPPROTO_IPV6:
                proto = htons(ETH_P_IPV6);
 
-               key_control->encapsulation = 1;
+               key_control->flags |= FLOW_DIS_ENCAPSULATION;
                if (flags & FLOW_DISSECTOR_F_STOP_AT_ENCAP)
                        goto out_good;