>> What do you think? > > Passing -1 is an error, it doesn't make sense to try and be > helpful to buggy userland.
Here is commit I actually change/fix: commit 9c7dafbfab15 ("net: Allow to create links with given ifindex") In this change done the opposite: check for ifm->ifi_index moved to register_netdevice() from rtnl_newlink(). Let me understand finally: why do I need to reverse and move check for dev->ifindex from register_netdevice() to rtnl_newlink()? That will partially revert commit I note above. -- Thanks, Serhey