[lkml]   [2006]   [Jan]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Busy inodes after unmount, be more verbose in generic_shutdown_super
    >>On Tue, Jan 17, Kirill Korotaev wrote:
    >>>Olaf, can you please check if my patch for busy inodes from -mm tree
    >>>helps you?
    >>>Patch name is fix-of-dcache-race-leading-to-busy-inodes-on-umount.patch
    >>This patch is just wrong. It is hiding bugs in file systems. The problem is
    >>that somewhere the reference counting on the vfsmount objects is wrong. The
    >>file system is unmounted before the last dentry is dereferenced. Either you
    >>didn't hold a reference to the proper vfsmount objects at all or you
    >>dereference it too early. See Al Viros patch series (search for "namei fixes")
    >>on how to fix this issues.
    > The only reason I've been carrying that patch is as a reminder that there's
    > a bug that we need to fix. It'd be good news if that bug had been fixed by
    > other means.
    > Kirill, do you know whether the bug is still present in 2.6.16-rc1?

    it exists in 2.6.15 and I see no changes in 2.6.16-rc1 except for
    cosmetics :(
    checked the git tree, dput() etc. the bug is definetely still here -
    nothing changed in this area.

    Sorry for bad news.

    The patch can be probably remade via introducing "notlocked_refs"
    counter on dentry. if shrinker()/umount() see such a dentry() with
    non-zero refcnt it can sleep as it is done in current patch. It would be
    a little bit cleaner/simpler. What do you think?
    If someone suggest any brilliant/helpfull idea I would be happy to
    improve it.


    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2006-01-19 11:09    [W:0.021 / U:21.280 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site