nvme: check mutex init return code in ctrlr_construct
Change-Id: I0561f45f96ad6e88e7cdfec44c1f20677de2cb19 Signed-off-by: GangCao <gang.cao@intel.com>
This commit is contained in:
parent
b93d5a7ae5
commit
6515577822
@ -1286,6 +1286,7 @@ int
|
|||||||
nvme_ctrlr_construct(struct spdk_nvme_ctrlr *ctrlr)
|
nvme_ctrlr_construct(struct spdk_nvme_ctrlr *ctrlr)
|
||||||
{
|
{
|
||||||
struct spdk_pci_id pci_id;
|
struct spdk_pci_id pci_id;
|
||||||
|
int rc;
|
||||||
|
|
||||||
nvme_ctrlr_set_state(ctrlr, NVME_CTRLR_STATE_INIT, NVME_TIMEOUT_INFINITE);
|
nvme_ctrlr_set_state(ctrlr, NVME_CTRLR_STATE_INIT, NVME_TIMEOUT_INFINITE);
|
||||||
ctrlr->flags = 0;
|
ctrlr->flags = 0;
|
||||||
@ -1298,7 +1299,10 @@ nvme_ctrlr_construct(struct spdk_nvme_ctrlr *ctrlr)
|
|||||||
|
|
||||||
TAILQ_INIT(&ctrlr->active_io_qpairs);
|
TAILQ_INIT(&ctrlr->active_io_qpairs);
|
||||||
|
|
||||||
nvme_mutex_init_recursive_shared(&ctrlr->ctrlr_lock);
|
rc = nvme_mutex_init_recursive_shared(&ctrlr->ctrlr_lock);
|
||||||
|
if (rc != 0) {
|
||||||
|
return rc;
|
||||||
|
}
|
||||||
|
|
||||||
if (nvme_transport_ctrlr_get_pci_id(ctrlr, &pci_id) == 0) {
|
if (nvme_transport_ctrlr_get_pci_id(ctrlr, &pci_id) == 0) {
|
||||||
ctrlr->quirks = nvme_get_quirks(&pci_id);
|
ctrlr->quirks = nvme_get_quirks(&pci_id);
|
||||||
@ -1306,7 +1310,7 @@ nvme_ctrlr_construct(struct spdk_nvme_ctrlr *ctrlr)
|
|||||||
|
|
||||||
TAILQ_INIT(&ctrlr->active_procs);
|
TAILQ_INIT(&ctrlr->active_procs);
|
||||||
|
|
||||||
return 0;
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
void
|
void
|
||||||
|
Loading…
Reference in New Issue
Block a user