[v4,0/3] Fix DM zone resource limits stacking

Message

Damien Le Moal June 5, 2024, 7:51 a.m. UTC

This is the updated patch 4/4 of the series "Zone write plugging and DM
zone fixes". This patch fixes DM zone resource limits stacking (max open
zones and max active zones limits). Patch 1 is new and is added to help
catch problems and eventual regressions of the handling of these limits.

Changes from v3:
 - Modify patch 1 to always check the zone resource limits values in
   disk_update_zone_resources(), including for DM devices that do not
   use zone write plugging. Simplify patch 2 accordingly by removing the
   same check and adjustment of the zone resource limits.
 - Added patch 3

Changes from v2:
 - Modify patch 1 to return an error for the case where the max open
   zones limit is greater than the max active zones limit.
 - Modify patch 2 to avoid duplicated actions on the limits and to
   remove warnings for unusual zone limits.

Changes from v1:
 - Added patch 1
 - Modified patch 2 to not cap the limits for a target with the number
   of sequential zones mapped but rather to use the device limits as is
   when more zones than the limits are mapped and 0 otherwise (no
   limits).

Damien Le Moal (3):
  block: Improve checks on zone resource limits
  dm: Improve zone resource limits handling
  dm: Remove unused macro DM_ZONE_INVALID_WP_OFST

 block/blk-settings.c |   8 ++
 block/blk-zoned.c    |  20 ++++-
 drivers/md/dm-zone.c | 206 +++++++++++++++++++++++++++++++++++--------
 3 files changed, 191 insertions(+), 43 deletions(-)

Comments

Johannes Thumshirn June 5, 2024, 12:40 p.m. UTC | #1

For the series,
Reviewed-by: Johannes Thumshirn <[e-mail]>