netdev - Re: [PATCH net] i40e: Prevent unwanted interface name changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <e71fe3bf-ec97-431e-b60c-634c5263ad82@intel.com>
Date: Thu, 21 Aug 2025 10:00:51 +0200
From: Przemek Kitszel <przemyslaw.kitszel@...el.com>
To: Calvin Owens <calvin@...nvd.org>, Michal Schmidt <mschmidt@...hat.com>
CC: <netdev@...r.kernel.org>, Tony Nguyen <anthony.l.nguyen@...el.com>,
	"Andrew Lunn" <andrew+netdev@...n.ch>, "David S. Miller"
	<davem@...emloft.net>, "Eric Dumazet" <edumazet@...gle.com>, Jakub Kicinski
	<kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, Jedrzej Jagielski
	<jedrzej.jagielski@...el.com>, "Ivan Vecera" <ivecera@...hat.com>,
	<intel-wired-lan@...ts.osuosl.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH net] i40e: Prevent unwanted interface name changes

On 8/20/25 19:41, Calvin Owens wrote:
> On Wednesday 08/20 at 08:31 -0700, Calvin Owens wrote:
>> On Wednesday 08/20 at 08:42 +0200, Michal Schmidt wrote:
>>> On Wed, Aug 20, 2025 at 6:30 AM Calvin Owens <calvin@...nvd.org> wrote:
>>>> The same naming regression which was reported in ixgbe and fixed in
>>>> commit e67a0bc3ed4f ("ixgbe: prevent from unwanted interface name
>>>> changes") still exists in i40e.
>>>>
>>>> Fix i40e by setting the same flag, added in commit c5ec7f49b480
>>>> ("devlink: let driver opt out of automatic phys_port_name generation").
>>>>
>>>> Fixes: 9e479d64dc58 ("i40e: Add initial devlink support")
>>>
>>> But this one's almost two years old. By now, there may be more users
>>> relying on the new name than on the old one.
>>> Michal
>>
>> Well, I was relying on the new ixgbe names, and I had to revert them
>> all in a bunch of configs yesterday after e67a0bc3ed4f :)

we have fixed (changed to old naming scheme) ixgbe right after the
kernel was used by real users (modulo usual delay needed to invent
a good solution)

> 
> And, even if it is e67a0bc3ed4f that introduced it, v6.7 was the first
> release with it. I strongly suspect most servers with i40e NICs running
> in the wild are running older kernels than that, and have not yet
> encountered the naming regression. But you probably have much better
> data about that than I do :)

RedHat patches their kernels with current code of the drivers that their
customers use (including i40e and ixgbe)
One could expect that changes made today to those will reach RHEL 10.3,
even if it would be named "kernel 6.12".

(*) the changes will likely be also in 10.2, but I don't want to make
any promises from Intel or Redhat here