[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1296846886-sup-1431@think>
Date: Fri, 04 Feb 2011 14:15:17 -0500
From: Chris Mason <chris.mason@...cle.com>
To: Tao Ma <tm@....ma>
Cc: Boaz Harrosh <bharrosh@...asas.com>,
Nick Piggin <npiggin@...il.com>,
linux-fsdevel <linux-fsdevel@...r.kernel.org>,
ext4 development <linux-ext4@...r.kernel.org>
Subject: Re: [BUG] v2.6.38-rc3+ BUG when calling destroy_inodecache at module unload
Excerpts from Tao Ma's message of 2011-02-04 03:36:59 -0500:
> On 02/04/2011 02:51 AM, Boaz Harrosh wrote:
> > Last good Kernel was 2.6.37
> > I'm doing a "mount" then "unmount". I think root is the only created inode.
> > rmmod is called immediately after "unmount" within a script
> >
> > if I only do unmount and manually call "modprobe --remove exofs" after a small while
> > all is fine.
> >
> > I get:
> > slab error in kmem_cache_destroy(): cache `exofs_inode_cache': Can't free all objects
> > Call Trace:
> > 77dfde08: [<6007e9a6>] kmem_cache_destroy+0x82/0xca
> > 77dfde38: [<7c1fa3da>] exit_exofs+0x1a/0x1c [exofs]
> > 77dfde48: [<60054c10>] sys_delete_module+0x1b9/0x217
> > 77dfdee8: [<60014d60>] handle_syscall+0x58/0x70
> > 77dfdf08: [<60024163>] userspace+0x2dd/0x38a
> > 77dfdfc8: [<600126af>] fork_handler+0x62/0x69
> >
> I also get a similar error when testing ext4 and a bug is opened there.
>
> https://bugzilla.kernel.org/show_bug.cgi?id=27652
>
> And I have done some simple investigation for ext4 and It looks as if now with the new *fs_i_callback doesn't free the inode to *fs_inode_cache immediately. So the old logic will destroy the inode cache before we free all the inode object.
>
> Since there are more than one fs affected by this, we may need to find a way in the VFS.
Sounds like we just need a synchronize_rcu call before we delete the
cache?
-chris
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists