From: Aleksandr Loktionov <aleksandr.loktionov@intel.com>

Date: Sat, 23 Jan 2021 00:22:23 +0000

Subject: i40e: Revert "i40e: don't report link up for a VF who hasn't enabled

 queues"

Git-commit: f559a356043a55bab25a4c00505ea65c50a956fb

Patch-mainline: v5.11-rc7

References: jsc#SLE-8025

This reverts commit 2ad1274fa35ace5c6360762ba48d33b63da2396c

VF queues were not brought up when PF was brought up after being

downed if the VF driver disabled VFs queues during PF down.

This could happen in some older or external VF driver implementations.

The problem was that PF driver used vf->queues_enabled as a condition

to decide what link-state it would send out which caused the issue.

Remove the check for vf->queues_enabled in the VF link notify.

Now VF will always be notified of the current link status.

Also remove the queues_enabled member from i40e_vf structure as it is

not used anymore. Otherwise VNF implementation was broken and caused

a link flap.

The original commit was a workaround to avoid breaking existing VFs though

it's really a fault of the VF code not the PF. The commit should be safe to

revert as all of the VFs we know of have been fixed. Also, since we now

know there is a related bug in the workaround, removing it is preferred.

Fixes: 2ad1274fa35a ("i40e: don't report link up for a VF who hasn't enabled")

Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>

Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>

Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>

Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>

Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>

---

 drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c |   13 +------------

 drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h |    1 -

 2 files changed, 1 insertion(+), 13 deletions(-)

--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c

+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c

@@ -55,12 +55,7 @@ static void i40e_vc_notify_vf_link_state

 	pfe.event = VIRTCHNL_EVENT_LINK_CHANGE;

 	pfe.severity = PF_EVENT_SEVERITY_INFO;

-

-	/* Always report link is down if the VF queues aren't enabled */

-	if (!vf->queues_enabled) {

-		pfe.event_data.link_event.link_status = false;

-		pfe.event_data.link_event.link_speed = 0;

-	} else if (vf->link_forced) {

+	if (vf->link_forced) {

 		pfe.event_data.link_event.link_status = vf->link_up;

 		pfe.event_data.link_event.link_speed =

 			(vf->link_up ? VIRTCHNL_LINK_SPEED_40GB : 0);

@@ -70,7 +65,6 @@ static void i40e_vc_notify_vf_link_state

 		pfe.event_data.link_event.link_speed =

 			i40e_virtchnl_link_speed(ls->link_speed);

 	}

-

 	i40e_aq_send_msg_to_vf(hw, abs_vf_id, VIRTCHNL_OP_EVENT,

 			       0, (u8 *)&pfe, sizeof(pfe), NULL);

 }

@@ -2388,8 +2382,6 @@ static int i40e_vc_enable_queues_msg(str

 		}

 	}

-	vf->queues_enabled = true;

-

 error_param:

 	/* send the response to the VF */

 	return i40e_vc_send_resp_to_vf(vf, VIRTCHNL_OP_ENABLE_QUEUES,

@@ -2411,9 +2403,6 @@ static int i40e_vc_disable_queues_msg(st

 	struct i40e_pf *pf = vf->pf;

 	i40e_status aq_ret = 0;

-	/* Immediately mark queues as disabled */

-	vf->queues_enabled = false;

-

 	if (!test_bit(I40E_VF_STATE_ACTIVE, &vf->vf_states)) {

 		aq_ret = I40E_ERR_PARAM;

 		goto error_param;

--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h

+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h

@@ -99,7 +99,6 @@ struct i40e_vf {

 	unsigned int tx_rate;	/* Tx bandwidth limit in Mbps */

 	bool link_forced;

 	bool link_up;		/* only valid if VF link is forced */

-	bool queues_enabled;	/* true if the VF queues are enabled */

 	bool spoofchk;

 	u16 num_vlan;