[PATCHv3] nvme: fix namespace removal list

Christoph Hellwig hch at lst.de
Thu Jun 13 10:11:15 PDT 2024


On Thu, Jun 13, 2024 at 09:42:46AM -0700, Keith Busch wrote:
> It would be neat to have a more elegant helper that we've been messing
> with, and be able to use a single srcu synchronize, but in the interest
> of time, this patch will fix the regression.
> 
> I was hoping to not have per-ns srcu sync's, but this path pretty
> much only gets called on the last namespace anyway, so it should
> functionally work out to be a single sync without to optimizations.

Yeah, compared to the 2-3 synchronize_srcu calls in nvme_ns_remove
one more probably doesn't matter right now..

Looks good:

Reviewed-by: Christoph Hellwig <hch at lst.de>




More information about the Linux-nvme mailing list