[PATCH v2] nvme: split nvme_update_zone_info

Kanchan Joshi joshi.k at samsung.com
Tue Apr 2 08:13:48 PDT 2024


On 4/2/2024 8:17 PM, Christoph Hellwig wrote:
> nvme_update_zone_info does (admin queue) I/O to the device and can fail.
> We fail to abort the queue limits update if that happen, but really
> should avoid with the frozen I/O queue as much as possible anyway.
> 
> Split the logic into a helper to query the information that can be
> called on an unfrozen queue and one to apply it to the queue limits.
> 
> Fixes: 9b130d681443 ("nvme: use the atomic queue limits update API")
> Reported-by: Kanchan Joshi<joshi.k at samsung.com>
> Signed-off-by: Christoph Hellwig<hch at lst.de>
> ---

Looks good.
Reviewed-by: Kanchan Joshi <joshi.k at samsung.com>



More information about the Linux-nvme mailing list