Re: [PATCH 4/4] block: rearrange a few request fields for better cache layout

[Jens Axboe]

On 1/10/18 11:43 AM, Omar Sandoval wrote:
>> -    INIT_LIST_HEAD(&rq->queuelist);
>>      /* csd/requeue_work/fifo_time is initialized before use */
>>      rq->q = data->q;
>>      rq->mq_ctx = data->ctx;
>> +    rq->rq_flags = 0;
>> +    rq->cpu = -1;
>>      rq->cmd_flags = op;
>>      if (data->flags & BLK_MQ_REQ_PREEMPT)
>>              rq->rq_flags |= RQF_PREEMPT;
>>      if (blk_queue_io_stat(data->q))
>>              rq->rq_flags |= RQF_IO_STAT;
>> -    rq->cpu = -1;
>> +    /* do not touch atomic flags, it needs atomic ops against the timer */
> 
> This comment was just removed in a previous patch but it snuck back in.

Eagle eyes - thanks, I will kill it.

-- 
Jens Axboe