<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:PMingLiU;
panose-1:2 2 5 0 0 0 0 0 0 0;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"\@PMingLiU";
panose-1:2 1 6 1 0 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
font-size:10.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt">Hi Stefan,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Sorry about that, resent the series here<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><a href="https://lists.ubuntu.com/archives/kernel-team/2023-March/138028.html">https://lists.ubuntu.com/archives/kernel-team/2023-March/138028.html</a><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">William<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal" style="margin-bottom:12.0pt"><b><span style="font-size:12.0pt;color:black">From:
</span></b><span style="font-size:12.0pt;color:black">kernel-team <kernel-team-bounces@lists.ubuntu.com> on behalf of Stefan Bader <stefan.bader@canonical.com><br>
<b>Date: </b>Friday, March 24, 2023 at 1:48 AM<br>
<b>To: </b>kernel-team@lists.ubuntu.com <kernel-team@lists.ubuntu.com><br>
<b>Subject: </b>NACK/Cmnt: [SRU][F:linux-bluefield][PATCH v1 0/5] Support hardware miss to tc action<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:11.0pt">External email: Use caution opening links or attachments<br>
<br>
<br>
On 23.03.23 20:15, William Tu wrote:<br>
> Currently tc miss interface only supports resuming from a specific tc chain.<br>
> If a packet modification is done before a missable action such as CT and<br>
> there is a miss in CT after it, this may cause a miss match when resuming<br>
> re-executing the same chain in software, and wrong packet count.<br>
> This use case for example is a stateless (static) nat.<br>
><br>
> * brief explanation of fixes<br>
><br>
> Add support for missing to a specific action instance, and support<br>
> of per action hardware stats to update what was actually done in hardware.<br>
><br>
> BugLink: <a href="https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugs.launchpad.net%2Fbugs%2F2012571&data=05%7C01%7Cwitu%40nvidia.com%7C5117e1b13ee54f5237b508db2c448a80%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C638152445141566541%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=ZunvfLuhMAEnnq8b4gWrOPUuoPhmFsT%2FHHOME8DcYNA%3D&reserved=0">
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugs.launchpad.net%2Fbugs%2F2012571&data=05%7C01%7Cwitu%40nvidia.com%7C5117e1b13ee54f5237b508db2c448a80%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C638152445141566541%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=ZunvfLuhMAEnnq8b4gWrOPUuoPhmFsT%2FHHOME8DcYNA%3D&reserved=0</a><br>
><br>
> Eric Dumazet (1):<br>
> net/sched: flower: fix fl_change() error recovery path<br>
><br>
> Paul Blakey (4):<br>
> net/sched: Provide act to offload action<br>
> net/sched: cls_api: Support hardware miss to tc action<br>
> net/sched: flower: Move filter handle initialization earlier<br>
> net/sched: flower: Support hardware miss to tc action<br>
><br>
> include/linux/skbuff.h | 6 +-<br>
> include/net/flow_offload.h | 2 +<br>
> include/net/pkt_cls.h | 30 +++--<br>
> include/net/sch_generic.h | 2 +<br>
> net/openvswitch/flow.c | 3 +-<br>
> net/sched/cls_api.c | 217 +++++++++++++++++++++++++++++++++++--<br>
> net/sched/cls_flower.c | 79 +++++++++-----<br>
> 7 files changed, 288 insertions(+), 51 deletions(-)<br>
><br>
<br>
Rejected for the following reasons:<br>
<br>
- incomplete, has follow-ups.<br>
<br>
-Stefan<o:p></o:p></span></p>
</div>
</div>
</body>
</html>