> -----Original Message----- > From: Intel-wired-lan <[email protected]> On Behalf Of > Jacob Keller > Sent: Friday, February 16, 2024 11:07 PM > To: Nguyen, Anthony L <[email protected]>; Intel Wired LAN > <[email protected]> > Cc: Keller, Jacob E <[email protected]>; Kitszel, Przemyslaw > <[email protected]> > Subject: [Intel-wired-lan] [PATCH iwl-next 2/4] ice: remove unnecessary > duplicate checks for VF VSI ID > > The ice_vc_fdir_param_check() function validates that the VSI ID of the > virtchnl flow director command matches the VSI number of the VF. This is > already checked by the call to ice_vc_isvalid_vsi_id() immediately following > this. > > This check is unnecessary since ice_vc_isvalid_vsi_id() already confirms this > by > checking that the VSI ID can locate the VSI associated with the VF structure. > > Furthermore, a following change is going to refactor the ice driver to report > VSI > IDs using a relative index for each VF instead of reporting the PF VSI number. > This additional check would break that logic since it enforces that the VSI ID > matches the VSI number. > > Since this check duplicates the logic in ice_vc_isvalid_vsi_id() and gets in > the > way of refactoring that logic, remove it. > > Signed-off-by: Jacob Keller <[email protected]> > Reviewed-by: Przemek Kitszel <[email protected]> > --- > drivers/net/ethernet/intel/ice/ice_virtchnl_fdir.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_fdir.c > b/drivers/net/ethernet/intel/ice/ice_virtchnl_fdir.c > index f001553e1a1a..8e4ff3af86c6 100644 > --- a/drivers/net/ethernet/intel/ice/ice_virtchnl_fdir.c > +++ b/drivers/net/ethernet/intel/ice/ice_virtchnl_fdir.c > @@ -94,9 +94,6 @@ ice_vc_fdir_param_check(struct ice_vf *vf, u16 vsi_id)
Tested-by: Rafal Romanowski <[email protected]>
