[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b05ffcad-d3ba-7327-54bb-d346f5303ce2@solarflare.com>
Date: Mon, 2 Mar 2020 16:07:35 +0000
From: Edward Cree <ecree@...arflare.com>
To: Jiri Pirko <jiri@...nulli.us>,
Pablo Neira Ayuso <pablo@...filter.org>
CC: Jamal Hadi Salim <jhs@...atatu.com>,
Jakub Kicinski <kuba@...nel.org>, <netdev@...r.kernel.org>,
<davem@...emloft.net>, <saeedm@...lanox.com>, <leon@...nel.org>,
<michael.chan@...adcom.com>, <vishal@...lsio.com>,
<jeffrey.t.kirsher@...el.com>, <idosch@...lanox.com>,
<aelior@...vell.com>, <peppe.cavallaro@...com>,
<alexandre.torgue@...com>, <xiyou.wangcong@...il.com>,
<mlxsw@...lanox.com>, Marian Pritsak <marianp@...lanox.com>
Subject: Re: [patch net-next 00/10] net: allow user specify TC filter HW stats
type
On 29/02/2020 08:01, Jiri Pirko wrote:
> Fri, Feb 28, 2020 at 08:59:09PM CET, pablo@...filter.org wrote:
>> Something like the sketch patch that I'm attaching?
> But why? Actions are separate entities, with separate counters. The
> driver is either able to offload that or not. Up to the driver to
> abstract this out.
+1
If any of this "fake up a counter action" stuff is common to several
drivers, we could maybe have some library functions to help them with
it, but the tc<->driver API shouldn't change (since the change adds
no expressiveness).
-ed
Powered by blists - more mailing lists