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: [email protected]
Phone: (650) 506-8127
_______________________________________________
Ocfs2-devel mailing list
[email protected]
http://oss.oracle.com/mailman/listinfo/ocfs2-devel
