Commit 6b702768 authored Apr 25, 2021 by Edwin Peer Committed by David S. Miller Apr 25, 2021

bnxt_en: allow VF config ops when PF is closed



It is perfectly legal for the stack to query and configure VFs via PF
NDOs while the NIC is administratively down.  Remove the unnecessary
check for the PF to be in open state.

Signed-off-by: Edwin Peer <edwin.peer@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent dd85fc0a

drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -49,10 +49,6 @@ static int bnxt_hwrm_fwd_async_event_cmpl(struct bnxt *bp,

		static int bnxt_vf_ndo_prep(struct bnxt *bp, int vf_id)
		{
		if (!test_bit(BNXT_STATE_OPEN, &bp->state)) {
		netdev_err(bp->dev, "vf ndo called though PF is down\n");
		return -EINVAL;
		}
		if (!bp->pf.active_vfs) {
		netdev_err(bp->dev, "vf ndo called though sriov is disabled\n");
		return -EINVAL;