Commit 826cb7b4 authored by Ariel Elior's avatar Ariel Elior Committed by David S. Miller

bnx2x: Disable VF access on PF removal

When the bnx2x driver is rmmoded, if VFs of a given PF will be assigned
to a VM then that PF will be unable to call `pci_disable_sriov()'.

If for that same PF there would also exist unassigned VFs in the hypervisor,
the result will be that after the removal there will still be virtual PCI
functions on the hypervisor.
If the bnx2x module were to be re-inserted, the result will be that the VFs
on the hypervisor will be re-probed directly following the PF's probe, even
though that in regular loading flow sriov is only enabled once PF is loaded.
The probed VF will then try to access its bar, causing a PCI error as the HW
is not in a state enabling such a request.

This patch adds a missing disablement procedure to the PF's removal, one that
sets registers viewable to the VF to indicate that the VFs have no permission
to access the bar, thus resulting in probe errors instead of PCI errors.
Signed-off-by: default avatarAriel Elior <ariele@broadcom.com>
Signed-off-by: default avatarYuval Mintz <yuvalmin@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent e3ed4eae
...@@ -2018,6 +2018,8 @@ int bnx2x_iov_init_one(struct bnx2x *bp, int int_mode_param, ...@@ -2018,6 +2018,8 @@ int bnx2x_iov_init_one(struct bnx2x *bp, int int_mode_param,
void bnx2x_iov_remove_one(struct bnx2x *bp) void bnx2x_iov_remove_one(struct bnx2x *bp)
{ {
int vf_idx;
/* if SRIOV is not enabled there's nothing to do */ /* if SRIOV is not enabled there's nothing to do */
if (!IS_SRIOV(bp)) if (!IS_SRIOV(bp))
return; return;
...@@ -2026,6 +2028,18 @@ void bnx2x_iov_remove_one(struct bnx2x *bp) ...@@ -2026,6 +2028,18 @@ void bnx2x_iov_remove_one(struct bnx2x *bp)
pci_disable_sriov(bp->pdev); pci_disable_sriov(bp->pdev);
DP(BNX2X_MSG_IOV, "sriov disabled\n"); DP(BNX2X_MSG_IOV, "sriov disabled\n");
/* disable access to all VFs */
for (vf_idx = 0; vf_idx < bp->vfdb->sriov.total; vf_idx++) {
bnx2x_pretend_func(bp,
HW_VF_HANDLE(bp,
bp->vfdb->sriov.first_vf_in_pf +
vf_idx));
DP(BNX2X_MSG_IOV, "disabling internal access for vf %d\n",
bp->vfdb->sriov.first_vf_in_pf + vf_idx);
bnx2x_vf_enable_internal(bp, 0);
bnx2x_pretend_func(bp, BP_ABS_FUNC(bp));
}
/* free vf database */ /* free vf database */
__bnx2x_iov_free_vfdb(bp); __bnx2x_iov_free_vfdb(bp);
} }
...@@ -3197,7 +3211,7 @@ int bnx2x_enable_sriov(struct bnx2x *bp) ...@@ -3197,7 +3211,7 @@ int bnx2x_enable_sriov(struct bnx2x *bp)
* the "acquire" messages to appear on the VF PF channel. * the "acquire" messages to appear on the VF PF channel.
*/ */
DP(BNX2X_MSG_IOV, "about to call enable sriov\n"); DP(BNX2X_MSG_IOV, "about to call enable sriov\n");
pci_disable_sriov(bp->pdev); bnx2x_disable_sriov(bp);
rc = pci_enable_sriov(bp->pdev, req_vfs); rc = pci_enable_sriov(bp->pdev, req_vfs);
if (rc) { if (rc) {
BNX2X_ERR("pci_enable_sriov failed with %d\n", rc); BNX2X_ERR("pci_enable_sriov failed with %d\n", rc);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment