Re: [PATCH] iget_locked [6/6]

From: Alexander Viro (viro@math.psu.edu)
Date: Fri May 10 2002 - 15:13:47 EST


On Fri, 10 May 2002, Jan Harkes wrote:

>
> As of the previous patch the inode_hashtable doesn't really need to be
> indexed by i_ino anymore, the only reason we still have to keep the
> hashvalue and i_ino identical is because of insert_inode_hash.
>
> Here we simply add an argument to insert_inode_hash. If at some point a
> FS specific getattr method is implemented it will be possible to
> completely remove all uses of i_ino in the VFS.

How about

static inline void insert_inode_hash(struct inode *inode)
{
        __insert_inode_hash(inode, inode->i_hash);
}

in fs.h and switching those who want something different to direct use
of __insert_inode_hash()?

It's going to remain a very common case and IMO it makes a lot of sense
to keep a simple helper for it. That has a nice property of getting
patch way smaller ;-)

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue May 14 2002 - 12:00:14 EST