All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vlad Buslov <vladbu@nvidia.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: Paul Blakey <paulb@nvidia.com>, <netdev@vger.kernel.org>,
	Saeed Mahameed <saeedm@nvidia.com>,
	Paolo Abeni <pabeni@redhat.com>,
	Eric Dumazet <edumazet@google.com>,
	Jamal Hadi Salim <jhs@mojatatu.com>,
	Cong Wang <xiyou.wangcong@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	"Oz Shlomo" <ozsh@nvidia.com>, Jiri Pirko <jiri@nvidia.com>,
	Roi Dayan <roid@nvidia.com>
Subject: Re: [PATCH net-next v5 0/6] net/sched: cls_api: Support hardware miss to tc action
Date: Sat, 28 Jan 2023 10:59:47 +0200	[thread overview]
Message-ID: <87zga3do7v.fsf@nvidia.com> (raw)
In-Reply-To: <20230128001810.08f02b0a@kicinski-fedora-PC1C0HJN>

On Sat 28 Jan 2023 at 00:18, Jakub Kicinski <kuba@kernel.org> wrote:
> On Sat, 28 Jan 2023 00:16:40 -0800 Jakub Kicinski wrote:
>> On Wed, 25 Jan 2023 17:32:12 +0200 Paul Blakey wrote:
>> > This series adds support for hardware miss to instruct tc to continue execution
>> > in a specific tc action instance on a filter's action list. The mlx5 driver patch
>> > (besides the refactors) shows its usage instead of using just chain restore.
>> > 
>> > Currently a filter's action list must be executed all together or
>> > not at all as driver are only able to tell tc to continue executing from a
>> > specific tc chain, and not a specific filter/action.
>> > 
>> > This is troublesome with regards to action CT, where new connections should
>> > be sent to software (via tc chain restore), and established connections can
>> > be handled in hardware.  
>> 
>> I'll mark this as Deferred - would be great if Red Hat OvS offload
>> folks and/or another vendor and/or Jamal gave their acks.
>
> Ignore that, it's already Changes Requested.. 🤷️

Hmm I'm looking at patchwork.kernel.org/project/netdevbpf and the series
state is "New". Is there some other tool for this? Also, since I didn't
receive any other replies to this series, where can I check the
requested changes? Sorry if I'm missing something obvious here.


  reply	other threads:[~2023-01-28  9:05 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-25 15:32 [PATCH net-next v5 0/6] net/sched: cls_api: Support hardware miss to tc action Paul Blakey
2023-01-25 15:32 ` [PATCH net-next v5 1/6] " Paul Blakey
2023-01-28 13:56   ` Jamal Hadi Salim
2023-01-28 16:45   ` Simon Horman
2023-01-28 16:52   ` Simon Horman
2023-01-25 15:32 ` [PATCH net-next v5 2/6] net/sched: flower: Move filter handle initialization earlier Paul Blakey
2023-01-28 16:45   ` Simon Horman
2023-01-25 15:32 ` [PATCH net-next v5 3/6] net/sched: flower: Support hardware miss to tc action Paul Blakey
2023-01-28 16:45   ` Simon Horman
2023-01-25 15:32 ` [PATCH net-next v5 4/6] net/mlx5: Refactor tc miss handling to a single function Paul Blakey
2023-01-25 15:32 ` [PATCH net-next v5 5/6] net/mlx5e: Rename CHAIN_TO_REG to MAPPED_OBJ_TO_REG Paul Blakey
2023-01-25 15:32 ` [PATCH net-next v5 6/6] net/mlx5e: TC, Set CT miss to the specific ct action instance Paul Blakey
2023-01-28  8:16 ` [PATCH net-next v5 0/6] net/sched: cls_api: Support hardware miss to tc action Jakub Kicinski
2023-01-28  8:18   ` Jakub Kicinski
2023-01-28  8:59     ` Vlad Buslov [this message]
2023-01-28 15:12       ` Vlad Buslov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87zga3do7v.fsf@nvidia.com \
    --to=vladbu@nvidia.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=jhs@mojatatu.com \
    --cc=jiri@nvidia.com \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=ozsh@nvidia.com \
    --cc=pabeni@redhat.com \
    --cc=paulb@nvidia.com \
    --cc=roid@nvidia.com \
    --cc=saeedm@nvidia.com \
    --cc=xiyou.wangcong@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.