On Tue, Sep 20, 2005 at 09:41:47PM -0400, John McCutchan wrote:
> I grepped all the filesystems
... in the tree
> , and they all seem to use
> generic_drop_inode, except for hugetlbfs, which seems to have the same
> logic of (!inode->i_nlink).
I have no problems with killing ->drop_inode(), but that should be
a) done for in-tree filesystems
b) announced on fsdevel, so that out-of-tree folks could deal
with that
c) given at least one release to avoid screwing them.
Christoph, could you send the patch you've mentioned? I'd rather avoid
duplicating what you've done...
> > BTW, what happens if one uses inotify on procfs? Or sysfs, for that matter?
> > Fundamental problem with that sucker is that you are playing games with
> > lifetime rules of inodes in a way that might be OK for some filesystems,
> > but violates a lot of assumptions made by other...
> >
>
> Honestly, I don't know. And I don't think I know enough to say with any
> certainty how either of them would work. Would a black list of
> filesystems that don't want inotify on them be acceptable?
Maybe... Alternatively, it might be interesting to see if differences
between filesystem requirements can be handled by secondary method
table that would be set at ->get_sb() time (or merged into super_operations
if we end up with few of them).
I'd certainly love to deal with sys_unlink() kludge that way, BTW - two
new methods, with vfs_unlink() ending with
up(&dentry->d_inode->i_sem);
/* NOTE: fast path should be dealt with in real version, this
is just a demo */
if (error)
return ERR_PTR(error);
else if (!dir->i_op->post_unlink)
return generic_post_unlink(dentry);
else
return dir->i_op->post_unlink(dentry);
}
with callers doing
p = vfs_unlink(dir, dentry);
...
up(&dir->i_sem);
err = vfs_finish_unlink(dir, p);
vfs_finish_unlink(dir, p) (again, that would need to be optimized for fast path)
if (IS_ERR(p))
return PTR_ERR(p);
if (!dir->i_op->finish_unlink)
return generic_finish_unlink(dir, p);
else
return dir->i_op->finish_unlink(dir, p);
generic_post_unlink(dentry): grab a reference to dentry->d_inode, call
d_delete(), return inode
generic_finish_unlink(dir, p): iput(p); return 0;
NFS: don't mess with inode refcount at all *and* make sure we don't
d_delete() sillyrenamed ones.
sys_unlink(): no more messing with inodes behind fs back.
Price: change of public API (vfs_unlink() changes the type of return value
*and* calling conventions - we need to pass its return value to
vfs_finish_unlink() after unlocking the parent).
> > BTW^2, what guarantees that inotify_unmount_inodes() will not happen while we
> > are in inotify_release()? That would happily keep watch refcount bumped,
> > so it would outlive inotify_unmount_inodes(). Sure, it would be dropped.
> > And call iput() on a pinned inode that had outlived the umount(). Oops...
>
> Good catch,
[snip]
Why don't you simply put watches on a list anchored at superblock and
go through that list instead of messing with inode lists?
Speaking of disappearing ->d_inode problem... How about collecting
watches on a temporary list while holding dcache_lock and then hitting
them all with "it's going away" if ->i_nlink has hit zero? You don't
need inode to be around for the last part - you only use it to find watches
in question. And that can be done without any allocations...
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
|
|