vswitchd: fix 32-bit builds
authorAnsis Atteka <aatteka@nicira.com>
Thu, 22 Nov 2012 21:00:05 +0000 (13:00 -0800)
committerAnsis Atteka <aatteka@nicira.com>
Fri, 23 Nov 2012 01:44:21 +0000 (03:44 +0200)
My previous 72e8bf28bb38e8816435c64859fb350215b6a9e6 (datapath:
add skb mark matching and set action) commit broke 32-bit builds.
This patch assures that size of struct flow is equal on both
32-bit and 64-bit architectures so that build asserts would
not fire anymore.

Acked-by: Ben Pfaff <blp@nicira.com>
Signed-off-by: Ansis Atteka <aatteka@nicira.com>
lib/flow.c
lib/flow.h
tests/test-bundle.c
tests/test-multipath.c

index 57eddba26776eadc4aca0a0f9e280937c9823c04..c2e99b7824f679a4c53016b7008c49ccb79742c2 100644 (file)
@@ -555,6 +555,7 @@ void
 flow_wildcards_init_exact(struct flow_wildcards *wc)
 {
     memset(&wc->masks, 0xff, sizeof wc->masks);
+    memset(wc->masks.zeros, 0, sizeof wc->masks.zeros);
 }
 
 /* Returns true if 'wc' matches every packet, false if 'wc' fixes any bits or
index fa7410e246e5954d81e918a3ab31fa3bfa459633..ca4143916dbacc3ad9cda62c7ee74ca808a7f8f9 100644 (file)
@@ -103,6 +103,7 @@ struct flow {
     uint8_t arp_tha[6];         /* ARP/ND target hardware address. */
     uint8_t nw_ttl;             /* IP TTL/Hop Limit. */
     uint8_t nw_frag;            /* FLOW_FRAG_* flags. */
+    uint8_t zeros[4];
 };
 BUILD_ASSERT_DECL(sizeof(struct flow) % 4 == 0);
 
index f2d9b8245156b14b9141f5f1b62c45db317f6b3b..aa8b6f0f47398ce98445a4d8ed7b147ebbf7a9a3 100644 (file)
@@ -136,6 +136,7 @@ main(int argc, char *argv[])
     flows = xmalloc(N_FLOWS * sizeof *flows);
     for (i = 0; i < N_FLOWS; i++) {
         random_bytes(&flows[i], sizeof flows[i]);
+        memset(flows[i].zeros, 0, sizeof flows[i].zeros);
         flows[i].regs[0] = OFPP_NONE;
     }
 
index 8a355677d607b1ffe22fea374b8d483fd63ae9cd..b990c1338c9e52383e02301ffa20ea8b9205f473 100644 (file)
@@ -60,6 +60,7 @@ main(int argc, char *argv[])
             struct flow flow;
 
             random_bytes(&flow, sizeof flow);
+            memset(flow.zeros, 0, sizeof flow.zeros);
 
             mp.max_link = n - 1;
             multipath_execute(&mp, &flow);