Skip to content

nvme_ns_remove() deadlock fix

Keith Busch linux-nvme 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 0 repliesNew
  2. v11 patch 0 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2nvme: switch to RCU freeing the namespace+0 -00 replies
1/2nvme: switch to RCU freeing the namespacedrivers/nvme/host/core.c drivers/nvme/host/pci.c+10 -20 replies
1/2nvme: switch to RCU freeing the namespace+0 -01 reply
1/2nvme: switch to RCU freeing the namespace+0 -01 reply
1/2nvme: switch to RCU freeing the namespace+0 -01 reply
1/2nvme: switch to RCU freeing the namespace+0 -00 replies
1/2nvme: switch to RCU freeing the namespacedrivers/nvme/host/core.c+9 -22 replies
1/2nvme: switch to RCU freeing the namespace+0 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 0/2] nvme_ns_remove() deadlock fix
Quoted 12 lines · Ming Lin

From: Ming Lin <ming.l at ssi.samsung.com>

Hi Sunad,

I don't have device that can do namespace attribute change. Could you help to verify these 2 patches fix the ns remove deadlock?

Thanks.

v2:
- switch to RCU freeing the namespace

Quoted 6 lines

Ming Lin (2):
nvme: switch to RCU freeing the namespace
nvme: fix nvme_ns_remove() deadlock

 drivers/nvme/host/core.c | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

This looks good, and successfully tested on a namespace management capable controller. These tests will be run more often from now on. Thanks for the patches.

Reviewed-by Keith Busch <keith.busch at intel.com>

Read the cover letter in its thread