Re: [dm-devel] [PATCH 02/12] dm-zoned: convert to xarray

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > The zones array is getting really large, and large arrays > tend to wreak havoc with the caches. s/caches/CPU cache, may be ? > So convert it to xarray to become more cache friendly. > > Signed-off-by: Hannes Reinecke > --- >

Re: [dm-devel] [PATCH 05/12] dm-zoned: add device pointer to struct dm_zone

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Add a pointer to the containing device to struct dm_zone and > kill dmz_zone_to_dev(). > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c | 47 > -- > drivers/md/dm-zoned-reclaim.c | 18

Re: [dm-devel] [PATCH 06/12] dm-zoned: add metadata pointer to struct dmz_dev

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Add a metadata pointer to struct dmz_dev and use it as argument > for blkdev_report_zones() instead of the metadata itself. > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c | 14 +- > drivers/md/dm-zoned.h

Re: [dm-devel] [PATCH 07/12] dm-zoned: add a 'reserved' zone flag

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Instead of counting the number of reserved zones in dmz_free_zone() > we should mark the zone as 'reserved' during allocation and simplify > dmz_free_zone(). > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c | 4 ++-- >

Re: [dm-devel] [PATCH 10/12] dm-zoned: support arbitrary number of devices

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Remove the hard-coded limit of two devices and support an unlimited > number of additional zoned devices. > With that we need to increase the device-mapper version number to > 3.0.0 as we've modified the interface. > > Signed-off-by: Hannes Reinecke >

Re: [dm-devel] [PATCH 04/12] dm-zoned: secondary superblock must reside on the same devices than primary superblock

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > The secondary superblock must reside on the same device than the > primary superblock, so there's no need to re-calculate the device. > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c | 5 ++--- > 1 file changed, 2

Re: [dm-devel] [PATCH 03/12] dm-zoned: use on-stack superblock for tertiary devices

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Checking the teriary superblock just consists of validating UUIDs, s/teriary/tertiary > crcs, and the generation number; it doesn't have contents which > would be required during the actual operation. > So we should use an on-stack superblock and

Re: [dm-devel] [PATCH 08/12] dm-zoned: move random and sequential zones into struct dmz_dev

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Random and sequential zones should be part of the respective > device structure to make arbitration between devices possible. > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c | 143 > +

Re: [dm-devel] [PATCH 09/12] dm-zoned: improve logging messages for reclaim

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Instead of just reporting the errno this patch adds some more > verbose debugging message in the reclaim path. > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-reclaim.c | 13 ++--- > 1 file changed, 10 insertions(+), 3

Re: [dm-devel] [PATCH 11/12] dm-zoned: round-robin load balancer for reclaiming zones

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > When reclaiming zones we should arbitrate between the zoned > devices to get a better throughput. So implement a simple > round-robin load balancer between the zoned devices. > > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c |

Re: [dm-devel] [PATCH 01/12] dm-zoned: add debugging message for reading superblocks

2020-05-24 Thread Damien Le Moal
On 2020/05/23 0:39, Hannes Reinecke wrote: > Signed-off-by: Hannes Reinecke > --- > drivers/md/dm-zoned-metadata.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/md/dm-zoned-metadata.c b/drivers/md/dm-zoned-metadata.c > index 4a2e351365c5..b0d3ed4ac56a 100644 > ---