On 6/6/24 1:39 PM, Christoph Hellwig wrote: > On Thu, Jun 06, 2024 at 08:52:32AM +0900, Damien Le Moal wrote: >> Unless you have a better idea, I think we need to pass a queue_limits struct >> pointer to blk_revalidate_disk_zones() -> disk_update_zone_resources(). This >> pointer can be NULL, in which case disk_update_zone_resources() needs to do the >> limit start_update+commit. But if it is not NULL, then >> disk_update_zone_resources() must use the passed limits. > > I think the right thing is to simply move the call to dm_revalidate_zones > out of dm_set_zones_restrictions and after the queue_limits_set call > in dm_table_set_restrictions. Indeed, that is a simpler approach. Resending with that fixed. -- Damien Le Moal Western Digital Research
