Re: [PATCH 4/5] block: Inline blk_rq_set_prio()

[Jens Axboe]

On 04/19/2017 12:20 AM, Christoph Hellwig wrote:
>> +    req->ioprio = ioprio_valid(bio_prio(bio)) ? bio_prio(bio) : ioc ?
>> +            ioc->ioprio : IOPRIO_PRIO_VALUE(IOPRIO_CLASS_NONE, 0);
> 
> I think this would be a tad cleaner with a traditional if / else if / else
> chain, e.g.
> 
>       if (ioprio_valid(bio_prio(bio)))
>               req->ioprio = bio_prio(bio);
>       else if (ioc)
>               req->ioprio = ioc->ioprio;
>       else
>               req->ioprio = IOPRIO_PRIO_VALUE(IOPRIO_CLASS_NONE, 0);

Agree, I hate ternaries with a vengeance, and a doubly nested one is
almost a shooting offense.

Bart, care to respin with this fixed and the GPL export modification?

-- 
Jens Axboe