On 29 Mar 2022, at 12:17, Kumar Amber wrote:

> Modified the dplcs info-get command output to include
> the count for different dpcls implementations.
>
> $ovs-appctl dpif-netdev/subtable-lookup-prio-get
>
> Available dpcls implementations:
>   autovalidator (Use count: 1, Priority: 5)
>   generic (Use count: 0, Priority: 1)
>   avx512_gather (Use count: 0, Priority: 3)
>
> Test case to verify changes:
>       1021: PMD - dpcls configuration     ok
>
> Signed-off-by: Kumar Amber <kumar.am...@intel.com>
> Signed-off-by: Harry van Haaren <harry.van.haa...@intel.com>
> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
> Co-authored-by: Harry van Haaren <harry.van.haa...@intel.com>
> Co-authored-by: Eelco Chaudron <echau...@redhat.com>
> Acked-by: Eelco Chaudron <echau...@redhat.com>
> ---
> v6:
> - Rebase to master
> v5:
> - change the info-incr and decr APIs.
> - Reduce the complexity of dpcls stats APIs.
> v4:
> - Fix comments on the patch.
> - Change API from an overloaded method of counting, to returning the
>   old and new subtable structs. This allows the caller to identify the
>   modified subtable implementations, and update the statistics accordingly.
> v3:
> - Fix comments on the patch.
> - Function API remains same, see discussion on OVS ML here:
>   "https://mail.openvswitch.org/pipermail/ovs-dev/2021-October/388737.html";
> v2:
> - Dependency merged rebased to master.

v6 changes look fine by me. There is one other question on the v5 from Ilya, 
not sure if you noticed.

Acked-by: Eelco Chaudron <echau...@redhat.com>

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to