Re: [PATCH 20/26] move xfs_bmbt_newroot to common code

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Mon, Aug 04, 2008 at 03:35:30AM +0200, Christoph Hellwig wrote:
> xfs_bmbt_newroot is a mostly generic implementation of moving from
> an inode root to a real block based root.  So move it to xfs_btree.c
> where it can use all the nice infrastructure there and make it pointer
> size agnostic
> 
> The new name for it is xfs_btree_iroot_to_root which is not very
> nice but at least slightly more descriptive than the old name.

.....
> +
> +     XFS_BTREE_TRACE_CURSOR(cur, XBT_ENTRY);
> +     XFS_BTREE_STATS_INC(cur, newroot);
> +
> +     level = cur->bc_nlevels - 1;
> +     /* XXX(hch): this should be an get_inode_from_root, right? */
> +     block = xfs_btree_get_block(cur, level, &bp);

Yes, probably should be. Also an assert to ensure the btree does
have the root in inode falg set might be appropriate....

Otherwise looks fine.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx