[ovs-dev] [PATCH 15/20] netdev-offload-dpdk-flow: Support offload of output action
Eli Britstein
elibr at mellanox.com
Thu Dec 5 06:48:11 UTC 2019
On 12/2/2019 4:25 PM, Ilya Maximets wrote:
> BTW, function netdev_dpdk_flow_api_supported() is not intended to be called
> in the offloading process. It's only for initialization phase. You can see
> the "/* TODO: Check if we able to offload some minimal flow. */" in the code
> and that might be destructive and unwanted for offloading process.
I guess the thought you had in mind when writing it was to call
rte_flow_validate once or more in order to find out offload capabilities
of that device, but for now it is only a comment.
In the future if/when we do call rte_flow_validate, I think it should
not be in this function, but in a function maybe more specific for that,
like
netdev_offload_dpdk_cap(), or something like that.
In current code for sure, I don't see any issue to call this function in
the offloading process, and not only during init.
More information about the dev
mailing list