[PATCH] scsi: target: tcmu: Adjust parameter in call to tcmu_blocks_release

* [PATCH] scsi: target: tcmu: Adjust parameter in call to tcmu_blocks_release
@ 2021-03-10 18:44 Bodo Stroesser
  2021-03-16  3:18 ` Martin K. Petersen
  2021-03-19  3:46 ` Martin K. Petersen
  0 siblings, 2 replies; 3+ messages in thread
From: Bodo Stroesser @ 2021-03-10 18:44 UTC (permalink / raw)
  To: linux-scsi, target-devel, Martin K. Petersen, Mike Christie
  Cc: Bodo Stroesser

In patch
 "scsi: target: tcmu: Replace radix_tree with XArray"
meaning of last parameter of tcmu_blocks_release was changed.
So in the callers we should subtract 1 from the previous
parameter.

Unfortunately that change got lost at one of the two places
where tcmu_blocks_release is called. That does not lead to any
problems, but we should adjust it anyway.

Signed-off-by: Bodo Stroesser <bostroesser@gmail.com>
---
 drivers/target/target_core_user.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/target/target_core_user.c b/drivers/target/target_core_user.c
index 9e1b115cb032..bdfc057f000c 100644
--- a/drivers/target/target_core_user.c
+++ b/drivers/target/target_core_user.c
@@ -1658,7 +1658,7 @@ static void tcmu_dev_kref_release(struct kref *kref)
 	xa_destroy(&udev->commands);
 	WARN_ON(!all_expired);
 
-	tcmu_blocks_release(&udev->data_blocks, 0, udev->dbi_max + 1);
+	tcmu_blocks_release(&udev->data_blocks, 0, udev->dbi_max);
 	bitmap_free(udev->data_bitmap);
 	mutex_unlock(&udev->cmdr_lock);
 
-- 
2.12.3


^ permalink raw reply related	[flat|nested] 3+ messages in thread