Re: [PATCH 28/37] xfs: move xfsagino_t to xfs_types.h

Christoph Hellwig
2012-04-29T21:49:19Z

On Mon, Apr 23, 2012 at 03:58:58PM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> Untangle the header file includes a bit by moving the definition of
> xfs_agino_t to xfs_types.h. This removes the dependency that xfs_ag.h has on
> xfs_inum.h, meaning we don't need to include xfs_inum.h everywhere we include
> xfs_ag.h.

This patch by itself looks good to me, but is there any good reason to
keep xfs_inum.h around at all?