All of lore.kernel.org
 help / color / mirror / Atom feed
From: lsorense@csclub.uwaterloo.ca (Lennart Sorensen)
To: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Ralph Sennhauser <ralph.sennhauser@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Gregory CLEMENT <gregory.clement@free-electrons.com>
Subject: Re: [Regression?] Commit cb4f71c429 deliberately changes order of network interfaces
Date: Wed, 24 Aug 2016 13:10:23 -0400	[thread overview]
Message-ID: <20160824171023.GD14311@csclub.uwaterloo.ca> (raw)
In-Reply-To: <20160824184334.281de5e0@free-electrons.com>

On Wed, Aug 24, 2016 at 06:43:34PM +0200, Thomas Petazzoni wrote:
> Well, just like the for the documentation aspect, you're seeing this
> from the OpenWRT/LEDE angle only. Other people are using plenty of
> other things.
> 
> We knew it would potentially cause some breakage, so it was a
> trade-off. I still believe the new arrangement is better, and you've so
> far been the only person reporting an issue with this (compared to
> numerous people being confused by the original ordering problem).

I didn't report it.  I just commented.  I will be affected when LEDE
does move their kernel past 4.4 I suppose, but I imagine something will
be done to deal with it.

I have run into issues with things being reordered on other systems
though, and it sure is annoying, especially when the new behaviour
removes the ability to control the order that was previously there
(that is not what is happening here of course).

> This is more problematic, and something to be investigated. I don't
> immediately see why the Marvell network interfaces would not be visible
> by udev, but I haven't tested.

Well certainly doing udevtrigger -n -v I see no ethernet devices (but
lots of other things).  Looking in sysfs it is possible to dereive which
ethX belongs to which port based on the directory names, but that's
probably not the most convinient manner to deal with it.

> The solution of adding an alias in the DT, and using that to name
> network interfaces has already been proposed multiple times, but has
> been rejected by the networking maintainer, who suggests to use
> userspace tools (udev or something else) to rename network interfaces.
> See for example https://patchwork.kernel.org/patch/4122441/, which was
> proposed by my colleague Boris Brezillon.

Sure, although it seems many embedded systems would rather avoid udev
(even more so after systemd seems to have taken it over), and in this
case I get the impression so far that udev may not even be able to help
on this system.  (although maybe I just did it wrong).

> You can always try to propose again a solution, but I doubt it will be
> accepted.

Yeah me too, hence why I can't be bothered to try.  I have accepted
there are some things the maintainers won't fix so I have a few patches 
I maintain myself and can keep doing so.  They are tiny after all.

-- 
Len Sorensen

  reply	other threads:[~2016-08-24 17:10 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-21 13:11 [Regression?] Commit cb4f71c429 deliberately changes order of network interfaces Ralph Sennhauser
2016-08-24 14:50 ` Thomas Petazzoni
2016-08-24 16:19   ` Lennart Sorensen
2016-08-24 16:43     ` Thomas Petazzoni
2016-08-24 17:10       ` Lennart Sorensen [this message]
2016-08-24 18:38         ` Lennart Sorensen
2016-08-24 17:10   ` Ralph Sennhauser
2016-08-24 18:07     ` Lennart Sorensen
2016-08-24 18:14       ` Thomas Petazzoni
2016-08-24 18:27         ` Lennart Sorensen
2016-08-24 19:52           ` Thomas Petazzoni
2016-08-24 20:51             ` Lennart Sorensen
2016-08-24 18:15     ` Thomas Petazzoni
2016-08-24 20:41       ` Ralph Sennhauser
2016-08-24 21:48         ` Jason Cooper
2016-08-25  7:38           ` Ralph Sennhauser
2016-08-25 19:40             ` Lennart Sorensen
2016-08-26  8:43             ` Gregory CLEMENT
2016-08-26 10:39               ` Ralph Sennhauser

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=20160824171023.GD14311@csclub.uwaterloo.ca \
    --to=lsorense@csclub.uwaterloo.ca \
    --cc=gregory.clement@free-electrons.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=ralph.sennhauser@gmail.com \
    --cc=thomas.petazzoni@free-electrons.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.