On Fri, 2022-05-27 at 12:02 +0200, Eelco Chaudron wrote:
> 
> 
> On 27 May 2022, at 11:00, Jianbo Liu wrote:
> 
> > This series is to add support for tc offloading of ovs metering,
> > and
> > enhance OVS to use new kernel feature which offload tc police
> > action to
> > hardware.
> > To do the offloading, new APIs for meter are added in netdev-
> > offload,
> > and OVS meters are mapped to tc police actions with one-to-one
> > relationship for dpif-netlink.
> 
> Thanks for the v5, I’ll try to find some time for this patch next
> week, but might be moved to the week after.
> 

Eelco, do you have plan to review them this week?

Thanks!
Jianbo

> //Eelco
> 
> > 
> > Notes:
> >     v5
> >     - Add the stats of dropped packets.
> >     - Enhance meter tests.
> >     - other misc changes.
> > 
> >     v4
> >     - Move meter offload API to netdev_flow_api.
> >     - Add tests to check if meter is offloaded.
> >     - Modify NEWS.
> > 
> >     v3
> >     - Add netdev-offload APIs for meter.
> >     - Move the implementation to lower netdev-offload-tc layer.
> > 
> >     v2
> >     - Move tc police parse call from last patch to 2nd patch.
> >       2nd patch is adding the parse lib func and add the call to it
> > in
> > that patch.
> >       Last patch is the put of tc police act.
> >       In 2nd patch also add empty switch case for tc police act put
> > as
> > the impl.
> >       is done in the last patch when support for put is being
> > added.
> > 
> > Jianbo Liu (8):
> >   netdev-offload: Add meter offload API
> >   tc: Add support parsing tc police action
> >   netdev-linux: Refactor put police action netlink message
> >   netdev-linux: Add functions to manipulate tc police action
> >   netdev-offload-tc: Implement meter offload API for tc
> >   netdev-offload-tc: Cleanup police actions with reserved indexes
> > on
> >     startup
> >   netdev-offload-tc: Offloading rules with police actions
> >   dpif-netlink: Offloading meter to tc police action
> > 
> >  NEWS                             |   2 +
> >  lib/dpif-netlink.c               |  31 ++-
> >  lib/netdev-linux.c               | 231 ++++++++++++++++++----
> >  lib/netdev-linux.h               |   6 +
> >  lib/netdev-offload-provider.h    |  19 ++
> >  lib/netdev-offload-tc.c          | 325
> > +++++++++++++++++++++++++++++++
> >  lib/netdev-offload.c             |  51 +++++
> >  lib/netdev-offload.h             |   8 +
> >  lib/tc.c                         | 246 +++++++++++++++++++----
> >  lib/tc.h                         |  14 ++
> >  tests/system-offloads-traffic.at |  48 +++++
> >  11 files changed, 900 insertions(+), 81 deletions(-)
> > 
> > -- 
> > 2.26.2
> 

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

Reply via email to