ice: Add ice_for_each_vf() macro

Currently we do "for (i = 0; i < pf->num_alloc_vfs; i++)" all over the
place. Many other places use macros to contain this repeated for loop,
So create the macro ice_for_each_vf(pf, i) that does the same thing.

There were a couple places we were using one loop variable and a VF
iterator, which were changed to using a local variable within the
ice_for_each_vf() macro.

Also in ice_alloc_vfs() we were setting pf->num_alloc_vfs after doing
"for (i = 0; i < num_alloc_vfs; i++)". Instead assign pf->num_alloc_vfs
right after allocating memory for the pf->vf array.

Signed-off-by: Brett Creeley <brett.creeley@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

diff --git a/drivers/net/ethernet/intel/ice/ice_ethtool.c b/drivers/net/ethernet/intel/ice/ice_ethtool.c
index 9ebd93e79aeb64889ea4447c47a5e631ed2765d6..b815f1df56988b1c19c96035116955136515c329 100644 (file)
--- a/drivers/net/ethernet/intel/ice/ice_ethtool.c
+++ b/drivers/net/ethernet/intel/ice/ice_ethtool.c
@@ -283,12 +283,15 @@ out:
  */
 static bool ice_active_vfs(struct ice_pf *pf)
 {
-       struct ice_vf *vf = pf->vf;
        int i;
 
-       for (i = 0; i < pf->num_alloc_vfs; i++, vf++)
+       ice_for_each_vf(pf, i) {
+               struct ice_vf *vf = &pf->vf[i];
+
                if (test_bit(ICE_VF_STATE_ACTIVE, vf->vf_states))
                        return true;
+       }
+
        return false;
 }
 

diff --git a/drivers/net/ethernet/intel/ice/ice_main.c b/drivers/net/ethernet/intel/ice/ice_main.c
index 639de467f120566701ce8bb1d748c3e463bae3bc..0c46e7fe72506492ed9406906cabe66b7fab7125 100644 (file)
--- a/drivers/net/ethernet/intel/ice/ice_main.c
+++ b/drivers/net/ethernet/intel/ice/ice_main.c
@@ -476,7 +476,7 @@ ice_prepare_for_reset(struct ice_pf *pf)
                ice_vc_notify_reset(pf);
 
        /* Disable VFs until reset is completed */
-       for (i = 0; i < pf->num_alloc_vfs; i++)
+       ice_for_each_vf(pf, i)
                ice_set_vf_state_qs_dis(&pf->vf[i]);
 
        /* clear SW filtering DB */
@@ -1295,7 +1295,7 @@ static void ice_handle_mdd_event(struct ice_pf *pf)
        }
 
        /* check to see if one of the VFs caused the MDD */
