On 5/19/23 21:56, Christoph Hellwig wrote:
On Fri, May 19, 2023 at 12:55:45PM -0700, Bart Van Assche wrote:
LIST_HEAD(rq_list);
- struct request *rq, *next;
+ LIST_HEAD(flush_list);
+ struct request *rq;
spin_lock_irq(&q->requeue_lock);
list_splice_init(&q->requeue_list, &rq_list);
+ list_splice_init(&q->flush_list, &flush_list);
spin_unlock_irq(&q->requeue_lock);
"rq_list" stands for "request_list". That name is now confusing since this patch
add a second request list (flush_list).
It is. But I think you were planning on doing a bigger rework in this
area anyway?
Yes, that's right. I plan to remove both rq_list and flush_list from
this function.
Bart.
