Re: [PATCH net-next 1/8] net/flower: Introduce hardware offload support

On Tue, Mar 1, 2016 at 7:01 PM, Jiri Pirko [off-list ref] wrote:

Tue, Mar 01, 2016 at 05:49:27PM CET, amir@vadai.me wrote:

quoted

On Tue, Mar 01, 2016 at 03:47:19PM +0100, Jiri Pirko wrote:

quoted

Tue, Mar 01, 2016 at 03:24:43PM CET, amir@vadai.me wrote:

quoted

This patch is based on a patch made by John Fastabend.

quoted

quoted

quoted

It adds support for offloading cls_flower.
A filter that is offloaded successfully by hardware, will not be added to
the hashtable and won't be processed by software.

quoted

quoted

That is wrong. User should explicitly specify to not include rule into sw
by SKIP_KERNEL flag (does not exist now, with John's recent patch we'll
have only SKIP_HW). Please add that in this patchset.

quoted

Why? If a rule is offloaded, why would the user want to reprocess it by software?

quoted

If the user use SKIP_HW, it will be processed by SW. Else, the user
would want it to be processed by HW or fallback to SW. I don't
understand in which case the user would like to have it done twice.

For example if you turn on the offloading by unsetting NETIF_F_HW_TC.
Or if someone inserts skbs into rx path directly, for example pktgen.
We need SKIP_KERNEL to be set by user, not implicit.

As discussed in netdev, we want to have three modes for TC offloads

1. SW only
2. HW only (and err if can't)
3. HW and if not supported fallback to SW

Now, from your reply, I understand we want a fourth mode

4. Both (HW and SW)

Do we agree that these four policies/modes make sense?

So you want #4 to be the default? can you elaborate why? note that for
the HW marking
case a "both" mode will be very inefficient, also for actions like
vlan push/pop, encap/decap, etc
the result will be just wrong... I don't think this should be the default.

Or.