On Thursday 27 March 2008, Bob Copeland wrote: > +static struct inode *omfs_alloc_inode(struct super_block *sb) > +{ > + struct omfs_inode_info *oinf; > + oinf = (struct omfs_inode_info *) kmem_cache_alloc(omfs_inode_cachep, > + GFP_KERNEL); > + if (!oinf) > + return NULL; > + return &oinf->vfs_inode; > +} If you could avoid the i_state member in omfs_inode_info, you could get rid of your own inode cache entirely. Your i_state seems to be mostly write-only, why is it needed in your file system? Arnd <>< -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html