diff options
author | Dave Peterson <dsp@llnl.gov> | 2006-03-26 01:38:49 -0800 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-26 08:57:07 -0800 |
commit | 472678ebd30d87cbe8d97562dcc0e46d1076040f (patch) | |
tree | 216f22045fb3b786f513f000b184e5fd449b58d1 /drivers/edac/i82875p_edac.c | |
parent | 6e5a8748507dea83386d1d76c58aeaed1ff5a1ec (diff) | |
download | op-kernel-dev-472678ebd30d87cbe8d97562dcc0e46d1076040f.zip op-kernel-dev-472678ebd30d87cbe8d97562dcc0e46d1076040f.tar.gz |
[PATCH] EDAC: kobject/sysfs fixes
- After we unregister a kobject, wait for our kobject release method
to call complete(). This causes us to wait until the kobject
reference count reaches 0. Otherwise, a task accessing the EDAC
sysfs interface can hold the reference count above 0 until after the
EDAC module has been unloaded. When the reference count finally
drops to 0, this will result in an attempt to call our release
method inside the EDAC module after the module has already been
unloaded.
This isn't the best fix, since a process can get stuck sleeping forever
uninterruptibly if the user does the following:
rmmod my_module < /sys/my_sysfs/file
I'll go back and implement a better fix later. However this should
be ok for now.
- Call edac_remove_sysfs_mci_device() from edac_mc_del_mc() rather
than from edac_mc_free(). Since edac_mc_add_mc() calls
edac_create_sysfs_mci_device(), edac_mc_del_mc() should call
edac_remove_sysfs_mci_device().
Signed-off-by: David S. Peterson <dsp@llnl.gov>
Cc: Greg KH <greg@kroah.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/edac/i82875p_edac.c')
0 files changed, 0 insertions, 0 deletions