| To: | jamal <hadi@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 3/9] PKT_SCHED: u32: make use of tcf_exts API |
| From: | Thomas Graf <tgraf@xxxxxxx> |
| Date: | Fri, 31 Dec 2004 13:03:44 +0100 |
| Cc: | "David S. Miller" <davem@xxxxxxxxxxxxx>, Patrick McHardy <kaber@xxxxxxxxx>, netdev@xxxxxxxxxxx |
| In-reply-to: | <1104468201.1047.201.camel@jzny.localdomain> |
| References: | <20041230122652.GM32419@postel.suug.ch> <20041230123108.GP32419@postel.suug.ch> <1104468201.1047.201.camel@jzny.localdomain> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
* jamal <1104468201.1047.201.camel@xxxxxxxxxxxxxxxx> 2004-12-30 23:43
> On Thu, 2004-12-30 at 07:31, Thomas Graf wrote:
>
> >
> > +static struct tcf_ext_map u32_ext_map = {
> > + .action = TCA_U32_ACT,
> > + .police = TCA_U32_POLICE
> > +};
>
> Repeated on all classifiers - perhaps a little macro magic in order? ;->
I thought about this but couldn't come up with one that makes it
actually easier. I don't want to hardcode action/police in a macro
because we might have extensions only needed for specific classifiers.
> Generally rest all looking good - I need to stare some more at the route
> and tcindex one but dont see any show stoppers off hand;
Yes please, those are the critical points.
|
| Previous by Date: | Re: [PATCH PKT_SCHED 0/17]: tc action cleanup + fixes, Thomas Graf |
|---|---|
| Next by Date: | Re: [PATCH 2/9] PKT_SCHED: tc filter extension API, Thomas Graf |
| Previous by Thread: | Re: [PATCH 3/9] PKT_SCHED: u32: make use of tcf_exts API, jamal |
| Next by Thread: | [PATCH 4/9] PKT_SCHED: fw: make use of tcf_exts API, Thomas Graf |
| Indexes: | [Date] [Thread] [Top] [All Lists] |