UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

ice: convert ice_reset_vf to standard error codes 

The ice_reset_vf function returns a boolean value indicating whether or
not the VF reset. This is a bit confusing since it means that callers
need to know how to interpret the return value when needing to indicate
an error.

Refactor the function and call sites to report a regular error code. We
still report success (i.e. return 0) in cases where the reset is in
progress or is disabled.

Existing callers don't care because they do not check the return value.
We keep the error code anyways instead of a void return because we
expect future code which may care about or at least report the error
value.

Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
This commit is contained in:
Jacob Keller 2022-02-22 16:27:06 -08:00 committed by Tony Nguyen
parent fe99d1c06c
commit 4fe193cc9d
2 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
drivers/net/ethernet/intel/ice/ice_vf_lib.c
View File

@ -446,10 +446,11 @@ void ice_reset_all_vfs(struct ice_pf *pf)
* @vf: pointer to the VF structure * @vf: pointer to the VF structure
* @is_vflr: true if VFLR was issued, false if not * @is_vflr: true if VFLR was issued, false if not
* *
* Returns true if the VF is currently in reset, resets successfully, or resets * Returns 0 if the VF is currently in reset, if the resets are disabled, or
* are disabled and false otherwise. * if the VF resets successfully. Returns an error code if the VF fails to
* rebuild.
*/ */
bool ice_reset_vf(struct ice_vf *vf, bool is_vflr) int ice_reset_vf(struct ice_vf *vf, bool is_vflr)
{ {
struct ice_pf *pf = vf->pf; struct ice_pf *pf = vf->pf;
struct ice_vsi *vsi; struct ice_vsi *vsi;
@ -466,13 +467,13 @@ bool ice_reset_vf(struct ice_vf *vf, bool is_vflr)
if (test_bit(ICE_VF_RESETS_DISABLED, pf->state)) { if (test_bit(ICE_VF_RESETS_DISABLED, pf->state)) {
dev_dbg(dev, "Trying to reset VF %d, but all VF resets are disabled\n", dev_dbg(dev, "Trying to reset VF %d, but all VF resets are disabled\n",
vf->vf_id); vf->vf_id);
return true; return 0;
} }
if (ice_is_vf_disabled(vf)) { if (ice_is_vf_disabled(vf)) {
dev_dbg(dev, "VF is already disabled, there is no need for resetting it, telling VM, all is fine %d\n", dev_dbg(dev, "VF is already disabled, there is no need for resetting it, telling VM, all is fine %d\n",
vf->vf_id); vf->vf_id);
return true; return 0;
} }
/* Set VF disable bit state here, before triggering reset */ /* Set VF disable bit state here, before triggering reset */
@ -532,7 +533,7 @@ bool ice_reset_vf(struct ice_vf *vf, bool is_vflr)
if (vf->vf_ops->vsi_rebuild(vf)) { if (vf->vf_ops->vsi_rebuild(vf)) {
dev_err(dev, "Failed to release and setup the VF%u's VSI\n", dev_err(dev, "Failed to release and setup the VF%u's VSI\n",
vf->vf_id); vf->vf_id);
return false; return -EFAULT;
} }
vf->vf_ops->post_vsi_rebuild(vf); vf->vf_ops->post_vsi_rebuild(vf);
@ -546,7 +547,7 @@ bool ice_reset_vf(struct ice_vf *vf, bool is_vflr)
dev_dbg(dev, "failed to clear malicious VF state for VF %u\n", dev_dbg(dev, "failed to clear malicious VF state for VF %u\n",
vf->vf_id); vf->vf_id);
return true; return 0;
} }
/** /**

6
drivers/net/ethernet/intel/ice/ice_vf_lib.h
View File

@ -212,7 +212,7 @@ int
ice_vf_set_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m); ice_vf_set_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m);
int int
ice_vf_clear_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m); ice_vf_clear_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m);
bool ice_reset_vf(struct ice_vf *vf, bool is_vflr); int ice_reset_vf(struct ice_vf *vf, bool is_vflr);
void ice_reset_all_vfs(struct ice_pf *pf); void ice_reset_all_vfs(struct ice_pf *pf);
#else /* CONFIG_PCI_IOV */ #else /* CONFIG_PCI_IOV */
static inline struct ice_vf *ice_get_vf_by_id(struct ice_pf *pf, u16 vf_id) static inline struct ice_vf *ice_get_vf_by_id(struct ice_pf *pf, u16 vf_id)
@ -270,9 +270,9 @@ ice_vf_clear_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m)
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
static inline bool ice_reset_vf(struct ice_vf *vf, bool is_vflr) static inline int ice_reset_vf(struct ice_vf *vf, bool is_vflr)
{ {
return true; return 0;
} }
static inline void ice_reset_all_vfs(struct ice_pf *pf) static inline void ice_reset_all_vfs(struct ice_pf *pf)