* [PATCH -next] blk-mq: Fix wrong wakeup batch configuration which will cause hang
@ 2022-01-27 10:00 Laibin Qiu
2022-01-27 14:17 ` Alex Xu (Hello71)
2022-01-27 17:17 ` Jens Axboe
0 siblings, 2 replies; 3+ messages in thread
From: Laibin Qiu @ 2022-01-27 10:00 UTC (permalink / raw)
To: axboe, andriy.shevchenko; +Cc: linux-block, linux-kernel, alex_y_xu
Commit 180dccb0dba4f ("blk-mq: fix tag_get wait task can't be
awakened") will recalculating wake_batch when inc or dec active_queues
to avoid wake_batch is > hctx_max_depth. At the same time, in order to
not affect performance as much as possible, the minimum wakeup batch is
set to 4. But when the QD is small (such as QD=1), if inc or dec
active_queues will increase wakeup batch, which will lead to hang.
Fix this problem with the following strategies:
QD : >= 32 | < 32
---------------------------------
wakeup batch: 8~4 | 3~1
Fixes: 180dccb0dba4f ("blk-mq: fix tag_get wait task can't be awakened")
Link: https://lore.kernel.org/linux-block/78cafe94-a787-e006-8851-69906f0c2128@huawei.com/T/#t
Reported-by: Alex Xu (Hello71) <alex_y_xu@yahoo.ca>
Signed-off-by: Laibin Qiu <qiulaibin@huawei.com>
---
lib/sbitmap.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/lib/sbitmap.c b/lib/sbitmap.c
index 6220fa67fb7e..09d293c30fd2 100644
--- a/lib/sbitmap.c
+++ b/lib/sbitmap.c
@@ -488,9 +488,13 @@ void sbitmap_queue_recalculate_wake_batch(struct sbitmap_queue *sbq,
unsigned int users)
{
unsigned int wake_batch;
+ unsigned int min_batch;
+ unsigned int depth = (sbq->sb.depth + users - 1) / users;
- wake_batch = clamp_val((sbq->sb.depth + users - 1) /
- users, 4, SBQ_WAKE_BATCH);
+ min_batch = sbq->sb.depth >= (4 * SBQ_WAIT_QUEUES) ? 4 : 1;
+
+ wake_batch = clamp_val(depth / SBQ_WAIT_QUEUES,
+ min_batch, SBQ_WAKE_BATCH);
__sbitmap_queue_update_wake_batch(sbq, wake_batch);
}
EXPORT_SYMBOL_GPL(sbitmap_queue_recalculate_wake_batch);
--
2.22.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH -next] blk-mq: Fix wrong wakeup batch configuration which will cause hang
2022-01-27 10:00 [PATCH -next] blk-mq: Fix wrong wakeup batch configuration which will cause hang Laibin Qiu
@ 2022-01-27 14:17 ` Alex Xu (Hello71)
2022-01-27 17:17 ` Jens Axboe
1 sibling, 0 replies; 3+ messages in thread
From: Alex Xu (Hello71) @ 2022-01-27 14:17 UTC (permalink / raw)
To: andriy.shevchenko, axboe, Laibin Qiu; +Cc: linux-block, linux-kernel
Excerpts from Laibin Qiu's message of January 27, 2022 5:00 am:
> Commit 180dccb0dba4f ("blk-mq: fix tag_get wait task can't be
> awakened") will recalculating wake_batch when inc or dec active_queues
> to avoid wake_batch is > hctx_max_depth. At the same time, in order to
> not affect performance as much as possible, the minimum wakeup batch is
> set to 4. But when the QD is small (such as QD=1), if inc or dec
> active_queues will increase wakeup batch, which will lead to hang.
>
> Fix this problem with the following strategies:
> QD : >= 32 | < 32
> ---------------------------------
> wakeup batch: 8~4 | 3~1
>
> Fixes: 180dccb0dba4f ("blk-mq: fix tag_get wait task can't be awakened")
> Link: https://lore.kernel.org/linux-block/78cafe94-a787-e006-8851-69906f0c2128@huawei.com/T/#t
> Reported-by: Alex Xu (Hello71) <alex_y_xu@yahoo.ca>
> Signed-off-by: Laibin Qiu <qiulaibin@huawei.com>
> ---
> lib/sbitmap.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/lib/sbitmap.c b/lib/sbitmap.c
> index 6220fa67fb7e..09d293c30fd2 100644
> --- a/lib/sbitmap.c
> +++ b/lib/sbitmap.c
> @@ -488,9 +488,13 @@ void sbitmap_queue_recalculate_wake_batch(struct sbitmap_queue *sbq,
> unsigned int users)
> {
> unsigned int wake_batch;
> + unsigned int min_batch;
> + unsigned int depth = (sbq->sb.depth + users - 1) / users;
>
> - wake_batch = clamp_val((sbq->sb.depth + users - 1) /
> - users, 4, SBQ_WAKE_BATCH);
> + min_batch = sbq->sb.depth >= (4 * SBQ_WAIT_QUEUES) ? 4 : 1;
> +
> + wake_batch = clamp_val(depth / SBQ_WAIT_QUEUES,
> + min_batch, SBQ_WAKE_BATCH);
> __sbitmap_queue_update_wake_batch(sbq, wake_batch);
> }
> EXPORT_SYMBOL_GPL(sbitmap_queue_recalculate_wake_batch);
> --
> 2.22.0
>
>
Tested-by: Alex Xu (Hello71) <alex_y_xu@yahoo.ca>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next] blk-mq: Fix wrong wakeup batch configuration which will cause hang
2022-01-27 10:00 [PATCH -next] blk-mq: Fix wrong wakeup batch configuration which will cause hang Laibin Qiu
2022-01-27 14:17 ` Alex Xu (Hello71)
@ 2022-01-27 17:17 ` Jens Axboe
1 sibling, 0 replies; 3+ messages in thread
From: Jens Axboe @ 2022-01-27 17:17 UTC (permalink / raw)
To: Laibin Qiu, andriy.shevchenko; +Cc: linux-kernel, linux-block, alex_y_xu
On Thu, 27 Jan 2022 18:00:47 +0800, Laibin Qiu wrote:
> Commit 180dccb0dba4f ("blk-mq: fix tag_get wait task can't be
> awakened") will recalculating wake_batch when inc or dec active_queues
> to avoid wake_batch is > hctx_max_depth. At the same time, in order to
> not affect performance as much as possible, the minimum wakeup batch is
> set to 4. But when the QD is small (such as QD=1), if inc or dec
> active_queues will increase wakeup batch, which will lead to hang.
>
> [...]
Applied, thanks!
[1/1] blk-mq: Fix wrong wakeup batch configuration which will cause hang
commit: 10825410b956dc1ed8c5fbc8bbedaffdadde7f20
Best regards,
--
Jens Axboe
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-27 17:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-27 10:00 [PATCH -next] blk-mq: Fix wrong wakeup batch configuration which will cause hang Laibin Qiu
2022-01-27 14:17 ` Alex Xu (Hello71)
2022-01-27 17:17 ` Jens Axboe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).