All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jiri Pirko <jiri@resnulli.us>
To: Jakub Kicinski <jakub.kicinski@netronome.com>
Cc: netdev@vger.kernel.org, davem@davemloft.net, jhs@mojatatu.com,
	xiyou.wangcong@gmail.com, simon.horman@netronome.com,
	john.hurley@netronome.com, dsahern@gmail.com, mlxsw@mellanox.com
Subject: Re: [patch net-next 0/9] net: sched: introduce chain templates support with offloading to mlxsw
Date: Tue, 26 Jun 2018 08:43:55 +0200	[thread overview]
Message-ID: <20180626064355.GQ2161@nanopsycho> (raw)
In-Reply-To: <20180625215850.001276b8@cakuba.netronome.com>

Tue, Jun 26, 2018 at 06:58:50AM CEST, jakub.kicinski@netronome.com wrote:
>On Mon, 25 Jun 2018 23:01:39 +0200, Jiri Pirko wrote:
>> From: Jiri Pirko <jiri@mellanox.com>
>> 
>> For the TC clsact offload these days, some of HW drivers need
>> to hold a magic ball. The reason is, with the first inserted rule inside
>> HW they need to guess what fields will be used for the matching. If
>> later on this guess proves to be wrong and user adds a filter with a
>> different field to match, there's a problem. Mlxsw resolves it now with
>> couple of patterns. Those try to cover as many match fields as possible.
>> This aproach is far from optimal, both performance-wise and scale-wise.
>> Also, there is a combination of filters that in certain order won't
>> succeed.
>> 
>> Most of the time, when user inserts filters in chain, he knows right away
>> how the filters are going to look like - what type and option will they
>> have. For example, he knows that he will only insert filters of type
>> flower matching destination IP address. He can specify a template that
>> would cover all the filters in the chain.
>
>Perhaps it's lack of sleep, but this paragraph threw me a little off
>the track.  IIUC the goal of this set is to provide a way to inform the
>HW about expected matches before any rule is programmed into the HW.
>Not before any rule is added to a particular chain.  One can just use
>the first rule in the chain to make a guess about the chain, but thanks
>to this set user can configure *all* chains before any rules are added.

The template is per-chain. User can use template for chain x and
not-use it for chain y. Up to him.

>
>And that's needed because once any rule is added the tcam config can no
>longer be easily modified?

Yes.

  reply	other threads:[~2018-06-26  6:45 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-25 21:01 [patch net-next 0/9] net: sched: introduce chain templates support with offloading to mlxsw Jiri Pirko
2018-06-25 21:01 ` [patch net-next 1/9] net: sched: push ops lookup bits into tcf_proto_lookup_ops() Jiri Pirko
2018-06-25 21:01 ` [patch net-next 2/9] net: sched: introduce chain templates Jiri Pirko
2018-06-25 21:01 ` [patch net-next 3/9] net: sched: cls_flower: move key/mask dumping into a separate function Jiri Pirko
2018-06-25 21:01 ` [patch net-next 4/9] net: sched: cls_flower: change fl_init_dissector to accept mask and dissector Jiri Pirko
2018-06-25 21:01 ` [patch net-next 5/9] net: sched: cls_flower: implement chain templates Jiri Pirko
2018-06-25 21:01 ` [patch net-next 6/9] net: sched: cls_flower: propagate chain teplate creation and destruction to drivers Jiri Pirko
2018-06-26  5:00   ` Jakub Kicinski
2018-06-26  6:40     ` Jiri Pirko
2018-06-25 21:01 ` [patch net-next 7/9] mlxsw: spectrum: Implement chain template hinting Jiri Pirko
2018-06-25 21:01 ` [patch net-next 8/9] selftests: forwarding: move shblock tc support check to a separate helper Jiri Pirko
2018-06-25 21:01 ` [patch net-next 9/9] selftests: forwarding: add tests for TC chain templates Jiri Pirko
2018-06-25 21:03 ` [patch iproute2/net-next] tc: introduce support for " Jiri Pirko
2018-06-26  4:58 ` [patch net-next 0/9] net: sched: introduce chain templates support with offloading to mlxsw Jakub Kicinski
2018-06-26  6:43   ` Jiri Pirko [this message]
2018-06-26  7:00     ` Jakub Kicinski
2018-06-26  7:12       ` Jiri Pirko
2018-06-26 21:18         ` Jakub Kicinski
2018-06-27  7:50           ` Jiri Pirko
2018-06-27 16:46             ` Samudrala, Sridhar
2018-06-27 17:04               ` Cong Wang
2018-06-28  6:18                 ` Jiri Pirko
2018-06-28 17:32                   ` Cong Wang
2018-06-27 18:36             ` Jakub Kicinski
2018-06-28  6:15               ` Jiri Pirko

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=20180626064355.GQ2161@nanopsycho \
    --to=jiri@resnulli.us \
    --cc=davem@davemloft.net \
    --cc=dsahern@gmail.com \
    --cc=jakub.kicinski@netronome.com \
    --cc=jhs@mojatatu.com \
    --cc=john.hurley@netronome.com \
    --cc=mlxsw@mellanox.com \
    --cc=netdev@vger.kernel.org \
    --cc=simon.horman@netronome.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.