lkml.org 
[lkml]   [2002]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] iget_locked [6/6]


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/

\
 
 \ /
  Last update: 2005-03-22 13:26    [W:0.032 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site