On Wed, Jun 16, 2021 at 11:08:10AM +0800, edwardh wrote: > @@ -1400,18 +1404,13 @@ void bio_endio(struct bio *bio) > if (bio->bi_end_io == bio_chain_endio) { > bio = __bio_chain_endio(bio); > goto again; > + } else { > + blk_throtl_bio_endio(bio); > + /* release cgroup info */ > + bio_uninit(bio); > + if (bio->bi_end_io) > + bio->bi_end_io(bio); No need for an else after a goto.