[ovs-dev] [PATCH 2/2] dpif-netdev: Indicate support for various ct features.

Justin Pettit jpettit at ovn.org
Thu Jul 20 05:35:24 UTC 2017


> On Jul 19, 2017, at 3:18 PM, Darrell Ball <dlu998 at gmail.com> wrote:
> 
> I could not see any;  it is good the impact from this hybrid support thing was limited
> to debug o/p then.
> I don't see any new issues from this patch addition, and it is the only reasonable
> option for backporting.
> 
> In your original commit message, you said:
> 
> "The userspace datapath hardcodes support for the features it supports,
> but it was missing "ct_state_nat", "ct_orig_tuple", and "ct_orig_tuple6"."
> 
> Can you modify that so that it reflects that it is partially true and the impact
> we now know/assume is limited to debug o/p per your confirmation..

Okay, I updated the patch as follows and pushed it to master:

-=-=-=-=-=-=-=-=-
The userspace datapath uses a structure to indicate supported features
that affects debug output.  This commit updates that structure to
indicate that "ct_state_nat", "ct_orig_tuple", and "ct_orig_tuple6" are
supported.
-=-=-=-=-=-=-=-=-

> Otherwise 
> Acked-by: Darrell Ball <dlu998 at gmail.com>

Thanks,

--Justin




More information about the dev mailing list