On 3/27/24 16:19, Hannes Reinecke wrote: > On 3/25/24 05:44, Damien Le Moal wrote: >> Allocating zone write plugs using a struct kmem_cache does not guarantee >> that enough write plugs can be allocated to simultaneously write up to >> the maximum number of active zones of a zoned block device. >> >> Avoid any issue with memory allocation by using a mempool with a size >> equal to the disk maximum number of open zones or maximum number of >> active zones, whichever is larger. For zoned devices that do not have >> open or active zone limits, the default 128 is used as the mempool size. >> If a change to the zone limits is detected, the mempool is resized in >> blk_revalidate_disk_zones(). >> >> Signed-off-by: Damien Le Moal <dlemoal@xxxxxxxxxx> >> --- >> block/blk-zoned.c | 62 ++++++++++++++++++++++++++++++------------ >> include/linux/blkdev.h | 3 ++ >> 2 files changed, 47 insertions(+), 18 deletions(-) >> > Why isn't this part of the previous patch? I tried to reduce the patch size... > But that shouldn't hold off the patchset, so: > > Reviewed-by: Hannes Reinecke <hare@xxxxxxx> Thanks. But I am about to send V3 and I removed the mempool and replaced it with a simple free list... > > Cheers, > > Hannes -- Damien Le Moal Western Digital Research
