On Fri, 07 Jul 2006 11:15:55 +0800 Ian Kent <raven@xxxxxxxxxx> wrote: > On Thu, 2006-07-06 at 18:56 -0700, Andrew Morton wrote: > > On Thu, 06 Jul 2006 11:11:07 -0400 > > Trond Myklebust <Trond.Myklebust@xxxxxxxxxx> wrote: > > > > > From: David Howells <dhowells@xxxxxxxxxx> > > > > > > Make sure all dentries refs are released before calling kill_anon_super() so > > > that the assumption that generic_shutdown_super() can completely destroy the > > > dentry tree for there will be no external references holds true. > > > > > > What was being done in the put_super() superblock op, is now done in the > > > kill_sb() filesystem op instead, prior to calling kill_anon_super(). > > > > > > This makes the struct autofs_sb_info::root member variable redundant (since > > > sb->s_root is still available), and so that is removed. The calls to > > > shrink_dcache_sb() are also removed since they're also redundant as > > > shrink_dcache_for_umount() will now be called after the cleanup routine. > > > > > > > AFAIK, Ian is still off thinking about this one? > > The patch looks fine to me but I wanted to give it a basic sanity test > before ACKing it. I haven't got to that yet partly because I wasn't sure > if the two patches from David would stand alone. Will they or will I > need to apply Davids whole patch set? I expect this patch is standalone, but it's not really useful or interesting to test it unless [21/21] is also applied, because the two patches will tend to stick together. Plus this one is supposed to fix a umount/shutdown-time BUG in the other. > Anyway, I'll have a go at it this afternoon. Thanks. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
