[PATCH 05/16] xfs: decouple inode and bmap btree header files

Dave Chinner david at fromorbit.com
Thu Oct 3 20:00:09 CDT 2013


On Thu, Oct 03, 2013 at 07:58:49AM -0700, Christoph Hellwig wrote:
> I like this a lot, but it also seems to move the in-core btree records
> and keys to the format header which seems wrong.  Maybe those should be
> centralized in xfs_btree.h instead?

Which structures are you talking about here? there's so many key and
pointer definitions I'm not sure if there's a specific one you had
in mind or whether you mean "all of them"...

Keep in mind here that the real dependency problem is the
xfs_bmbt_rec_host definition being require by xfs_inode_fork.h.
That's an in-core definition, and if we move that to another header
file, we simply create a different dependency instead of removing
the dependency altogether....

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list