get_empty_inode is hopefully going to die in the current, non-static version.
> (Another thing
> we could do is require every inode to have a superblock - that's probably
> where it will go in time.)
Any inode that gets into the icache already has and superblock.
If any other are left they really should use a different allocator.
> We put this inside get_empty_inode:
>
> if (inode) {
> inode->i_dev = sb->s_dev;
> inode->i_blkbits = sb->s_blocksize_bits;
> }
>
> then rename it new_inode. But this is outside of the scope of the fs.h work
> I'm doing, don't you think?
Rename your current get_empty_inode to __get_empty_inode and mark it
static. Add a new get_empty_inode that calls __get_empty_inode(NULL) or
better let it die.
Christoph
-- Of course it doesn't work. We've performed a software upgrade. - 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/