Ben Pfaff <b...@ovn.org> writes:

> These patches fix all of the test failures on big-endian architectures.
> I tested on s390x, this machine;
>       https://db.debian.org/machines.cgi?host=zelenka
>
> Ben Pfaff (5):
>   types: Change ofp_port_t from uint16_t to uint32_t.
>   netdev-native-tnl: Fix treatment of GRE key on big-endian systems.
>   ofp-print: Sort queues before printing in OFPT_QUEUE_GET_CONFIG_REPLY.
>   tests: Fix select group test on big-endian systems.
>   tests: Avoid endianness sensitivity in MPLS handling test.
>
>  include/openvswitch/flow.h  |  1 -
>  include/openvswitch/types.h | 21 ++++++++++++----
>  lib/byte-order.h            | 31 ++++++++++++++++++++++-
>  lib/netdev-native-tnl.c     |  5 ++--
>  lib/ofp-actions.c           |  2 +-
>  lib/ofp-print.c             | 60 
> +++++++++++++++++++++++++++++++++------------
>  tests/ofp-print.at          |  8 +++---
>  tests/ofproto-dpif.at       | 16 ++++++------
>  tests/ofproto.at            |  4 +--
>  utilities/ovs-ofctl.c       |  2 +-
>  10 files changed, 109 insertions(+), 41 deletions(-)

I got tripped up a little with 2/5 (I started reviewing this for
be->host) and 4/5.  Thank goodness for coffee.

I haven't had a chance to test this (I have a be machine somewhere...),
but it looks good to me.

Acked-by: Aaron Conole <acon...@redhat.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to