On 6/10/24 4:56 PM, Damien Le Moal wrote:
> With the switch to using the zone append emulation of the block layer
> zone write plugging, the macro DM_ZONE_INVALID_WP_OFST is no longer used
> in dm-zone.c. Remove its definition.
>
> Fixes: f211268ed1f9 ("dm: Use the block layer zone append emulation")
> Signed-off-by: Damien Le Moal <dlemoal@xxxxxxxxxx>
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> Reviewed-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>
> Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
> Reviewed-by: Niklas Cassel <cassel@xxxxxxxxxx>
> ---
> drivers/md/dm-zone.c | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/md/dm-zone.c b/drivers/md/dm-zone.c
> index d9f8b7c0957a..70719bf32a2e 100644
> --- a/drivers/md/dm-zone.c
> +++ b/drivers/md/dm-zone.c
> @@ -13,8 +13,6 @@
>
> #define DM_MSG_PREFIX "zone"
>
> -#define DM_ZONE_INVALID_WP_OFST UINT_MAX
> -
> /*
> * For internal zone reports bypassing the top BIO submission path.
> */
> @@ -285,8 +283,6 @@ static int device_get_zone_resource_limits(struct dm_target *ti,
> return ret;
> }
>
> - zlim->mapped_nr_seq_zones += zc.target_nr_seq_zones;
> -
> /*
> * If the target does not map any sequential zones, then we do not need
> * any zone resource limits.
> @@ -317,6 +313,13 @@ static int device_get_zone_resource_limits(struct dm_target *ti,
> zlim->lim->max_open_zones =
> min_not_zero(max_open_zones, zlim->lim->max_open_zones);
>
> + /*
> + * Also count the total number of sequential zones for the mapped
> + * device so that when we are done inspecting all its targets, we are
> + * able to check if the mapped device actually has any sequential zones.
> + */
> + zlim->mapped_nr_seq_zones += zc.target_nr_seq_zones;
> +
Jens,
I messed up the rebase for v7: this hunk belongs to patch 3.
So I will resend a V8. My apologies for the churn.
> return 0;
> }
>
--
Damien Le Moal
Western Digital Research
