Commit c01163dbd1b8 ("PCI/PM: Always disable PTM for all devices during
suspend") made the PCI core to suspend (disable) PTM before driver
suspend hooks are called. In case of igc what happens is that on suspend
path PCI core calls pci_suspend_ptm() then igc suspend hook that calls
igc_down() that ends up calling igc_ptp_reset() (which according to the
comment is actually needed for re-enabling the device). Anyways that
function also poll IGC_PTM_STAT that will end up timing out because PTM
is already disabled:[ 160.716119] igc 0000:03:00.0 enp3s0: Timeout reading IGC_PTM_STAT register There should be no reason resetting the hardware on suspend path so fix this by avoiding the reset. Signed-off-by: Mika Westerberg <[email protected]> --- drivers/net/ethernet/intel/igc/igc.h | 2 +- drivers/net/ethernet/intel/igc/igc_ethtool.c | 6 +++--- drivers/net/ethernet/intel/igc/igc_main.c | 13 +++++++------ 3 files changed, 11 insertions(+), 10 deletions(-) diff --git a/drivers/net/ethernet/intel/igc/igc.h b/drivers/net/ethernet/intel/igc/igc.h index a427f05814c1..21d6eba1cc09 100644 --- a/drivers/net/ethernet/intel/igc/igc.h +++ b/drivers/net/ethernet/intel/igc/igc.h @@ -349,7 +349,7 @@ struct igc_adapter { }; void igc_up(struct igc_adapter *adapter); -void igc_down(struct igc_adapter *adapter); +void igc_down(struct igc_adapter *adapter, bool reset); int igc_open(struct net_device *netdev); int igc_close(struct net_device *netdev); int igc_setup_tx_resources(struct igc_ring *ring); diff --git a/drivers/net/ethernet/intel/igc/igc_ethtool.c b/drivers/net/ethernet/intel/igc/igc_ethtool.c index 0122009bedd0..4d1bcc19255f 100644 --- a/drivers/net/ethernet/intel/igc/igc_ethtool.c +++ b/drivers/net/ethernet/intel/igc/igc_ethtool.c @@ -638,7 +638,7 @@ igc_ethtool_set_ringparam(struct net_device *netdev, goto clear_reset; } - igc_down(adapter); + igc_down(adapter, true); /* We can't just free everything and then setup again, * because the ISRs in MSI-X mode get passed pointers @@ -737,7 +737,7 @@ static int igc_ethtool_set_pauseparam(struct net_device *netdev, if (adapter->fc_autoneg == AUTONEG_ENABLE) { hw->fc.requested_mode = igc_fc_default; if (netif_running(adapter->netdev)) { - igc_down(adapter); + igc_down(adapter, true); igc_up(adapter); } else { igc_reset(adapter); @@ -2077,7 +2077,7 @@ igc_ethtool_set_link_ksettings(struct net_device *netdev, /* reset the link */ if (netif_running(adapter->netdev)) { - igc_down(adapter); + igc_down(adapter, true); igc_up(adapter); } else { igc_reset(adapter); diff --git a/drivers/net/ethernet/intel/igc/igc_main.c b/drivers/net/ethernet/intel/igc/igc_main.c index 99a4c99ddd57..84f5d92c81cc 100644 --- a/drivers/net/ethernet/intel/igc/igc_main.c +++ b/drivers/net/ethernet/intel/igc/igc_main.c @@ -5308,8 +5308,9 @@ void igc_update_stats(struct igc_adapter *adapter) /** * igc_down - Close the interface * @adapter: board private structure + * @reset: issue reset */ -void igc_down(struct igc_adapter *adapter) +void igc_down(struct igc_adapter *adapter, bool reset) { struct net_device *netdev = adapter->netdev; struct igc_hw *hw = &adapter->hw; @@ -5365,7 +5366,7 @@ void igc_down(struct igc_adapter *adapter) adapter->link_speed = 0; adapter->link_duplex = 0; - if (!pci_channel_offline(adapter->pdev)) + if (reset && !pci_channel_offline(adapter->pdev)) igc_reset(adapter); /* clear VLAN promisc flag so VFTA will be updated if necessary */ @@ -5383,7 +5384,7 @@ void igc_reinit_locked(struct igc_adapter *adapter) { while (test_and_set_bit(__IGC_RESETTING, &adapter->state)) usleep_range(1000, 2000); - igc_down(adapter); + igc_down(adapter, true); igc_up(adapter); clear_bit(__IGC_RESETTING, &adapter->state); } @@ -5437,7 +5438,7 @@ static int igc_change_mtu(struct net_device *netdev, int new_mtu) adapter->max_frame_size = max_frame; if (netif_running(netdev)) - igc_down(adapter); + igc_down(adapter, true); netdev_dbg(netdev, "changing MTU from %d to %d\n", netdev->mtu, new_mtu); WRITE_ONCE(netdev->mtu, new_mtu); @@ -6301,7 +6302,7 @@ static int __igc_close(struct net_device *netdev, bool suspending) if (!suspending) pm_runtime_get_sync(&pdev->dev); - igc_down(adapter); + igc_down(adapter, !suspending); igc_release_hw_control(adapter); @@ -7628,7 +7629,7 @@ static pci_ers_result_t igc_io_error_detected(struct pci_dev *pdev, } if (netif_running(netdev)) - igc_down(adapter); + igc_down(adapter, true); pci_disable_device(pdev); rtnl_unlock(); -- 2.50.1
