Commit ab7470bc authored by Ahmed Zaki's avatar Ahmed Zaki Committed by Tony Nguyen
Browse files

ice: Fix stats after PF reset 



After a core PF reset, the VFs were showing wrong Rx/Tx stats. This is a
regression in commit 6624e780 ("ice: split ice_vsi_setup into smaller
functions") caused by missing to set "stat_offsets_loaded = false" in the
ice_vsi_rebuild() path.

Fixes: 6624e780 ("ice: split ice_vsi_setup into smaller functions")
Signed-off-by: default avatarAhmed Zaki <ahmed.zaki@intel.com>
Reviewed-by: default avatarAlexander Lobakin <aleksander.lobakin@intel.com>
Tested-by: default avatarRafal Romanowski <rafal.romanowski@intel.com>
Reviewed-by: default avatarLeon Romanovsky <leonro@nvidia.com>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 225c6579

drivers/net/ethernet/intel/ice/ice_lib.c

+5 −0
Original line number Diff line number Diff line
@@ -2745,6 +2745,8 @@ ice_vsi_cfg_def(struct ice_vsi *vsi, struct ice_vsi_cfg_params *params) 
			goto unroll_vector_base;



		ice_vsi_map_rings_to_vectors(vsi);

		vsi->stat_offsets_loaded = false;



		if (ice_is_xdp_ena_vsi(vsi)) {

			ret = ice_vsi_determine_xdp_res(vsi);

			if (ret)
@@ -2793,6 +2795,9 @@ ice_vsi_cfg_def(struct ice_vsi *vsi, struct ice_vsi_cfg_params *params) 
		ret = ice_vsi_alloc_ring_stats(vsi);

		if (ret)

			goto unroll_vector_base;



		vsi->stat_offsets_loaded = false;



		/* Do not exit if configuring RSS had an issue, at least

		 * receive traffic on first queue. Hence no need to capture

		 * return value