Commit f1d2337d authored by Justin Tee's avatar Justin Tee Committed by Martin K. Petersen
Browse files

scsi: lpfc: Reinitialize internal VMID data structures after FLOGI completion 



After enabling VMID, an issue LIP test was erasing fabric switch VMID
information.

Introduce a lpfc_reinit_vmid() routine, which reinitializes all VMID data
structures upon FLOGI completion in fabric topology.

Signed-off-by: default avatarJustin Tee <justin.tee@broadcom.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 21681b81

drivers/scsi/lpfc/lpfc_crtn.h

+1 −0
Original line number Diff line number Diff line
@@ -683,6 +683,7 @@ int lpfc_vmid_get_appid(struct lpfc_vport *vport, char *uuid, 
			union lpfc_vmid_io_tag *tag);

void lpfc_vmid_vport_cleanup(struct lpfc_vport *vport);

int lpfc_issue_els_qfpa(struct lpfc_vport *vport);

void lpfc_reinit_vmid(struct lpfc_vport *vport);



void lpfc_sli_rpi_release(struct lpfc_vport *vport,

			  struct lpfc_nodelist *ndlp);

drivers/scsi/lpfc/lpfc_els.c

+3 −0
Original line number Diff line number Diff line
@@ -1123,6 +1123,9 @@ lpfc_cmpl_els_flogi(struct lpfc_hba *phba, struct lpfc_iocbq *cmdiocb, 
	if (sp->cmn.priority_tagging)

		vport->phba->pport->vmid_flag |= (LPFC_VMID_ISSUE_QFPA |

						  LPFC_VMID_TYPE_PRIO);

	/* reinitialize the VMID datastructure before returning */

	if (lpfc_is_vmid_enabled(phba))

		lpfc_reinit_vmid(vport);



	/*

	 * Address a timing race with dev_loss.  If dev_loss is active on

drivers/scsi/lpfc/lpfc_vmid.c

+39 −0
Original line number Diff line number Diff line
@@ -284,3 +284,42 @@ int lpfc_vmid_get_appid(struct lpfc_vport *vport, char *uuid, 
	}

	return rc;

}



/*

 * lpfc_reinit_vmid - reinitializes the vmid data structure

 * @vport: pointer to vport data structure

 *

 * This routine reinitializes the vmid post flogi completion

 *

 * Return codes

 *	None

 */

void

lpfc_reinit_vmid(struct lpfc_vport *vport)

{

	u32 bucket, i, cpu;

	struct lpfc_vmid *cur;

	struct lpfc_vmid *vmp = NULL;

	struct hlist_node *tmp;



	write_lock(&vport->vmid_lock);

	vport->cur_vmid_cnt = 0;



	for (i = 0; i < vport->max_vmid; i++) {

		vmp = &vport->vmid[i];

		vmp->flag = LPFC_VMID_SLOT_FREE;

		memset(vmp->host_vmid, 0, sizeof(vmp->host_vmid));

		vmp->io_rd_cnt = 0;

		vmp->io_wr_cnt = 0;



		if (vmp->last_io_time)

			for_each_possible_cpu(cpu)

				*per_cpu_ptr(vmp->last_io_time, cpu) = 0;

	}



	/* for all elements in the hash table */

	if (!hash_empty(vport->hash_table))

		hash_for_each_safe(vport->hash_table, bucket, tmp, cur, hnode)

			hash_del(&cur->hnode);

	write_unlock(&vport->vmid_lock);

}