Re: [PATCH 12/49] xfs: move inode fork definitions to a new header file

Mark Tinguely
2013-07-25T20:40:38Z

On 07/19/13 01:24, Dave Chinner wrote:
> From: Dave Chinner<dchinner@redhat.com>
>
> The inode fork definitions are a combination of on-disk format
> definition and in-memory tracking and manipulation. They are both
> shared with userspace, so move them all into their own file so
> sharing is easy to do and track. This removes all inode fork
> related information from xfs_inode.h.
>
> Do the same for the all the C code that currently resides in
> xfs_inode.c for the same reason.
>
> Signed-off-by: Dave Chinner<dchinner@redhat.com>
> ---

summary:

Mostly a move of routines/structures from xfs_inode.c/h to
xfs_inode_fork.c/h.

xfs_validate_extents() is no longer a STATIC, as expected.

xfs_iformat() is renamed xfs_iformat_fork().

function declarations in xfs_inode_fork.h have the typedef converted to
structs.

Looks good.

Reviewed-by: Mark Tinguely <tinguely@sgi.com>