i40e: remove unnecessary msleep() delay in i40e_free_vfs
authorJacob Keller <jacob.e.keller@intel.com>
Wed, 19 Apr 2017 13:25:53 +0000 (09:25 -0400)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Sun, 30 Apr 2017 11:19:55 +0000 (04:19 -0700)
The delay was added because of a desire to ensure that the VF driver can
finish up removing. However, pci_disable_sriov already has its own
ssleep() call that will sleep for an entire second, so there is no
reason to add extra delay on top of this by using msleep here. In
practice, an msleep() won't have a huge impact on timing but there is no
real value in keeping it, so lets just simplify the code and remove it.

Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40e/i40e_main.c
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
drivers/net/ethernet/intel/i40evf/i40evf_main.c

index f44affc7e08cc36dba3d71036f8fb17d78fc2ac6..20850a646e6cefb58005c0ebe2fc4de49c87c561 100644 (file)
@@ -47,7 +47,7 @@ static const char i40e_driver_string[] =
 
 #define DRV_VERSION_MAJOR 2
 #define DRV_VERSION_MINOR 1
-#define DRV_VERSION_BUILD 7
+#define DRV_VERSION_BUILD 14
 #define DRV_VERSION __stringify(DRV_VERSION_MAJOR) "." \
             __stringify(DRV_VERSION_MINOR) "." \
             __stringify(DRV_VERSION_BUILD)    DRV_KERN
index 2a47a6474366bf616242635e740e713fb9a3b776..29f53f032c3c375e4e1f68f4d21ea34881853ce6 100644 (file)
@@ -1219,8 +1219,6 @@ void i40e_free_vfs(struct i40e_pf *pf)
        else
                dev_warn(&pf->pdev->dev, "VFs are assigned - not disabling SR-IOV\n");
 
-       msleep(20); /* let any messages in transit get finished up */
-
        /* free up VF resources */
        tmp = pf->num_alloc_vfs;
        pf->num_alloc_vfs = 0;
index 5915273c372f8d7f435f897fe68650239925084d..3ea81bd0db32cdd12e499b1dc6a2d2f30b5043ce 100644 (file)
@@ -46,7 +46,7 @@ static const char i40evf_driver_string[] =
 
 #define DRV_VERSION_MAJOR 2
 #define DRV_VERSION_MINOR 1
-#define DRV_VERSION_BUILD 7
+#define DRV_VERSION_BUILD 14
 #define DRV_VERSION __stringify(DRV_VERSION_MAJOR) "." \
             __stringify(DRV_VERSION_MINOR) "." \
             __stringify(DRV_VERSION_BUILD) \