Skip to content

Commit

Permalink
net/iavf: release large VF when closing device
Browse files Browse the repository at this point in the history
[ upstream commit cdd8d6dfd156709b05ce363d7c5f491cc927330a ]

Multiple vfs created based on one PF. If one of the vf requests a large
number of queues, the queue resources are not released when exiting the
program, resulting in these queues cannot be available to other vfs.

This commit fixes the issue by resetting the number of queues to default
when closing the dev (refer to the kernel driver) .

Fixes: e436cd4 ("net/iavf: negotiate large VF and request more queues")

Signed-off-by: Shiyang He <shiyangx.he@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
  • Loading branch information
heshiyax authored and kevintraynor committed Jul 11, 2023
1 parent 9d5f649 commit 45daf22
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions drivers/net/iavf/iavf_ethdev.c
Original file line number Diff line number Diff line change
Expand Up @@ -2660,6 +2660,18 @@ iavf_dev_close(struct rte_eth_dev *dev)
if (vf->promisc_unicast_enabled || vf->promisc_multicast_enabled)
iavf_config_promisc(adapter, false, false);

/*
* Release redundant queue resource when close the dev
* so that other vfs can re-use the queues.
*/
if (vf->lv_enabled) {
ret = iavf_request_queues(dev, IAVF_MAX_NUM_QUEUES_DFLT);
if (ret)
PMD_DRV_LOG(ERR, "Reset the num of queues failed");

vf->max_rss_qregion = IAVF_MAX_NUM_QUEUES_DFLT;
}

iavf_shutdown_adminq(hw);
if (vf->vf_res->vf_cap_flags & VIRTCHNL_VF_OFFLOAD_WB_ON_ITR) {
/* disable uio intr before callback unregister */
Expand Down

0 comments on commit 45daf22

Please sign in to comment.