RE: Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-26 Thread Po Liu
gmail.com; > simon.hor...@netronome.com; pa...@netfilter.org; > mo...@mellanox.com; m-kariche...@ti.com; > andre.gue...@linux.intel.com; step...@networkplumber.org; Edward > Cree > Subject: Re: [v1,net-next 3/4] net: qos: police action add index for tc > flo

Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-26 Thread Jamal Hadi Salim
On 2020-06-24 8:34 p.m., Po Liu wrote: -Original Message- That is the point i was trying to get to. Basically: You have a counter table which is referenced by "index" You also have a meter/policer table which is referenced by "index". They should be one same group and same

RE:Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-24 Thread Po Liu
l.com; > simon.hor...@netronome.com; pa...@netfilter.org; > mo...@mellanox.com; m-kariche...@ti.com; > andre.gue...@linux.intel.com; step...@networkplumber.org; Edward > Cree > Subject: Re: [v1,net-next 3/4] net: qos: police action add index for tc > flower offloading &

Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-24 Thread Jamal Hadi Salim
On 2020-06-23 7:52 p.m., Po Liu wrote: Hi Jamal, My question: Is this any different from how stats are structured? [..] My question: Why cant you apply the same semantics for the counters? Does your hardware have an indexed counter/stats table? If yes then you Yes, That is the

RE: Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Po Liu
gmail.com; > simon.hor...@netronome.com; pa...@netfilter.org; > mo...@mellanox.com; m-kariche...@ti.com; > andre.gue...@linux.intel.com; step...@networkplumber.org; Edward > Cree > Subject: Re: [v1,net-next 3/4] net: qos: police action add index for tc > flower offloading > &

Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Jamal Hadi Salim
On 2020-06-23 7:55 a.m., Po Liu wrote: [..] My question: Is this any different from how stats are structured? I don't know I fully catch the question. Are you trying to get how many frames for each filter chain passing one index policing action? If one index police action bind to multiple

RE: Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Po Liu
gmail.com; > simon.hor...@netronome.com; pa...@netfilter.org; > mo...@mellanox.com; m-kariche...@ti.com; > andre.gue...@linux.intel.com; step...@networkplumber.org; Edward > Cree > Subject: Re: [v1,net-next 3/4] net: qos: police action add index for tc > flower offloading &

Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Jamal Hadi Salim
This certainly brings an interesting point which i brought up earlier when Jiri was doing offloading of stats. In this case the action index is being used as the offloaded policer index (note: there'd need to be a check whether the index is infact acceptable to the h/w etc unless there 2^32

RE:Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Po Liu
@mojatatu.com; xiyou.wangc...@gmail.com; > simon.hor...@netronome.com; pa...@netfilter.org; > mo...@mellanox.com; m-kariche...@ti.com; > andre.gue...@linux.intel.com; step...@networkplumber.org > Subject: Re: [v1,net-next 3/4] net: qos: police action add index for tc > flower offloading

RE: [EXT] Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Po Liu
l.com; > simon.hor...@netronome.com; pa...@netfilter.org; > mo...@mellanox.com; m-kariche...@ti.com; > andre.gue...@linux.intel.com; step...@networkplumber.org > Subject: [EXT] Re: [v1,net-next 3/4] net: qos: police action add index for tc > flower offloading > > Caution: EXT

Re: [v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Ido Schimmel
On Tue, Jun 23, 2020 at 02:34:11PM +0800, Po Liu wrote: > From: Po Liu > > Hardware may own many entries for police flow. So that make one(or > multi) flow to be policed by one hardware entry. This patch add the > police action index provide to the driver side make it mapping the > driver

[v1,net-next 3/4] net: qos: police action add index for tc flower offloading

2020-06-23 Thread Po Liu
From: Po Liu Hardware may own many entries for police flow. So that make one(or multi) flow to be policed by one hardware entry. This patch add the police action index provide to the driver side make it mapping the driver hardware entry index. Signed-off-by: Po Liu ---