[ovs-dev] [PATCH v2 01/15] vswitchd: Deprecate packet buffering in OVS 2.7.

[Jarno Rajahalme]

> On Aug 29, 2016, at 10:53 AM, Ben Pfaff <blp at ovn.org> wrote:
> 
> On Mon, Aug 22, 2016 at 04:31:27PM -0700, Jarno Rajahalme wrote:
>> OVS implementation of buffering packets that are sent to the
>> controller is not compliant with the OpenFlow specifications after
>> OpenFlow 1.0.  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, we
>> propose to remove the feature altogether, starting in OVS 2.7.  This
>> patch announces this in 'NEWS', and adds detial to the FAQ question
>> about backet buffering.
>> 
>> Signed-off-by: Jarno Rajahalme <jarno at ovn.org>
> 
> The terminology here is a little confusing.  I believe that the
> intention is to remove packet buffering from OVS 2.7.  Usually, I think
> of "deprecation" as a way of discouraging users from using a feature
> that will be removed in a future release.  So, if we follow that
> terminology, then 2.6 will deprecate packet buffering and 2.7 will
> remove it.
> 

Oops, thanks for the review! Fixed and push to master and branch-2.6.

  Jarno

> If you're willing to make that change to the naming, then:
> Acked-by: Ben Pfaff <blp at ovn.org>