From: Stefano Brivio <sbrivio@redhat•com>
To: Fernando Fernandez Mancera <fmancera@suse•de>
Cc: netdev@vger•kernel.org, yuhuang@redhat•com,
justin.iurman@gmail•com, horms@kernel•org, pabeni@redhat•com,
kuba@kernel•org, edumazet@google•com, davem@davemloft•net,
idosch@nvidia•com, dsahern@kernel•org,
Chris Adams <linux@cmadams•net>,
David Gibson <david@gibson•dropbear.id.au>,
Beniamino Galvani <bgalvani@redhat•com>,
Thorsten Leemhuis <regressions@leemhuis•info>,
Andrew Lunn <andrew@lunn•ch>,
ihuguet@redhat•com, regressions@lists•linux.dev
Subject: Re: [PATCH net v2] Revert "ipv6: preserve insertion order for same-scope addresses"
Date: Fri, 29 May 2026 13:41:02 +0200 (CEST) [thread overview]
Message-ID: <20260529134045.56330243@elisabeth> (raw)
In-Reply-To: <20260529112357.5079-1-fmancera@suse.de>
On Fri, 29 May 2026 13:23:57 +0200
Fernando Fernandez Mancera <fmancera@suse•de> wrote:
> Chris Adams reported that preserving insertion order for same-scope
> addresses is causing SSH connections to be dropped after stopping a VM
> while running NetworkManager.
>
> NetworkManager caches the IPv6 address configuration, when a RA arrives,
> it determines the list of addresses to configure and checks if the
> addresses are already in the right order in the kernel. If they aren't,
> NetworkManager removes and re-adds them to achieve the desired order.
>
> As the order changes, NetworkManager is confused and reconfigures the
> addresses on every update. In addition, this would also affect to cloud
> tooling that relies on IPv6 addresses order to identify primary and
> secondaries addresses.
By the way, I'm still looking into this part, trying to find
"problematic" examples.
And I couldn't find any, yet, because it looks like there's always a
_single_ IPv6 address being used as a secondary for a primary IPv4
address.
>
> This reverts commit cb3de96eea66f5e4a580086c6a1be46e765f97f4.
>
> Fixes: cb3de96eea66 ("ipv6: preserve insertion order for same-scope addresses")
> Reported-by: Chris Adams <linux@cmadams•net>
> Closes: https://lore.kernel.org/netdev/20260521135310.GC977@cmadams.net/
> Signed-off-by: Fernando Fernandez Mancera <fmancera@suse•de>
> ---
> v2: updated commit description to make it more accurate
> ---
> net/ipv6/addrconf.c | 2 +-
> tools/testing/selftests/net/ioam6.sh | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c
> index 5476b6536eb7..bb84a78b80f6 100644
> --- a/net/ipv6/addrconf.c
> +++ b/net/ipv6/addrconf.c
> @@ -1013,7 +1013,7 @@ ipv6_link_dev_addr(struct inet6_dev *idev, struct inet6_ifaddr *ifp)
> list_for_each(p, &idev->addr_list) {
> struct inet6_ifaddr *ifa
> = list_entry(p, struct inet6_ifaddr, if_list);
> - if (ifp_scope > ipv6_addr_src_scope(&ifa->addr))
> + if (ifp_scope >= ipv6_addr_src_scope(&ifa->addr))
> break;
> }
>
> diff --git a/tools/testing/selftests/net/ioam6.sh b/tools/testing/selftests/net/ioam6.sh
> index b2b99889942f..845c26dd01a9 100755
> --- a/tools/testing/selftests/net/ioam6.sh
> +++ b/tools/testing/selftests/net/ioam6.sh
> @@ -273,8 +273,8 @@ setup()
> ip -netns $ioam_node_beta link set ioam-veth-betaR name veth1 &>/dev/null
> ip -netns $ioam_node_gamma link set ioam-veth-gamma name veth0 &>/dev/null
>
> - ip -netns $ioam_node_alpha addr add 2001:db8:1::2/64 dev veth0 &>/dev/null
> ip -netns $ioam_node_alpha addr add 2001:db8:1::50/64 dev veth0 &>/dev/null
> + ip -netns $ioam_node_alpha addr add 2001:db8:1::2/64 dev veth0 &>/dev/null
> ip -netns $ioam_node_alpha link set veth0 up &>/dev/null
> ip -netns $ioam_node_alpha link set lo up &>/dev/null
> ip -netns $ioam_node_alpha route add 2001:db8:2::/64 \
--
Stefano
next prev parent reply other threads:[~2026-05-29 11:41 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-29 11:23 [PATCH net v2] Revert "ipv6: preserve insertion order for same-scope addresses" Fernando Fernandez Mancera
2026-05-29 11:41 ` Stefano Brivio [this message]
2026-05-29 11:45 ` Fernando Fernandez Mancera
2026-05-29 12:06 ` Chris Adams
2026-06-01 2:03 ` Matthieu Baerts
2026-06-01 13:35 ` Stefano Brivio
2026-06-01 14:01 ` Íñigo Huguet
2026-06-01 14:22 ` Thorsten Leemhuis
2026-06-03 5:46 ` Matthieu Baerts
2026-06-03 6:53 ` Íñigo Huguet
2026-06-03 7:17 ` Thorsten Leemhuis
2026-06-03 7:29 ` Fernando Fernandez Mancera
2026-06-03 8:00 ` Ido Schimmel
2026-06-03 8:06 ` Fernando Fernandez Mancera
2026-06-03 9:27 ` Matthieu Baerts
2026-06-03 8:02 ` David Gibson
2026-06-02 6:44 ` IPv6 address insertion order (was Re: [PATCH net v2] Revert "ipv6: preserve insertion order for same-scope addresses") David Gibson
2026-06-02 12:46 ` Andrew Lunn
2026-06-03 1:56 ` David Gibson
2026-06-02 13:21 ` Ido Schimmel
2026-06-03 2:34 ` David Gibson
2026-06-03 7:47 ` Ido Schimmel
2026-06-03 15:45 ` Stefano Brivio
2026-06-04 1:26 ` David Gibson
2026-06-04 18:39 ` Ido Schimmel
2026-06-04 22:55 ` Jakub Kicinski
2026-06-05 10:13 ` Stefano Brivio
2026-06-03 15:47 ` Nicolas Dichtel
2026-05-29 20:20 ` [PATCH net v2] Revert "ipv6: preserve insertion order for same-scope addresses" patchwork-bot+netdevbpf
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=20260529134045.56330243@elisabeth \
--to=sbrivio@redhat$(echo .)com \
--cc=andrew@lunn$(echo .)ch \
--cc=bgalvani@redhat$(echo .)com \
--cc=davem@davemloft$(echo .)net \
--cc=david@gibson$(echo .)dropbear.id.au \
--cc=dsahern@kernel$(echo .)org \
--cc=edumazet@google$(echo .)com \
--cc=fmancera@suse$(echo .)de \
--cc=horms@kernel$(echo .)org \
--cc=idosch@nvidia$(echo .)com \
--cc=ihuguet@redhat$(echo .)com \
--cc=justin.iurman@gmail$(echo .)com \
--cc=kuba@kernel$(echo .)org \
--cc=linux@cmadams$(echo .)net \
--cc=netdev@vger$(echo .)kernel.org \
--cc=pabeni@redhat$(echo .)com \
--cc=regressions@leemhuis$(echo .)info \
--cc=regressions@lists$(echo .)linux.dev \
--cc=yuhuang@redhat$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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