Re: [PATCH net-next] net: stmmac: allow ethtool action on PCI devices if device is down

[Corinna Vinschen <vinschen@redhat.com>]

On Apr 25 06:54, Jakub Kicinski wrote:
> On Tue, 25 Apr 2023 10:14:01 +0200 Corinna Vinschen wrote:
> > is that patch still under review or is it refused, given the
> > current behaviour is not actually incorrect?
> 
> That's not what Heiner said. Does the device not link the netdev
> correctly to the bus device?

No, that's not the problem.

> Core already does pm_get/pm_put
> around ethtool calls.

Yeah, right.  I made a more thorough test now and it appears that
the calls don't even have any effect.

For testing I created a driver which simply skips the check for
netdev_running() and compared with the driver including my patch.
I tested almost all ethtool calls fetching and setting values and
there's no difference in behaviour.

Worse, while it's possible to change a lot of settings when skipping the
netdev_running() check, , some really basic stuff doesn't work as
desired.  I. e., setting autoneg or a fixed speed while the interface is
down is taken without complaints, but it has no effect when the
interface goes up again.

So we can just scratch my patch.  Sorry wasting your time.


Corinna