Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4726bcf3 authored by Keith Busch's avatar Keith Busch Committed by Christoph Hellwig
Browse files

nvme-pci: add missing unlock for reset error 



The reset work holds a mutex to prevent races with removal modifying the
same resources, but was unlocking only on success. Unlock on failure
too.

Fixes: 5c959d73 ("nvme-pci: fix rapid add remove sequence")
Signed-off-by: default avatarKeith Busch <keith.busch@intel.com>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent aef1897c

drivers/nvme/host/pci.c

+5 −3
Original line number Diff line number Diff line
@@ -2560,15 +2560,15 @@ static void nvme_reset_work(struct work_struct *work) 
	mutex_lock(&dev->shutdown_lock);

	result = nvme_pci_enable(dev);

	if (result)

		goto out;

		goto out_unlock;



	result = nvme_pci_configure_admin_queue(dev);

	if (result)

		goto out;

		goto out_unlock;



	result = nvme_alloc_admin_tags(dev);

	if (result)

		goto out;

		goto out_unlock;



	/*

	 * Limit the max command size to prevent iod->sg allocations going
@@ -2651,6 +2651,8 @@ static void nvme_reset_work(struct work_struct *work) 
	nvme_start_ctrl(&dev->ctrl);

	return;



 out_unlock:

	mutex_unlock(&dev->shutdown_lock);

 out:

	nvme_remove_dead_ctrl(dev, result);

}