diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2006-09-27 04:49:40 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-27 11:26:10 -0400 |
commit | 1a1d92c10dd24bbdc28b3d6e2d03ec199dd3a65b (patch) | |
tree | fade83955f75e718e39153d6f81d221403338bed /fs/ntfs/super.c | |
parent | f52720ca5f48574e347dff35ffe6b389ace61537 (diff) |
[PATCH] Really ignore kmem_cache_destroy return value
* Rougly half of callers already do it by not checking return value
* Code in drivers/acpi/osl.c does the following to be sure:
(void)kmem_cache_destroy(cache);
* Those who check it printk something, however, slab_error already printed
the name of failed cache.
* XFS BUGs on failed kmem_cache_destroy which is not the decision
low-level filesystem driver should make. Converted to ignore.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/ntfs/super.c')
-rw-r--r-- | fs/ntfs/super.c | 28 |
1 files changed, 5 insertions, 23 deletions
diff --git a/fs/ntfs/super.c b/fs/ntfs/super.c index 74e0ee8fce72..6b2712f10dd2 100644 --- a/fs/ntfs/super.c +++ b/fs/ntfs/super.c | |||
@@ -3248,32 +3248,14 @@ ictx_err_out: | |||
3248 | 3248 | ||
3249 | static void __exit exit_ntfs_fs(void) | 3249 | static void __exit exit_ntfs_fs(void) |
3250 | { | 3250 | { |
3251 | int err = 0; | ||
3252 | |||
3253 | ntfs_debug("Unregistering NTFS driver."); | 3251 | ntfs_debug("Unregistering NTFS driver."); |
3254 | 3252 | ||
3255 | unregister_filesystem(&ntfs_fs_type); | 3253 | unregister_filesystem(&ntfs_fs_type); |
3256 | 3254 | kmem_cache_destroy(ntfs_big_inode_cache); | |
3257 | if (kmem_cache_destroy(ntfs_big_inode_cache) && (err = 1)) | 3255 | kmem_cache_destroy(ntfs_inode_cache); |
3258 | printk(KERN_CRIT "NTFS: Failed to destory %s.\n", | 3256 | kmem_cache_destroy(ntfs_name_cache); |
3259 | ntfs_big_inode_cache_name); | 3257 | kmem_cache_destroy(ntfs_attr_ctx_cache); |
3260 | if (kmem_cache_destroy(ntfs_inode_cache) && (err = 1)) | 3258 | kmem_cache_destroy(ntfs_index_ctx_cache); |
3261 | printk(KERN_CRIT "NTFS: Failed to destory %s.\n", | ||
3262 | ntfs_inode_cache_name); | ||
3263 | if (kmem_cache_destroy(ntfs_name_cache) && (err = 1)) | ||
3264 | printk(KERN_CRIT "NTFS: Failed to destory %s.\n", | ||
3265 | ntfs_name_cache_name); | ||
3266 | if (kmem_cache_destroy(ntfs_attr_ctx_cache) && (err = 1)) | ||
3267 | printk(KERN_CRIT "NTFS: Failed to destory %s.\n", | ||
3268 | ntfs_attr_ctx_cache_name); | ||
3269 | if (kmem_cache_destroy(ntfs_index_ctx_cache) && (err = 1)) | ||
3270 | printk(KERN_CRIT "NTFS: Failed to destory %s.\n", | ||
3271 | ntfs_index_ctx_cache_name); | ||
3272 | if (err) | ||
3273 | printk(KERN_CRIT "NTFS: This causes memory to leak! There is " | ||
3274 | "probably a BUG in the driver! Please report " | ||
3275 | "you saw this message to " | ||
3276 | "linux-ntfs-dev@lists.sourceforge.net\n"); | ||
3277 | /* Unregister the ntfs sysctls. */ | 3259 | /* Unregister the ntfs sysctls. */ |
3278 | ntfs_sysctl(0); | 3260 | ntfs_sysctl(0); |
3279 | } | 3261 | } |