[PATCH 27/49] xfs: consolidate extent swap code

Mark Tinguely tinguely at sgi.com
Fri Jul 26 16:16:55 CDT 2013


On 07/19/13 01:24, Dave Chinner wrote:
> From: Dave Chinner<dchinner at redhat.com>
>
> So we don't need xfs_dfrag.h in userspace anymore, move the
> extent swap ioctl structure definition to xfs_fs.h
> where most of the other ioctl structure definitions are.
>
> Now that we don't need separate files for extent swapping,
> separate the basic file descriptor checking code to xfs_ioctl.c,
> and the code that does the extent swap operation to xfs_bmap_util.c.
> This cleanly separates the user interface code from the physical
> mechanism used to do the extent swap.
>
> Signed-off-by: Dave Chinner<dchinner at redhat.com>
> ---

Looks good. Only change is xfs_swapext() is rename xfs_ioc_swapext() 
when moved to xfs_ioctl.c.

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



More information about the xfs mailing list