Subject: Re: [PATCH net-next v7 02/10] net: ethtool: add support for configuring hds-thresh

From: Jakub Kicinski <[email-masked]>
Date: Mon, 6 Jan 2025 19:04:56 -0800

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.