Commit 56adcda5 authored Dec 22, 2021 by Darrick J. Wong Committed by Greg Kroah-Hartman Jan 11, 2022

xfs: map unwritten blocks in XFS_IOC_{ALLOC,FREE}SP just like fallocate



commit 983d8e60f50806f90534cc5373d0ce867e5aaf79 upstream.

The old ALLOCSP/FREESP ioctls in XFS can be used to preallocate space at
the end of files, just like fallocate and RESVSP.  Make the behavior
consistent with the other ioctls.

Reported-by: Kirill Tkhai <ktkhai@virtuozzo.com>
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent ca4b6a60

fs/xfs/xfs_ioctl.c

+2 −1

Original line number	Original line	Diff line number	Diff line
	@@ -729,7 +729,8 @@ xfs_ioc_space(
	flags \|= XFS_PREALLOC_CLEAR;		flags \|= XFS_PREALLOC_CLEAR;
	if (bf->l_start > XFS_ISIZE(ip)) {		if (bf->l_start > XFS_ISIZE(ip)) {
	error = xfs_alloc_file_space(ip, XFS_ISIZE(ip),		error = xfs_alloc_file_space(ip, XFS_ISIZE(ip),
	bf->l_start - XFS_ISIZE(ip), 0);		bf->l_start - XFS_ISIZE(ip),
			XFS_BMAPI_PREALLOC);
	if (error)		if (error)
	goto out_unlock;		goto out_unlock;
	}		}