netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Igor Maravić" <igorm@etf.rs>
To: David Miller <davem@davemloft.net>
Cc: netdev@vger.kernel.org
Subject: Re: MPLS for Linux kernel
Date: Wed, 23 Nov 2011 08:09:31 +0100	[thread overview]
Message-ID: <CAFdo_mVb4aCcWT8a9pk8Ypt9+GnLqSDDiPOAwOsgWZRcTH_NaQ@mail.gmail.com> (raw)
In-Reply-To: <20111122.164909.156852889818363753.davem@davemloft.net>

OK,
Thanks

2011/11/22 David Miller <davem@davemloft.net>:
> From: Igor Maravić <igorm@etf.rs>
> Date: Tue, 22 Nov 2011 22:41:44 +0100
>
>> 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 :-)
>

  reply	other threads:[~2011-11-23  7:09 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-20 21:59 MPLS for Linux kernel Igor Maravić
2011-11-21 15:01 ` Jorge Boncompte [DTI2]
2011-11-21 17:17   ` Stephen Hemminger
2011-11-21 17:46     ` Jorge Boncompte [DTI2]
2011-11-21 18:29   ` David Miller
2011-11-21 19:18     ` Jorge Boncompte [DTI2]
2011-11-22  8:52       ` Igor Maravić
2011-11-22 12:30         ` Jorge Boncompte [DTI2]
2011-11-22 13:55           ` Igor Maravić
2011-11-22 14:33             ` Jorge Boncompte [DTI2]
2011-11-22 14:35               ` Jorge Boncompte [DTI2]
2011-11-22 15:51               ` Igor Maravić
2011-11-22 21:41                 ` Igor Maravić
2011-11-22 21:49                   ` David Miller
2011-11-23  7:09                     ` Igor Maravić [this message]
2011-11-24 23:39                     ` Glen Turner
2011-11-25  5:43                       ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFdo_mVb4aCcWT8a9pk8Ypt9+GnLqSDDiPOAwOsgWZRcTH_NaQ@mail.gmail.com \
    --to=igorm@etf.rs \
    --cc=davem@davemloft.net \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).