On Wed, Dec 09, 2009 at 05:51:17PM +0300, Dmitry Monakhov wrote: > >From other point of view, we may change inode structure like this: > struct quota_ptr > { > struct dquot *dquot[MAXQUOTAS]; > }; > struct quota_rsv_ptr > { > struct dquot *dquot[MAXQUOTAS]; > qsize_t reservation; > }; > struct inode { > .... > #ifdef CONFIG_QUOTA > union { > struct quota_ptr i_dquot; > struct quota_rsv_ptr i_dquot_rsv; > }; > #endif > .... > }; What's wrong with: #ifdef CONFIG_QUOTA union { struct dquot *i_dquot[MAXQUOTAS]; struct quota_rsv_ptr i_dquot_rsv; }; #endif voila, no changes needed. -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html