diff mbox series

null_blk: Print correct max open zones limit in null_init_zoned_dev()

Message ID 20240528062852.437599-1-dlemoal@kernel.org (mailing list archive)
State New, archived
Headers show
Series null_blk: Print correct max open zones limit in null_init_zoned_dev() | expand

Commit Message

Damien Le Moal May 28, 2024, 6:28 a.m. UTC
When changing the maximum number of open zones, print that number
instead of the total number of zones.

Fixes: dc4d137ee3b7 ("null_blk: add support for max open/active zone limit for zoned devices")
Cc: stable@vger.kernel.org
Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
---
 drivers/block/null_blk/zoned.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Niklas Cassel May 28, 2024, 6:54 a.m. UTC | #1
On Tue, May 28, 2024 at 03:28:52PM +0900, Damien Le Moal wrote:
> When changing the maximum number of open zones, print that number
> instead of the total number of zones.
> 
> Fixes: dc4d137ee3b7 ("null_blk: add support for max open/active zone limit for zoned devices")
> Cc: stable@vger.kernel.org
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
> ---
>  drivers/block/null_blk/zoned.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/block/null_blk/zoned.c b/drivers/block/null_blk/zoned.c
> index 5b5a63adacc1..79c8e5e99f7f 100644
> --- a/drivers/block/null_blk/zoned.c
> +++ b/drivers/block/null_blk/zoned.c
> @@ -108,7 +108,7 @@ int null_init_zoned_dev(struct nullb_device *dev,
>  	if (dev->zone_max_active && dev->zone_max_open > dev->zone_max_active) {
>  		dev->zone_max_open = dev->zone_max_active;
>  		pr_info("changed the maximum number of open zones to %u\n",
> -			dev->nr_zones);
> +			dev->zone_max_open);
>  	} else if (dev->zone_max_open >= dev->nr_zones - dev->zone_nr_conv) {
>  		dev->zone_max_open = 0;
>  		pr_info("zone_max_open limit disabled, limit >= zone count\n");
> -- 
> 2.45.1
> 

Reviewed-by: Niklas Cassel <cassel@kernel.org>
Jens Axboe May 28, 2024, 12:57 p.m. UTC | #2
On Tue, 28 May 2024 15:28:52 +0900, Damien Le Moal wrote:
> When changing the maximum number of open zones, print that number
> instead of the total number of zones.
> 
> 

Applied, thanks!

[1/1] null_blk: Print correct max open zones limit in null_init_zoned_dev()
      commit: 233e27b4d21c3e44eb863f03e566d3a22e81a7ae

Best regards,
diff mbox series

Patch

diff --git a/drivers/block/null_blk/zoned.c b/drivers/block/null_blk/zoned.c
index 5b5a63adacc1..79c8e5e99f7f 100644
--- a/drivers/block/null_blk/zoned.c
+++ b/drivers/block/null_blk/zoned.c
@@ -108,7 +108,7 @@  int null_init_zoned_dev(struct nullb_device *dev,
 	if (dev->zone_max_active && dev->zone_max_open > dev->zone_max_active) {
 		dev->zone_max_open = dev->zone_max_active;
 		pr_info("changed the maximum number of open zones to %u\n",
-			dev->nr_zones);
+			dev->zone_max_open);
 	} else if (dev->zone_max_open >= dev->nr_zones - dev->zone_nr_conv) {
 		dev->zone_max_open = 0;
 		pr_info("zone_max_open limit disabled, limit >= zone count\n");