ofp-util: Use correct cookie value in "packet_in"s when no flow involved.
authorBen Pfaff <blp@nicira.com>
Tue, 22 Oct 2013 23:57:46 +0000 (16:57 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 23 Oct 2013 04:12:06 +0000 (21:12 -0700)
commitd4fa4e792e55c60142f86a82822132ac584559ce
tree4c327c3ccf4823580b0c6e89db026cf2b541c663
parent22ee3544769ae9e9d35e8efa92b4b50468badf09
ofp-util: Use correct cookie value in "packet_in"s when no flow involved.

OpenFlow 1.3 uses all-1-bits in a packet_in to indicate that the packet_in
was not generated by a flow, but Open vSwitch incorrectly used 0.  This
fixes the problem.

For consistency, this commit also changes NXT_PACKET_IN to use all-1-bits
for this case, event though NXT_PACKET_IN was previously defined to use
zero.  This doesn't appear to make a difference for the NVP controller; if
it causes a problem for some other controller then I will revert that part
of the change.

Found by inspection.

Signed-off-by: Ben Pfaff <blp@nicira.com>
include/openflow/nicira-ext.h
lib/ofp-print.c
lib/ofp-util.c
lib/ofp-util.h
ofproto/ofproto-dpif-upcall.c
ofproto/ofproto-dpif-xlate.c
tests/ofproto-dpif.at