On Thu, Nov 4, 2021 at 3:03 AM Jens Axboe <axboe@xxxxxxxxx> wrote: > > On 11/2/21 10:00 PM, Yi Zhang wrote: > >>> > >>> Hello Jens, > >>> > >>> I guess the issue could be the following code run without grabbing > >>> ->q_usage_counter from blk_mq_alloc_request() and blk_mq_alloc_request_hctx(). > >>> > >>> .rq_flags = q->elevator ? RQF_ELV : 0, > >>> > >>> then elevator is switched to real one from none, and check on q->elevator > >>> becomes not consistent. > >> > >> Indeed, that’s where I was going with this. I have a patch, testing it locally but it’s getting late. Will send it out tomorrow. The nice benefit is that it allows dropping the weird ref get on plug flush, and batches getting the refs as well. > >> > > > > Hi Jens > > Here is the log in case you still need it. :) > > Can you retry with the updated for-next pulled into -git? Hi Jens Sorry for the delay, the issue cannot be reproduced now. > > -- > Jens Axboe > -- Best Regards, Yi Zhang
