On 08/04/2017 01:43 PM, Bart Van Assche wrote:
> On Fri, 2017-08-04 at 09:04 -0600, Jens Axboe wrote:
>> +static void blk_mq_check_inflight(struct blk_mq_hw_ctx *hctx,
>> +                              struct request *rq, void *priv,
>> +                              bool reserved)
>> +{
>> +    struct mq_inflight *mi = priv;
>> +
>> +    if (!test_bit(REQ_ATOM_STARTED, &rq->atomic_flags))
>> +            return;
>> +
>> +    /*
>> +     * Count as inflight if it either matches the partition we asked
>> +     * for, or if it's the root
>> +     */
>> +    if (rq->part == mi->part || mi->part->partno)
>> +            mi->inflight[0]++;
>> +}
> 
> Hello Jens,
> 
> How to check for the root? Does partno == 0 or partno != 0 represent the
> root? Please note that I'm not familiar with the partition code.

if partno != 0, it's a partition. So for that, account for root too. I
guess the comment could be better...

-- 
Jens Axboe

Reply via email to