Messages in this thread | | | Date | Wed, 14 Nov 2012 10:46:15 +0400 | From | Cyrill Gorcunov <> | Subject | Re: [patch 3/7] fs, notify: Add file handle entry into inotify_inode_mark |
| |
On Tue, Nov 13, 2012 at 02:38:08PM -0800, Andrew Morton wrote: > On Tue, 13 Nov 2012 12:00:32 +0400 > Cyrill Gorcunov <gorcunov@openvz.org> wrote: > > > > Dumb question: do we really need inotify_inode_mark.fhandle at all? > > > What prevents us from assembling this info on demand when ->show_fdinfo() is > > > called? > > > > exportfs requires the dentry to be passed as an argument while inotify works > > with inodes instead and at moment of show-fdinfo the target dentry might be > > already deleted but inode yet present as far as I remember. > > How can the c/r restore code reestablish the inode data if the dentry > isn't there any more?
By "deleted" I meant deleted from dcache, thus when we call for open_by_handle_at with fhandle, the kernel reconstruct the path and we simply read the /proc/self/fd/ link, and then pass this path to inotify_add_watch.
| |