[Ocfs2-devel] [PATCH] ocfs2: Cache some system inodes of other nodes.

[Joel Becker <Joel.Becker@oracle.com>]

On Fri, Aug 13, 2010 at 09:21:26AM +0800, Tao Ma wrote:
> Sunil Mushran wrote:
> >        if (is_global_system_inode(type)
> >                arr = &(osb->global_system_inodes[type]);
> >        else {
> >                BUG_ON(slot == OCFS2_INVALID_SLOT);
> >                BUG_ON(type < OCFS2_FIRST_LOCAL_SYSTEM_INODE ||
> >                        type > LOCAL_GROUP_QUOTA_SYSTEM_INODE);
> >                tmp = (slot * NUM_LOCAL_SYSTEM_INODES) +
> >                      (type - OCFS2_FIRST_LOCAL_SYSTEM_INODE);
> >                arr = &(osb->local_system_inodes[tmp];
> >        }
> >
> >How does this look?
> oh, since both of you prefer a simple system_inode array, I will
> rework it. Thanks.

	In Sunil's proposal, you can easily keep the "which remote
inodes to keep" question very local.

		if ((slot == my_slot) ||
		    (type == TYPE1) ||
		    (type == TYPE2) ...) {
			local_system_inodes[x] = inode;

Joel


-- 

"Conservative, n.  A statesman who is enamoured of existing evils,
 as distinguished from the Liberal, who wishes to replace them
 with others."
	- Ambrose Bierce, The Devil's Dictionary

Joel Becker
Consulting Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127