igb: fix link test skipping when interface is admin down
authorKohei Enju <enjuk@amazon.com>
Fri, 15 Aug 2025 06:26:31 +0000 (15:26 +0900)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Tue, 9 Sep 2025 16:48:30 +0000 (09:48 -0700)
commitd709f178abca22a4d3642513df29afe4323a594b
tree8ccc98f6e22daae927e95522dc8fbf75aed8f5f3
parent75871a525a596ff4d16c4aebc0018f8d0923c9b1
igb: fix link test skipping when interface is admin down

The igb driver incorrectly skips the link test when the network
interface is admin down (if_running == false), causing the test to
always report PASS regardless of the actual physical link state.

This behavior is inconsistent with other drivers (e.g. i40e, ice, ixgbe,
etc.) which correctly test the physical link state regardless of admin
state.
Remove the if_running check to ensure link test always reflects the
physical link state.

Fixes: 8d420a1b3ea6 ("igb: correct link test not being run when link is down")
Signed-off-by: Kohei Enju <enjuk@amazon.com>
Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>
Tested-by: Rinitha S <sx.rinitha@intel.com> (A Contingent worker at Intel)
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/igb/igb_ethtool.c