diff options
| author | Jarno Rajahalme <jarno@ovn.org> | 2016-08-30 10:20:51 -0700 |
|---|---|---|
| committer | Jarno Rajahalme <jarno@ovn.org> | 2016-08-30 10:20:51 -0700 |
| commit | c184807ced554c1dc7b69b3cd4f59cd85575fdf1 (patch) | |
| tree | 8a477f382ec8b177ea7a436753933efc8ede606c /ofproto/fail-open.c | |
| parent | 88e62998a2db1f8c9f3df2ef9f8d0cf8a2277262 (diff) | |
| download | openvswitch-c184807ced554c1dc7b69b3cd4f59cd85575fdf1.tar.gz | |
lib: Retire packet buffering feature.
OVS implementation of buffering packets that are sent to the
controller is not compliant with the OpenFlow specifications after
OpenFlow 1.0, which is possibly true since OpenFlow 1.0 is not really
specifying the packet buffering behavior.
OVS implementation executes the buffered packet against the actions of
the modified or added rule, whereas OpenFlow (since 1.1) specifies
that the packet should be matched against the flow table 0 and
processed accordingly.
Rather than fix this behavior, and potentially break OVS users, the
packet buffering feature is removed altogether. After all, such
packet buffering is an optional OpenFlow feature, and as such any
possible users should continue to work without this feature.
This patch also makes OVS check the received 'buffer_id' values more
rigorously, and fixes some internal users accordingly.
Found by inspection.
Signed-off-by: Jarno Rajahalme <jarno@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'ofproto/fail-open.c')
| -rw-r--r-- | ofproto/fail-open.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ofproto/fail-open.c b/ofproto/fail-open.c index d8ec2131e..3c3579ef1 100644 --- a/ofproto/fail-open.c +++ b/ofproto/fail-open.c @@ -31,7 +31,6 @@ #include "openvswitch/vlog.h" #include "ofproto.h" #include "ofproto-provider.h" -#include "pktbuf.h" #include "poll-loop.h" #include "rconn.h" #include "timeval.h" |