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