ice: Add ethtool private flag to make forcing link down optional

Add new infrastructure for implementing ethtool private flags using the
existing pf->flags bitmap to store them, and add the link-down-on-close
ethtool private flag to optionally bring down the PHY link when the
interface is administratively downed.

Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Anirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

1 files changed, 8 insertions, 6 deletions

diff --git a/drivers/net/ethernet/intel/ice/ice_main.c b/drivers/net/ethernet/intel/ice/ice_main.c
index b27bc7d6d605..e36db981f047 100644
--- a/drivers/net/ethernet/intel/ice/ice_main.c
+++ b/drivers/net/ethernet/intel/ice/ice_main.c
@@ -3029,7 +3029,7 @@ out:
  */
 int ice_down(struct ice_vsi *vsi)
 {
-        int i, tx_err, rx_err, link_err;
+        int i, tx_err, rx_err, link_err = 0;
 
         /* Caller of this function is expected to set the
          * vsi->state __ICE_DOWN bit
@@ -3054,11 +3054,13 @@ int ice_down(struct ice_vsi *vsi)
 
         ice_napi_disable_all(vsi);
 
-        link_err = ice_force_phys_link_state(vsi, false);
-        if (link_err)
-                netdev_err(vsi->netdev,
-                           "Failed to set physical link down, VSI %d error %d\n",
-                           vsi->vsi_num, link_err);
+        if (test_bit(ICE_FLAG_LINK_DOWN_ON_CLOSE_ENA, vsi->back->flags)) {
+                link_err = ice_force_phys_link_state(vsi, false);
+                if (link_err)
+                        netdev_err(vsi->netdev,
+                                   "Failed to set physical link down, VSI %d error %d\n",
+                                   vsi->vsi_num, link_err);
+        }
 
         ice_for_each_txq(vsi, i)
                 ice_clean_tx_ring(vsi->tx_rings[i]);