lkml.org 
[lkml]   [2008]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: Linux 2.6.26-rc4
    From
    Date

    On Tue, 2008-06-03 at 11:52 +0100, Al Viro wrote:
    > On Tue, Jun 03, 2008 at 12:45:33PM +0200, Miklos Szeredi wrote:
    >
    > > I think it must be autofs4 doing something weird. Like this in
    > > autofs4_lookup_unhashed():
    > >
    > > /*
    > > * Make the rehashed dentry negative so the VFS
    > > * behaves as it should.
    > > */
    > > if (inode) {
    > > dentry->d_inode = NULL;
    >
    > Lovely. If we ever step into that with somebody else (no matter who)
    > holding a reference to that dentry, we are certainly well and truly
    > buggered. It's not just mount(2) - everything in the tree assumes that
    > holding a reference to positive dentry guarantees that it remains
    > positive.

    The intent here is that, the dentry above is unhashed at this point, and
    if hasn't been reclaimed by the VFS, it is made negative and replaces
    the unhashed negative dentry passed to ->lookup(). The reference count
    is incremented to account for the reference held by the path walk.

    What am I doing wrong here?

    Ian




    \
     
     \ /
      Last update: 2008-06-03 15:49    [W:0.121 / U:29.372 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site