Change a test so that the result will be the same in both
little-endian and big-endian systems.

Reported-by: Mijo Safradin <m...@linux.vnet.ibm.com>
Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>
---
 lib/classifier.c    |    3 ++-
 tests/classifier.at |    4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/classifier.c b/lib/classifier.c
index fe43bd1..1985efe 100644
--- a/lib/classifier.c
+++ b/lib/classifier.c
@@ -1336,7 +1336,8 @@ miniflow_and_mask_matches_flow_wc(const struct miniflow 
*flow,
             /* Only unwildcard if none of the differing bits is already
              * exact-matched. */
             if (!(flow_u32_value(&wc->masks, idx) & diff)) {
-                /* Keep one bit of the difference. */
+                /* Keep one bit of the difference.  The selected bit may be
+                 * different in big-endian v.s. little-endian systems. */
                 *flow_u32_lvalue(&wc->masks, idx) |= rightmost_1bit(diff);
             }
             return false;
diff --git a/tests/classifier.at b/tests/classifier.at
index 4d23be5..eb97022 100644
--- a/tests/classifier.at
+++ b/tests/classifier.at
@@ -43,9 +43,9 @@ AT_CHECK([tail -2 stdout], [0],
   [Megaflow: recirc_id=0,tcp,in_port=2,nw_dst=192.168.0.0/16,nw_frag=no
 Datapath actions: 1
 ])
-AT_CHECK([ovs-appctl ofproto/trace br0 
'in_port=1,dl_src=50:54:00:00:00:05,dl_dst=50:54:00:00:00:07,dl_type=0x0800,nw_src=192.168.0.1,nw_dst=192.168.0.2,nw_proto=6,nw_tos=0,nw_ttl=128,tp_src=8,tp_dst=80'],
 [0], [stdout])
+AT_CHECK([ovs-appctl ofproto/trace br0 
'in_port=1,dl_src=50:54:00:00:00:05,dl_dst=50:54:00:00:00:07,dl_type=0x0800,nw_src=192.168.0.1,nw_dst=11.1.2.15,nw_proto=6,nw_tos=0,nw_ttl=128,tp_src=8,tp_dst=80'],
 [0], [stdout])
 AT_CHECK([tail -2 stdout], [0],
-  [Megaflow: recirc_id=0,ip,in_port=1,nw_dst=0.0.0.0/2.0.0.0,nw_frag=no
+  [Megaflow: recirc_id=0,ip,in_port=1,nw_dst=1.0.0.0/1.0.0.0,nw_frag=no
 Datapath actions: drop
 ])
 AT_CHECK([ovs-appctl ofproto/trace br0 
'in_port=1,dl_src=50:54:00:00:00:05,dl_dst=50:54:00:00:00:07,dl_type=0x0800,nw_src=192.168.0.1,nw_dst=10.1.2.15,nw_proto=6,nw_tos=0,nw_ttl=128,tp_src=8,tp_dst=80'],
 [0], [stdout])
-- 
1.7.10.4

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to