Patch "blk-mq: fix "bad unlock balance detected" on q->srcu in __blk_mq_run_dispatch_ops" has been added to the 6.1-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    blk-mq: fix "bad unlock balance detected" on q->srcu in __blk_mq_run_dispatch_ops

to the 6.1-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     blk-mq-fix-bad-unlock-balance-detected-on-q-srcu-in-.patch
and it can be found in the queue-6.1 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 55cb484ac38dc54e421beb925ae5f7429300b0a6
Author: Chris Leech <cleech@xxxxxxxxxx>
Date:   Fri Mar 10 09:09:13 2023 +0800

    blk-mq: fix "bad unlock balance detected" on q->srcu in __blk_mq_run_dispatch_ops
    
    [ Upstream commit 00e885efcfbb8712d3e1bfc1ae30639c15ca1d3b ]
    
    The 'q' parameter of the macro __blk_mq_run_dispatch_ops may not be one
    local variable, such as, it is rq->q, then request queue pointed by
    this variable could be changed to another queue in case of
    BLK_MQ_F_TAG_QUEUE_SHARED after 'dispatch_ops' returns, then
    'bad unlock balance' is triggered.
    
    Fixes the issue by adding one local variable for doing srcu lock/unlock.
    
    Fixes: 2a904d00855f ("blk-mq: remove hctx_lock and hctx_unlock")
    Cc: Marco Patalano <mpatalan@xxxxxxxxxx>
    Signed-off-by: Chris Leech <cleech@xxxxxxxxxx>
    Signed-off-by: Ming Lei <ming.lei@xxxxxxxxxx>
    Link: https://lore.kernel.org/r/20230310010913.1014789-1-ming.lei@xxxxxxxxxx
    Signed-off-by: Jens Axboe <axboe@xxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/block/blk-mq.h b/block/blk-mq.h
index ef59fee62780d..a7482d2cc82e7 100644
--- a/block/blk-mq.h
+++ b/block/blk-mq.h
@@ -378,12 +378,13 @@ static inline bool hctx_may_queue(struct blk_mq_hw_ctx *hctx,
 #define __blk_mq_run_dispatch_ops(q, check_sleep, dispatch_ops)	\
 do {								\
 	if ((q)->tag_set->flags & BLK_MQ_F_BLOCKING) {		\
+		struct blk_mq_tag_set *__tag_set = (q)->tag_set; \
 		int srcu_idx;					\
 								\
 		might_sleep_if(check_sleep);			\
-		srcu_idx = srcu_read_lock((q)->tag_set->srcu);	\
+		srcu_idx = srcu_read_lock(__tag_set->srcu);	\
 		(dispatch_ops);					\
-		srcu_read_unlock((q)->tag_set->srcu, srcu_idx);	\
+		srcu_read_unlock(__tag_set->srcu, srcu_idx);	\
 	} else {						\
 		rcu_read_lock();				\
 		(dispatch_ops);					\



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux