On 6/6/24 4:58 PM, Christoph Hellwig wrote: >> +static int dm_set_zone_resource_limits(struct mapped_device *md, >> + struct dm_table *t, struct queue_limits *lim) > > Is there much of a point in splitting this out of > dm_set_zones_restrictions when almost nothing is left in > dm_set_zone_resource_limits after this changes? Indeed, we can merge the code and avoid the additional function. Will do that. > > Either way the logic looks good to me: > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- Damien Le Moal Western Digital Research