[SRU][F:linux-bluefield][PATCH v1 0/5] Support hardware miss to tc action

[William Tu]

Hi Bartlomiej,
Thanks for your reviews of this series.
I will fix them and resubmit!
William

From: Bartlomiej Zolnierkiewicz <bartlomiej.zolnierkiewicz at canonical.com>
Date: Tuesday, March 28, 2023 at 9:18 AM
To: William Tu <witu at nvidia.com>
Cc: kernel-team at lists.ubuntu.com <kernel-team at lists.ubuntu.com>, Bodong Wang <bodong at nvidia.com>, Vladimir Sokolovsky <vlad at nvidia.com>, dann.frazier at canonical.com <dann.frazier at canonical.com>
Subject: Re: [SRU][F:linux-bluefield][PATCH v1 0/5] Support hardware miss to tc action
External email: Use caution opening links or attachments


On Thu, Mar 23, 2023 at 8:18 PM William Tu <witu at nvidia.com> wrote:
>
> Currently tc miss interface only supports resuming from a specific tc chain.
> If a packet modification is done before a missable action such as CT and
> there is a miss in CT after it, this may cause a miss match when resuming
> re-executing the same chain in software, and wrong packet count.
> This use case for example is a stateless (static) nat.
>
> * brief explanation of fixes
>
> Add support for missing to a specific action instance, and support
> of per action hardware stats to update what was actually done in hardware.
>
> BugLink: https://bugs.launchpad.net/bugs/2012571
>
> Eric Dumazet (1):
>   net/sched: flower: fix fl_change() error recovery path
>
> Paul Blakey (4):
>   net/sched: Provide act to offload action
>   net/sched: cls_api: Support hardware miss to tc action
>   net/sched: flower: Move filter handle initialization earlier
>   net/sched: flower: Support hardware miss to tc action
>
>  include/linux/skbuff.h     |   6 +-
>  include/net/flow_offload.h |   2 +
>  include/net/pkt_cls.h      |  30 +++--
>  include/net/sch_generic.h  |   2 +
>  net/openvswitch/flow.c     |   3 +-
>  net/sched/cls_api.c        | 217 +++++++++++++++++++++++++++++++++++--
>  net/sched/cls_flower.c     |  79 +++++++++-----
>  7 files changed, 288 insertions(+), 51 deletions(-)

Hi William,

Please try using the "--dry-run" parameter for the "git send-email"
command to make sure that your submission is correct (no duplicate
emails and correctly threaded). This parameter makes the command do
everything except actually sending the emails.

You may also try sending the submission to your email address first
(to double check everything) before posting to a mailing list.

Please also see my replies to individual patches.

Best Regards,
Bartlomiej
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230328/c9e6a691/attachment.html>