Subject: Re: [PATCH v3 net-next 1/4] rtnetlink: enable alt_ifname for setlink/newlink

From: Jakub Kicinski <[email-masked]>
Date: Thu, 7 Apr 2022 09:11:19 -0700

On Thu, 7 Apr 2022 14:34:33 +0200 Florent Fourcot wrote:
> > This patch needs to be after patch 3, AFAICT. Otherwise someone running
> > a bisection and landing in between the two will have a buggy build.
> 
> I double checked and this patch compiles perfectly fine when applied 
> first, at least with my .config.
> Could you be more specific on what I'm missing?

rtnl_group_changelink() was passing NULL as ifname, but with just 
this patch applied we'll start using tb[IFLA_IFNAME] directly. 

It's better to have your patch which rejects group settings with
tb[IFLA_IFNAME] first, to avoid changing the group + IFLA_IFNAME
behavior subtly multiple times.