On 30.04, Daniel Borkmann wrote: > On 04/30/2015 01:36 AM, Patrick McHardy wrote: > ... > >You obviously realize this callchain is fully made up by yourself > > Hence, I wrote path, not call chain, I guess that should have been > clear. > > [...] > >The difference is very simple: where we had an indirect call to > >q->enqueue before (and a lot of crap that didn't belong there), > >we now have a call to nf_hook_slow, followed by the hook invocation. > > Sure, but what I wanted to express is that from an architectural > point of view down to invoking a classifier, we now have a list of > hooks, one element of those can be ingress qdisc and that one can > have lists of classifier/actions by itself, etc. That doesn't seem > sound wrt 'where it really belongs'. > > Personally, I have no objections if you want to use netfilter on > ingress, don't get me wrong, but that ingress qdisc part doesn't > really fit in there, and moving it behind netfilter facilities does > have additional cost for a packet. It does, and Pablo made up for this cost by moving the TTL tracking to ingress where it actually belongs. This would obviously also be possible without his patchset, but the fact is, nobody seemed to have cared for the past ten years, we have from today a basically zero cost feature and people can decide what the want for themselves, there is no added cost for people not using it, there is no added cost for people using ingress. As for "really fit", ingress started off as living on a netfilter hook and only moved to a direct invocation somewhere around 2.6.12 IIRC. As I was mentioning earlier, ingress *qdisc* is a hack in itself since there is no freaking queuing, its a hack to invoke classifier chains, so actually netfilter is *the better* abstraction to do this. And it's not netfilter, its packet hooks for classification purposes. To me all these arguments have to very negative sounds. It has always been like this - and it hasn't. And we want ingress queueing to be the exclusive way to do this. And this quite frankly sucks for all the reasons stated before. Its a hack, its not queueing, it doesn't cost anything, let the people decide what they actually want to use. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
