Re: [PATCH 1/2] blk-mq-debugfs: support rq_qos

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/16/18 9:09 AM, Christoph Hellwig wrote:
> On Sun, Dec 16, 2018 at 09:04:37AM -0700, Jens Axboe wrote:
>> It would fit better with the style in that code do to:
>>
>> rqos = q->rq_qos;
>> while (rqos) {
>>     ....
>>     rqos = rqos->next;
>> }
> 
> But that is just a really strangly written for loop to start with..

I think it looks fine, and it's more readable than using a for()
loop for something like this.

> Maybe Bart need to send a cleanup patch for the whole file, especially
> given that this patch has been applied already.

As mentioned in the same thread, I dropped these since I had to
amend a commit anyway. So we're free to do whatever.

-- 
Jens Axboe




[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux