ofp-util: Consistently treat OpenFlow xids as network byte order.
authorBen Pfaff <blp@nicira.com>
Tue, 16 Nov 2010 19:00:25 +0000 (11:00 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 17 Nov 2010 17:21:09 +0000 (09:21 -0800)
commit44381c1b4e01d051a8cc9ffccb8c37a0c2d4c808
tree639546b2d58747e105235e927f1202ccdd24142a
parent72fae175990e9a4732ba4d368946f3854e994075
ofp-util: Consistently treat OpenFlow xids as network byte order.

The 'xid' in an ofp_header is not interpreted by the receiver but only by
the sender, so it need not be in any particular byte order.  OVS used to
try to take advantage of this to avoid host/network byte order conversions
for this field.  Older code in OVS, therefore, treats xid as being in host
byte order.  However, as time went on, I forgot that I had introduced this
trick, and so newer code treats xid as being in network byte order.

This commit fixes up the situation by consistently treating xid as being
in network byte order.  I think that this will be less surprising and
easier to remember in the future.

This doesn't fix any actual bugs except that some log messages would have
printed xids in the wrong byte order.
lib/ofp-print.c
lib/ofp-util.c
lib/ofp-util.h
lib/vconn.c
utilities/ovs-ofctl.c