-       for (i = 0; i < pf->num_alloc_vfs; i++) {
+       ice_for_each_vf(pf, i) {
                struct ice_vf *vf = &pf->vf[i];
 
                bool vf_mdd_detected = false;

diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
index 3860e9c964b9c03c794b32b10fbe44d1b83a9e7c..88cb0f9e928e8851f17ce3e8872a2add18618c7c 100644 (file)
--- a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
@@ -78,10 +78,11 @@ ice_vc_vf_broadcast(struct ice_pf *pf, enum virtchnl_ops v_opcode,
                    enum virtchnl_status_code v_retval, u8 *msg, u16 msglen)
 {
        struct ice_hw *hw = &pf->hw;
-       struct ice_vf *vf = pf->vf;
        int i;
 
-       for (i = 0; i < pf->num_alloc_vfs; i++, vf++) {
+       ice_for_each_vf(pf, i) {
+               struct ice_vf *vf = &pf->vf[i];
+
                /* Not all vfs are enabled so skip the ones that are not */
                if (!test_bit(ICE_VF_STATE_INIT, vf->vf_states) &&
                    !test_bit(ICE_VF_STATE_ACTIVE, vf->vf_states))
@@ -331,7 +332,7 @@ void ice_free_vfs(struct ice_pf *pf)
                usleep_range(1000, 2000);
 
        /* Avoid wait time by stopping all VFs at the same time */
-       for (i = 0; i < pf->num_alloc_vfs; i++)
+       ice_for_each_vf(pf, i)
                if (test_bit(ICE_VF_STATE_QS_ENA, pf->vf[i].vf_states))
                        ice_dis_vf_qs(&pf->vf[i]);
 
@@ -1077,7 +1078,7 @@ static bool ice_config_res_vfs(struct ice_pf *pf)
                ice_irq_dynamic_ena(hw, NULL, NULL);
 
        /* Finish resetting each VF and allocate resources */
-       for (v = 0; v < pf->num_alloc_vfs; v++) {
+       ice_for_each_vf(pf, v) {
                struct ice_vf *vf = &pf->vf[v];
 
                vf->num_vf_qs = pf->num_vf_qps;
@@ -1120,10 +1121,10 @@ bool ice_reset_all_vfs(struct ice_pf *pf, bool is_vflr)
                return false;
 
        /* Begin reset on all VFs at once */
-       for (v = 0; v < pf->num_alloc_vfs; v++)
+       ice_for_each_vf(pf, v)
                ice_trigger_vf_reset(&pf->vf[v], is_vflr, true);
 
-       for (v = 0; v < pf->num_alloc_vfs; v++) {
+       ice_for_each_vf(pf, v) {
                struct ice_vsi *vsi;
 
                vf = &pf->vf[v];
@@ -1168,7 +1169,7 @@ bool ice_reset_all_vfs(struct ice_pf *pf, bool is_vflr)
                dev_warn(dev, "VF reset check timeout\n");
 
        /* free VF resources to begin resetting the VSI state */
-       for (v = 0; v < pf->num_alloc_vfs; v++) {
+       ice_for_each_vf(pf, v) {
                vf = &pf->vf[v];
 
                ice_free_vf_res(vf);
@@ -1308,7 +1309,7 @@ void ice_vc_notify_link_state(struct ice_pf *pf)
 {
        int i;
 
-       for (i = 0; i < pf->num_alloc_vfs; i++)
+       ice_for_each_vf(pf, i)
                ice_vc_notify_vf_link_state(&pf->vf[i]);
 }
 
@@ -1392,9 +1393,10 @@ static int ice_alloc_vfs(struct ice_pf *pf, u16 num_alloc_vfs)
                goto err_pci_disable_sriov;
        }
        pf->vf = vfs;
+       pf->num_alloc_vfs = num_alloc_vfs;
 
        /* apply default profile */
-       for (i = 0; i < num_alloc_vfs; i++) {
+       ice_for_each_vf(pf, i) {
                vfs[i].pf = pf;
                vfs[i].vf_sw_id = pf->first_sw;
                vfs[i].vf_id = i;
@@ -1403,7 +1405,6 @@ static int ice_alloc_vfs(struct ice_pf *pf, u16 num_alloc_vfs)
                set_bit(ICE_VIRTCHNL_VF_CAP_L2, &vfs[i].vf_caps);
                vfs[i].spoofchk = true;
        }
-       pf->num_alloc_vfs = num_alloc_vfs;
 
        /* VF resources get allocated with initialization */
        if (!ice_config_res_vfs(pf)) {
@@ -1542,7 +1543,7 @@ void ice_process_vflr_event(struct ice_pf *pf)
            !pf->num_alloc_vfs)
                return;
 
-       for (vf_id = 0; vf_id < pf->num_alloc_vfs; vf_id++) {
+       ice_for_each_vf(pf, vf_id) {
                struct ice_vf *vf = &pf->vf[vf_id];
                u32 reg_idx, bit_idx;
 

diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.h b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.h
index 611f45100438ce6ffa3c5a6258b68e1eba52cedb..4647d636ed36e07c921f5e4c903f43effde7bc93 100644 (file)
--- a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.h
+++ b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.h
@@ -40,6 +40,9 @@
 #define ICE_DFLT_INTR_PER_VF           (ICE_DFLT_QS_PER_VF + 1)
 #define ICE_MAX_VF_RESET_WAIT          15
 
+#define ice_for_each_vf(pf, i) \
+       for ((i) = 0; (i) < (pf)->num_alloc_vfs; (i)++)
+
 /* Specific VF states */
 enum ice_vf_states {
        ICE_VF_STATE_INIT = 0,          /* PF is initializing VF */