On Thu, 2018-12-20 at 14:00 -0700, Jens Axboe wrote:
> On 12/20/18 1:56 PM, Bart Van Assche wrote:
> > @@ -96,6 +97,9 @@ static void blk_mq_check_inflight(struct blk_mq_hw_ctx
> > *hctx,
> > {
> > struct mq_inflight *mi = priv;
> >
> > + if (rq->q != mi->q)
> > + return;
>
> Aren't you back to square one with this one, if the tags are shared? You
> can't dereference it before you know it matches.
My patch can only work if the new rq->q = NULL assignment in
__blk_mq_free_request()
is executed before the request tag is freed and if freeing a tag does not happen
concurrently with any bt_iter() call. Would you accept that I add a seqlock to
avoid
this scenario?
Thanks,
Bart.
