Re: [PATCH v3] blktrace: Fix potentail deadlock between delete & sysfs ops
From: Bart Van Assche
Date: Fri Aug 18 2017 - 14:07:57 EST
On Fri, 2017-08-18 at 13:54 -0400, Waiman Long wrote:
> Instead, a global blktrace
> mutex will be used to serialize the read/write of the blktrace sysfs
> attributes.
Hello Waiman,
Using a mutex to serialize code is wrong. What is needed is exact
documentation of what data structures and/or member variables are
protected by bdev->bd_mutex and by blktrace_mutex.
Also, please spell "potential" correctly in the subject of this patch.
Thanks,
Bart.