Re: [PATCH 3/3] blk-mq: Use llist_head for blk_cpu_done

From: Christoph Hellwig
Date: Mon Jan 25 2021 - 04:07:11 EST


> +static void blk_mq_complete_send_ipi(struct request *rq)
> +{
> + struct llist_head *list;
> + unsigned int cpu;
> +
> + cpu = rq->mq_ctx->cpu;
> + list = &per_cpu(blk_cpu_done, cpu);
> + if (llist_add(&rq->ipi_list, list)) {
> + INIT_CSD(&rq->csd, __blk_mq_complete_request_remote, rq);
> + smp_call_function_single_async(cpu, &rq->csd);
> + }
> +}

Nit: it would be nice to initialize cpu and list in the declaration
lines.

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>