[ovs-dev] [PATCH ovs V8 03/26] other-config: Add hw-offload switch to control netdev flow offloading

Roi Dayan roid at mellanox.com
Tue May 16 09:12:41 UTC 2017



On 08/05/2017 15:38, Simon Horman wrote:
> On Wed, May 03, 2017 at 06:07:54PM +0300, Roi Dayan wrote:
>> From: Paul Blakey <paulb at mellanox.com>
>>
>> Add a new configuration option - hw-offload that enables netdev
>> flow api. Enabling this option will allow offloading flows
>> using netdev implementation instead of the kernel datapath.
>> This configuration option defaults to false - disabled.
>>
>> Signed-off-by: Paul Blakey <paulb at mellanox.com>
>> Reviewed-by: Roi Dayan <roid at mellanox.com>
>> Reviewed-by: Simon Horman <simon.horman at netronome.com>
>
> ...
>
> This does not appear to address Flavio Leitner's review of v7 of this patch.
>

as mentioned, email problems, I looked in the online archive and 
addressed for V9.

as suggested, I removed the extern from netdev_flow_api_enabled and 
added function netdev_is_flow_api_enabled().

I didn't add netdev_flow_api_supported as it seems redundant..?
we compile tc.c in Linux only so we can't have 
netdev_set_flow_api_enabled() implementation so will keep it empty.


More information about the dev mailing list