Thread (17 messages) flat view 17 messages, 5 authors, 2011-11-25

Re: MPLS for Linux kernel

From: Igor Maravić <hidden>
Date: 2011-11-23 07:09:36

OK,
Thanks

2011/11/22 David Miller [off-list ref]:
From: Igor Maravić <redacted>
Date: Tue, 22 Nov 2011 22:41:44 +0100
quoted
I would like to know what is necesary for MPLS implementation to have,
and to do, so it would be accepted in upstream kernel?
A long and laborious back and forth review process, taking into consideration
not just the technical details of the patches themselves, but the top level
and overall design.

That's what it will take.

Taking someone else's work, fixing all the bugs and cleaning them up is
far from sufficient for a feature of this nature.  There is natural
overlap all over and we have to make sure the implementation bits are
going into the right places.

One issue of constant contention is that people want to add all of their
favorite packet filtering and packet mangling into their protocol handling
code, with all kinds of custom controls and configuration mechanisms.

WE HATE THIS.

We have the packet scheduler classifiers and packet actions for a reason,
and we want them to used instead of ignored.

We are going through the same thing in the review process for the openvswitch
code, which brings up another design question for MPLS, which is whether MPLS
can be better implemented in terms of openvswitch.

You're in the unfortunate position of submitting a feature that has a
lot of overlap with many other subsystems, existing code, and features
being submitted at the same time.  We want as much reuse as possible,
and we want it all designed right before it gets integrated.

I frankly don't care very much about MPLS personally, it's such a
fringe facility.  So if people just argue themselves into oblivion and
no forward progress is made, just like last time an MPLS submission
was attempted, that's also fine with me :-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help