Dpif-netdev should not implement internal HW offload details. If need to "apply 
on all ports", it needs to be done in offload layer.
However, in arch level, there is a problem with the proposed series. It will 
create a meter object per port, while in SW it is one object, that can be 
shared between multiple flows, on different ports.

>-----Original Message-----
>From: Simon Horman <[email protected]>
>Sent: Friday, 16 December 2022 17:51
>To: [email protected]
>Cc: Eelco Chaudron <[email protected]>; Ilya Maximets
><[email protected]>; Eli Britstein <[email protected]>; Chaoyong He
><[email protected]>; [email protected]; Peng Zhang
><[email protected]>; Simon Horman <[email protected]>
>Subject: [PATCH 0/6] Add support for DPDK meter HW offload
>
>External email: Use caution opening links or attachments
>
>
>Hi,
>
>this series adds support for DPDK meter HW offload
>
>* Patch 1/6: Add netdev provider API for HW offload of DPDK meters
>* Patch 2/6: Add DPIF API to offload OpenFlow meters to DPDK
>* Patch 3/6: Implement netdev provider API for HW offload of DPDK meters
>* Patch 4/6: Add more DPDK meter algorithms
>* Patch 4/6: Add support for meter action ti DPDK HW offload
>* Patch 4/6: Add CI builds with ALLOW_EXPERIMENTAL_API
>
>Peng Zhang (6):
>  netdev-offload: Add DPDK meter offload API
>  dpif-netdev: Offloading meter with DPDK
>  netdev-offload-dpdk: Implement meter offload API for DPDK
>  netdev-dpdk: add meter algorithms
>  netdev-dpdk-offload: Add support for meter action
>  ci: add the opts about ALLOW_EXPERIMENTAL_API
>
> .ci/linux-build.sh                   |   4 +
> .github/workflows/build-and-test.yml |  31 ++++
> Documentation/howto/dpdk.rst         |   5 +-
> lib/dpif-netdev.c                    | 102 +++++++++++
> lib/netdev-dpdk.c                    | 243 +++++++++++++++++++++++++++
> lib/netdev-dpdk.h                    |  41 +++++
> lib/netdev-offload-dpdk.c            | 101 +++++++++++
> lib/netdev-offload-provider.h        |  30 ++++
> lib/netdev-offload.c                 |  59 +++++++
> lib/netdev-offload.h                 |   9 +
> 10 files changed, 623 insertions(+), 2 deletions(-)
>
>--
>2.30.2

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to