datapath: Don't track IP TOS value two different ways.
authorBen Pfaff <blp@nicira.com>
Tue, 27 Jul 2010 17:02:07 +0000 (10:02 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 3 Aug 2010 03:16:32 +0000 (20:16 -0700)
commitabfec865566e6cce961cc8660de1ddfdc85dae5f
tree1c791f895eed3943bb0a9b04ec074449784e9234
parent8084a2800f82c6d0ec438cb8f2de24518af2c712
datapath: Don't track IP TOS value two different ways.

Originally, the datapath didn't care about IP TOS at all.  Then, to support
NetFlow, we made it keep track of the last-seen IP TOS value on a per-flow
basis.  Then, to support OpenFlow 1.0, we added a nw_tos field to
odp_flow_key.  We don't need both methods, so this commit drops the
NetFlow-specific tracking.

This introduces a small kernel ABI break: upgrading the kernel module
without upgrading the OVS userspace will mean that NetFlow records will
all show an IP TOS value of 0.  I don't consider that to be a serious
problem.
datapath/datapath.c
datapath/flow.c
datapath/flow.h
include/openvswitch/datapath-protocol.h
lib/dpif-netdev.c
lib/flow.c
ofproto/netflow.c
ofproto/netflow.h
ofproto/ofproto.c