diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2007-02-14 00:33:35 -0800 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-02-14 08:09:56 -0800 |
commit | 4b998d4f6e559fb8eba8141263b2fddc9556391b (patch) | |
tree | 0d18c4954908192c3d9bdc5e2a6431e17050d320 /drivers/zorro/names.c | |
parent | 208367eeec8d6f6fcee1a0672af2149aeee7b658 (diff) | |
download | kernel-crypto-4b998d4f6e559fb8eba8141263b2fddc9556391b.tar.gz kernel-crypto-4b998d4f6e559fb8eba8141263b2fddc9556391b.tar.xz kernel-crypto-4b998d4f6e559fb8eba8141263b2fddc9556391b.zip |
[PATCH] sysctl: cdrom: don't set de->owner
There is no need for open files in /proc/sys/XXX to hold a reference count on
the module that provides the file to prevent module unload races. While there
is code active in the module p->used in the sysctl_table_header is
incremented, preventing the sysctl from being unregisted. Once the sysctl is
unregistered it cannot be found. Open files are also not a problem as they
revalidate the sysctl information and bump p->used before accessing module
code.
So setting de->owner is unnecessary, makes for a bad example and gets in my
way of removing ctl_table->de.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Acked-by: Jens Axboe <jens.axboe@oracle.com>
Acked-by: James Bottomley <James.Bottomley@SteelEye.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/zorro/names.c')
0 files changed, 0 insertions, 0 deletions