Re: [PATCH v7] loop: Fix NULL pointer dereference in lo_rw_aio()
From: Bart Van Assche
Date: Mon Aug 31 2026 - 14:15:57 EST
On 8/28/26 10:17 PM, Tetsuo Handa wrote:
Calling synchronize_rcu() makes sure that no more queue_work() calls are
made from loop_queue_work() from loop_queue_rq(). Since loop_queue_rq() is
called with RCU read lock, subsequent loop_queue_rq() calls which are made
after synchronize_rcu() returned shall see lo->lo_state != Lo_bound and
return with BLK_STS_IOERR.
Please add a comment above the synchronize_rcu() call that explains the
purpose of that call.
That is OK. But after you agreed that my patch works as expected.I appreciate your work and agree that this patch takes the right
direction. But I would appreciate it if you would take another look at
my review comments.
Thanks,
Bart.