On Sun, Aug 31, 2025 at 02:06:19AM +0900, Kohei Enju wrote:
> The current implementation of igc driver doesn't power up PHY before
> link test in igc_ethtool_diag_test(), causing the link test to always
> report FAIL when admin state is down and PHY is consequently powered
> down.
>
> To test the link state regardless of admin state, let's power up PHY in
> case of PHY down before link test.
Maybe you should power the PHY down again after the test?
Alternatively, just return -ENOTDOWN is the network is admin down.
Andrew
