On Fri 23-12-22 03:16:35, Kemeng Shi wrote: > Use helper macro RQ_BFQQ to get bfqq of request. > > Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx> Yeah, why not. Feel free to add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > --- > block/bfq-iosched.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/block/bfq-iosched.c b/block/bfq-iosched.c > index 9441fc98961a..c3c4c83ee95f 100644 > --- a/block/bfq-iosched.c > +++ b/block/bfq-iosched.c > @@ -6683,14 +6683,14 @@ static struct bfq_queue *bfq_init_rq(struct request *rq) > return NULL; > > /* > - * Assuming that elv.priv[1] is set only if everything is set > + * Assuming that RQ_BFQQ(rq) is set only if everything is set > * for this rq. This holds true, because this function is > * invoked only for insertion or merging, and, after such > * events, a request cannot be manipulated any longer before > * being removed from bfq. > */ > - if (rq->elv.priv[1]) > - return rq->elv.priv[1]; > + if (RQ_BFQQ(rq)) > + return RQ_BFQQ(rq); > > bic = icq_to_bic(rq->elv.icq); > > -- > 2.30.0 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR
