diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2010-02-11 15:21:53 -0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-03-07 17:04:51 -0800 |
commit | 6992f5334995af474c2b58d010d08bc597f0f2fe (patch) | |
tree | bef650bfb9355b8d7cbb7a5acb3632c02ec7fbe7 /kernel/params.c | |
parent | a2db6842873c8e5a70652f278d469128cb52db70 (diff) | |
download | lwn-6992f5334995af474c2b58d010d08bc597f0f2fe.tar.gz lwn-6992f5334995af474c2b58d010d08bc597f0f2fe.zip |
sysfs: Use one lockdep class per sysfs attribute.
Acknowledge that the logical sysfs rwsem has one instance per
sysfs attribute with different locking depencencies for different
attributes.
There is a sysfs idiom where writing to one sysfs file causes the
addition or removal of other sysfs files. Lumping all of the
sysfs attributes together in one lock class causes lockdep to
generate lots of false positives.
This introduces the requirement that non-static sysfs attributes
need to be initialized with sysfs_attr_init or sysfs_bin_attr_init.
Strictly speaking this requirement only exists when lockdep is
enabled, and when lockdep is enabled we get a bit fat warning
if this requirement is not met.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Acked-by: WANG Cong <xiyou.wangcong@gmail.com>
Cc: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'kernel/params.c')
0 files changed, 0 insertions, 0 deletions