Re: [PATCH net-next 2/2] net/mlx5e: Log error messages when extack is not present

[Jakub Kicinski <kuba@kernel.org>]

On Wed, 4 Jun 2025 00:01:05 +0300 Yael Chemla wrote:
> Ethtool APIs: While Netlink support was introduced around versions
> 5.6–5.8, many LTS distributions (e.g., Ubuntu 20.04, CentOS 7) still
> ship with older userspace ethtool utilities that rely on ioctl for
> certain operations. In these ioctl-based paths, the extack pointer
> passed down to the driver may legitimately be NULL.

Ubuntu 20.04 is EOL, and so is CentOS 7.

> TC APIs: Even though TC is predominantly Netlink-based, extack can still
> be NULL in certain internal driver code, legacy handling, or specific
> test/debug scenarios.
> 
> If a narrower scope is preferred, I can revise the patch to include only
> the ethtool-related changes, which were the primary motivation behind
> this work.

Do you expect the users to update their kernels but not be able 
to update the CLI? Finding and migrating the stragglers to netlink
is probably a better time investment. The "print to logs if extack 
is NULL" idea keeps coming back since the early days of extack.
We filled the few remaining gaps since then (mostly the ability
to use formatted strings). We should only break the general policy
of "users should switch to netlink if they want meaningful errors"
for really narrow, and well defined cases.