Re: [RFC PATCH net-next 1/1] Introduce skbmod action

On Sat, Jul 16, 2016 at 7:28 AM, Jamal Hadi Salim [off-list ref] wrote:

On 16-07-12 01:10 PM, Cong Wang wrote:

quoted

On Mon, Jul 11, 2016 at 5:12 AM, Jamal Hadi Salim [off-list ref]
wrote:

quoted

From: Jamal Hadi Salim <jhs@mojatatu.com>

This action is intended to be an upgrade from a usability perspective
from pedit. Compare this:

Definitely agree we need a more user-friendly interface.

quoted

pedit is a good starting point - but once you start going to a large
number of policies then from a programmability, ops and usability point
of view you need something with more succint params.

But it is still unclear why we can't just build something on top of
pedit? Since pedit accepts keys like u32, user-space is free to
introduce any wrapper on top of it.

That hasnt served us well so far for reasons i described earlier
especially in the dump case.
It hasnt served u32 as a classifier well either (yes, those dump
wrappers exist on the tc user code); i am almost willing to bet
you didnt even know such a feature existed ;->

But this still sounds like it is solvable, just not yet solved, right?
And we don't have to change kernel space unless we can't
fix user-space.