On Tue, 2004-12-21 at 19:31, Thomas Graf wrote:
> * Patrick McHardy <41C7F833.4000909@xxxxxxxxx> 2004-12-21 11:17
> > Could you make your patchset available somehow ?
>
> http://people.suug.ch/~tgr/patches/queue/
>
> Unfinished and untested.
I just took a quick glimpse.
1)Recall: Policer will have to die at some point - only reason for its
existence is for backward compat.
New iproute2 code sooner than later stop using that inteface so we can
kill it. I suspect we can kill it in a year or two and definetely the
day 2.7 comes out.
2) The name tcf_attrs doesnt sound right - attributes are normally
data pieces not methods. Cant think of a good name.
3) What can i say? dang - this indev thing is getting out of control ;->
If you are going to go this far for beautification sake then
kill the .indev thing please before it becomes a beast. Do what we
discussed a while back:
- have a generic very basic extended generic match API which indev uses
that gets invoked from the classifier. It should take no more than one
page to write the indev extension - if it exceeds that you are doing
something wrong. There should be capability to mix and match these
extenders so i can say in u32 something like:
match ip src X
match extend indev src eth0
match protocol tcp
match extended metadata fwmark 0x10
etc.
I think its time we did this right than defering.
Of course all backward compatibility rules apply ;->
cheers,
jamal
|