Re: [net-next PATCH 0/7] tc offload for cls_u32 on ixgbe

On 2/4/2016 3:19 PM, Pablo Neira Ayuso wrote:

On Thu, Feb 04, 2016 at 10:16:56AM +0100, Jiri Pirko wrote:

quoted

Wed, Feb 03, 2016 at 10:27:32AM CET, john.fastabend@gmail.com wrote:

quoted

Also by adding get_parse_graph and set_parse_graph attributes as
in my previous flow_api work we can build programmable devices
and programmatically learn when rules can or can not be loaded
into the hardware. Again future work.

Any comments/feedback appreciated.

Sorry if you get this twice it doesn't look like my original response
made it to netdev and the laptop I replied on charger blew up.

quoted

I like this being thin and elegant solution. However, ~2 years ago when I
pushed openvswitch kernel datapath offload patchset, people were yelling
at me that it is not generic enough solution, that tc has to be able
to use the api (Jamal :)), nftables as well.

The other problem with OVS is if you have the capabilities to do
wildcard lookups (e.g. TCAM/SRAM/etc) then offloading the exact
match table in OVS is really inefficient use of the resource. You
really want to load the megaflow table into hardware. I just don't
think its a good scheme for what you want.

I would be glad to join this debate during NetDev 1.1 too.

great.

I think we should provide a solution that allows people uses both
tc and nftables, this would require a bit of generic infrastructure on
top of it so we don't restrict users to one single solution, in other
words, we allow the user to select its own poison.

quoted

Now this patch is making offload strictly tc-based and nobody seems to
care :) I do. I think that we might try to find some generic middle
layer.

If we can build the universal model for 'tc' and 'nftable' we should
unify them higher in the stack? It doesn't make sense to me for the
driver folks to try and create the unified model for two subsystems
if we don't think its worthwhile in software as well.

I agree and I'll be happy to help to push this ahead. Let's try to sit
and get together to resolve this.

Great.

See you soon.