On Tue, Aug 04, 2026 at 12:52:16PM +0200, Jan Kara wrote: > Mostly looks good. Just I think bio_blkcg() should gracefully handle the > case where bio->bi_blkg is NULL (and return NULL in that case). That way > you can also get rid of somewhat odd pattern: > > if (!bio_blkg(bio)) > return ...; > do something with bio_blkcg(bio) > > You can then just check bio_blkcg(bio) directly which is much more obvious.
Yes. Looking at the whole series I'm also not sure that this makes too much sense as a split out patch as the next one touches more than half of the callsite anyway.
