On 5/1/24 07:50, Damien Le Moal wrote: > On 5/1/24 00:26, Christoph Hellwig wrote: >> Looks good: >> >> Reviewed-by: Christoph Hellwig <hch@xxxxxx> >> >> Although I suspect that at some point blk_revalidate_zone_cb should >> grow separate helpers for the conventional and sequential required cases >> as the amount of code in that switch statement is getting a bit out of >> bounds.. > > Yep, will clean that up later (next cycle). I did it now. It is a nice cleanup. I added a patch to the series for that. -- Damien Le Moal Western Digital Research