On Mon, 2005-25-07 at 22:09 +0200, Thomas Graf wrote:
> * David S. Miller <[EMAIL PROTECTED]> 2005-07-25 12:09
> > Ok, so can we agree on the following patch?  ing_filter() should
> > never see a NULL ->input_dev, and we could add a BUG() check
> > there for that invariant if you want.
> 
> This looks good to me, a input_dev==NULL means the packets
> origin is local. 

Not necessarily true. But shouldnt matter in this case - that code
should just set it.

> The code block which uses input_dev is dead, TC_NCLS is never
> set as of now. Jamal might have use for it though? Generally,
> tcf_action_exec() may be called on both ingress and egress.

I mentioned the meta action already; the dummy device replacement
for IMQ also uses it.
The ematch shouldnt really care about interpretting it - it should just
classify on it.

cheers,
jamal

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to