• Amir Goldstein's avatar
    ovl: hash non-dir by lower inode for fsnotify · 764baba8
    Amir Goldstein authored
    Commit 31747eda ("ovl: hash directory inodes for fsnotify")
    fixed an issue of inotify watch on directory that stops getting
    events after dropping dentry caches.
    
    A similar issue exists for non-dir non-upper files, for example:
    
    $ mkdir -p lower upper work merged
    $ touch lower/foo
    $ mount -t overlay -o
    lowerdir=lower,workdir=work,upperdir=upper none merged
    $ inotifywait merged/foo &
    $ echo 2 > /proc/sys/vm/drop_caches
    $ cat merged/foo
    
    inotifywait doesn't get the OPEN event, because ovl_lookup() called
    from 'cat' allocates a new overlay inode and does not reuse the
    watched inode.
    
    Fix this by hashing non-dir overlay inodes by lower real inode in
    the following cases that were not hashed before this change:
     - A non-upper overlay mount
     - A lower non-hardlink when index=off
    
    A helper ovl_hash_bylower() was added to put all the logic and
    documentation about which real inode an overlay inode is hashed by
    into one place.
    
    The issue dates back to initial version of overlayfs, but this
    patch depends on ovl_inode code that was introduced in kernel v4.13.
    
    Cc: <stable@vger.kernel.org> #v4.13
    Signed-off-by: default avatarAmir Goldstein <amir73il@gmail.com>
    Signed-off-by: default avatarMiklos Szeredi <mszeredi@redhat.com>
    764baba8
inode.c 20 KB