[lvc-project] [PATCH v1] net/ethtool/ioctl: ensure that we have phy ops before using them
Andrew Lunn
andrew at lunn.ch
Tue Nov 15 18:40:07 MSK 2022
On Mon, Nov 14, 2022 at 09:07:05PM -0800, Jakub Kicinski wrote:
> On Mon, 14 Nov 2022 11:15:32 +0300 Daniil Tatianin wrote:
> > + if (!(phydev && phy_ops && phy_ops->get_stats) &&
> > + !ops->get_ethtool_phy_stats)
>
> This condition is still complicated.
>
> > + return -EOPNOTSUPP;
>
> The only way this crash can happen is if driver incorrectly returns
> non-zero stats count but doesn't have a callback to read the stats.
> So WARN_ON() would be in order here.
Hi Daniil
I'm missing the patch itself, and b4 does not return it. Please
consider reposting. Since this appear to be to do with PHY statistics,
you should Cc: the PHY maintainers.
Andrew
More information about the lvc-project
mailing list