On Fri, 10 Mar 2006, Andrew Morton wrote:

> Ian Kent <[EMAIL PROTECTED]> wrote:
> >
> > @@ -337,10 +340,34 @@ static void *autofs4_follow_link(struct 
> >     if (oz_mode || !lookup_type)
> >             goto done;
> >  
> > +   /*
> > +    * If the dentry contains directories then it is an
> > +    * autofs multi-mount with no root offset. So don't
> > +    * try to mount it again.
> > +    */
> > +   spin_lock(&dcache_lock);
> > +   if (!list_empty(&dentry->d_subdirs)) {
> > +           spin_unlock(&dcache_lock);
> > +           goto done;
> > +   }
> > +   spin_unlock(&dcache_lock);
> > +
> 
> Can list_empty(&dentry->d_subdirs) become false right here, after the lock
> was dropped?  If so, what happens?

Yep. I think so.
Not what I want to happen.

> 
> 
> >     status = try_to_fill_dentry(dentry, 0);
> 

_______________________________________________
autofs mailing list
[email protected]
http://linux.kernel.org/mailman/listinfo/autofs

Reply via email to