On Fri, Jun 30, 2023 at 05:39:33PM +0900, Damien Le Moal wrote: > + if (queue_is_mq(q)) > + return blk_revalidate_disk_zones(nullb->disk, NULL); > + > return 0; I'd write this as: if (!queue_is_mq(q))) return 0; return blk_revalidate_disk_zones(nullb->disk, NULL); but that's just cosmetic. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>