Commit 4705de73 authored by Dennis Zhou's avatar Dennis Zhou Committed by Jens Axboe

blkcg: put back rcu lock in blkcg_bio_issue_check()

I was a little overzealous in removing the rcu_read_lock() call from
blkcg_bio_issue_check() and it broke blk-throttle. Put it back.

Fixes: e35403a034bf ("blkcg: associate blkg when associating a device")
Signed-off-by: default avatarDennis Zhou <dennis@kernel.org>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent d3fcdff1
...@@ -764,6 +764,8 @@ static inline bool blkcg_bio_issue_check(struct request_queue *q, ...@@ -764,6 +764,8 @@ static inline bool blkcg_bio_issue_check(struct request_queue *q,
struct blkcg_gq *blkg; struct blkcg_gq *blkg;
bool throtl = false; bool throtl = false;
rcu_read_lock();
if (!bio->bi_blkg) { if (!bio->bi_blkg) {
char b[BDEVNAME_SIZE]; char b[BDEVNAME_SIZE];
...@@ -791,6 +793,7 @@ static inline bool blkcg_bio_issue_check(struct request_queue *q, ...@@ -791,6 +793,7 @@ static inline bool blkcg_bio_issue_check(struct request_queue *q,
blkcg_bio_issue_init(bio); blkcg_bio_issue_init(bio);
rcu_read_unlock();
return !throtl; return !throtl;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment