Netdev Archive on lore.kernel.org
help / color / mirror / Atom feed
* Re: Interface naming on v5.14-rc1 regressed, possible unintended revert of b28d8f0c25a9?
       [not found] ` <PH0PR12MB54813D21F0F3338CD7764DE3DC149@PH0PR12MB5481.namprd12.prod.outlook.com>
@ 2021-07-19 10:49   ` Niklas Schnelle
  0 siblings, 0 replies; only message in thread
From: Niklas Schnelle @ 2021-07-19 10:49 UTC (permalink / raw)
  To: Parav Pandit, David S.Miller; +Cc: netdev, Stefan Raspl, Viktor Mihajlovski

On Tue, 2021-07-13 at 16:13 +0000, Parav Pandit wrote:
> Hi Niklas,
> 
> > From: Niklas Schnelle <schnelle@linux.ibm.com>
> > Sent: Tuesday, July 13, 2021 9:18 PM
> > 
> > Hi Parav, Hi Dave,
> > 
> > With v5.13 you added commit b28d8f0c25a9 ("devlink: Correct VIRTUAL port
> > to not have phys_port attributes") which changed the interface names of
> > VFs from e.g. ens2f0np0v0 to ens2f0v0. I did see that change for the v5.13-rc
> > cycle.
> > 
> > Now I just noticed that with v5.14-rc1 the old name is back. Looking at the
> > above commit and the code in v5.14-rc1 it looks to me like that commit was
> > partially reverted. In particular, the return of -EOPNOTSUPP from
> > __devlink_port_phys_port_name_get() in case of port flavour "virtual" is
> > gone while the part touching devlink_nl_port_attrs_put() seems to be there.
> > 
> > Now for some reason looking at git blame I still see the original commit
> > acf1ee44ca5da ("devlink: Introduce devlink port flavour
> > virtual") for the non removed lines. Could it be that somehow this got lost
> > while resolving a merge conflict?
> > 
> 
> Thanks for the bisecting and reporting.
> I searched further.
> I found that, it got introduced by below merge commit.
> but I haven’t yet found the patch that removed the return of -EOPNOTSUPP.
> 
> commit 126285651b7f95282a0afe3a1b0221419b31d989
> Merge: 9977d6f56bac 3822d0670c9d
> Author: David S. Miller <davem@davemloft.net>
> Date:   Mon Jun 7 13:01:52 2021 -0700
> 
>     Merge ra.kernel.org:/pub/scm/linux/kernel/git/netdev/net
> 
>     Bug fixes overlapping feature additions and refactoring, mostly.
> 
>     Signed-off-by: David S. Miller <davem@davemloft.net>

Just a FYI, I still see interface names with '..np0' suffix on v5.14-
rc2. Any progress on tracking down what happened? Also I think I
accidentially used the wrong mail address for the netdev list before,
sorry about that.


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-07-19 10:49 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1a11d3079ef3f69627e6e5ef244448a704babae3.camel@linux.ibm.com>
     [not found] ` <PH0PR12MB54813D21F0F3338CD7764DE3DC149@PH0PR12MB5481.namprd12.prod.outlook.com>
2021-07-19 10:49   ` Interface naming on v5.14-rc1 regressed, possible unintended revert of b28d8f0c25a9? Niklas Schnelle

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).