On Mon, 6 Jan 2025 18:48:54 -0800 Jakub Kicinski wrote:
> > * @module_fw_flash_in_progress: Module firmware flashing is in progress.
> > @@ -1141,6 +1148,7 @@ int ethtool_virtdev_set_link_ksettings(struct net_device *dev,
> > struct ethtool_netdev_state {
> > struct xarray rss_ctx;
> > struct mutex rss_lock;
> > + u32 hds_thresh;
>
> this value is checked in devmem.c but nothing ever sets it.
> net/ethtool/rings.c needs to handle it like it handles
> dev->ethtool->hds_config
Oh, I see you set it in the driver in patch 8.
That should work, my only concern is that this is not how
any of the other ethtool config options work today.
And there isn't any big warning in the code here telling
driver authors that they are responsible for the state update.
So even tho your patches are correct I still think it's better
to handle it like hds_config, just for consistency.
