diff options
author | 2020-06-27 09:31:55 +0200 | |
---|---|---|
committer | 2020-06-29 09:09:08 -0600 | |
commit | 81630e27fff3dc1ccbf64ecb48a70170d7071545 (patch) | |
tree | c1b579d89bee2c2250897649177b2c6492e1cf40 /include/linux/blk-cgroup.h | |
parent | block: move the initial blkg lookup into blkg_tryget_closest (diff) | |
download | linux-81630e27fff3dc1ccbf64ecb48a70170d7071545.tar.gz linux-81630e27fff3dc1ccbf64ecb48a70170d7071545.tar.bz2 linux-81630e27fff3dc1ccbf64ecb48a70170d7071545.zip |
blk-cgroup: remove the !bio->bi_blkg check in blkcg_bio_issue_check
This is purely a sanity check for grave programming errors. Remove it
to simplify further work in this area.
Acked-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include/linux/blk-cgroup.h')
-rw-r--r-- | include/linux/blk-cgroup.h | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/include/linux/blk-cgroup.h b/include/linux/blk-cgroup.h index 60df97202314..8e86b598316c 100644 --- a/include/linux/blk-cgroup.h +++ b/include/linux/blk-cgroup.h @@ -543,24 +543,11 @@ static inline void blkcg_bio_issue_init(struct bio *bio) static inline bool blkcg_bio_issue_check(struct request_queue *q, struct bio *bio) { - struct blkcg_gq *blkg; + struct blkcg_gq *blkg = bio->bi_blkg; bool throtl = false; rcu_read_lock(); - - if (!bio->bi_blkg) { - char b[BDEVNAME_SIZE]; - - WARN_ONCE(1, - "no blkg associated for bio on block-device: %s\n", - bio_devname(bio, b)); - bio_associate_blkg(bio); - } - - blkg = bio->bi_blkg; - throtl = blk_throtl_bio(q, blkg, bio); - if (!throtl) { struct blkg_iostat_set *bis; int rwd, cpu; |