Commit 5315837d authored Jan 27, 2011 by Dave Chinner Committed by Alex Elder Jan 28, 2011

xfs: limit extsize to size of AGs and/or MAXEXTLEN



The extent size hint can be set to larger than an AG. This means
that the alignment process can push the range to be allocated
outside the bounds of the AG, resulting in assert failures or
corrupted bmbt records. Similarly, if the extsize is larger than the
maximum extent size supported, the alignment process will produce
extents that are too large to fit into the bmbt records, resulting
in a different type of assert/corruption failure.

Fix this by limiting extsize at the time іt is set firstly to be
less than MAXEXTLEN, then to be a maximum of half the size of the
AGs in the filesystem for non-realtime inodes. Realtime inodes do
not allocate out of AGs, so don't have to be restricted by the size
of AGs.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Alex Elder <aelder@sgi.com>

parent 4ce15989

fs/xfs/linux-2.6/xfs_ioctl.c

+18 −2

Original line number	Diff line number	Diff line
		@@ -985,10 +985,22 @@ xfs_ioctl_setattr(

		/*
		* Extent size must be a multiple of the appropriate block
		* size, if set at all.
		* size, if set at all. It must also be smaller than the
		* maximum extent size supported by the filesystem.
		*
		* Also, for non-realtime files, limit the extent size hint to
		* half the size of the AGs in the filesystem so alignment
		* doesn't result in extents larger than an AG.
		*/
		if (fa->fsx_extsize != 0) {
		xfs_extlen_t size;
		xfs_fsblock_t extsize_fsb;

		extsize_fsb = XFS_B_TO_FSB(mp, fa->fsx_extsize);
		if (extsize_fsb > MAXEXTLEN) {
		code = XFS_ERROR(EINVAL);
		goto error_return;
		}

		if (XFS_IS_REALTIME_INODE(ip) \|\|
		((mask & FSX_XFLAGS) &&
		@@ -997,6 +1009,10 @@ xfs_ioctl_setattr(
		mp->m_sb.sb_blocklog;
		} else {
		size = mp->m_sb.sb_blocksize;
		if (extsize_fsb > mp->m_sb.sb_agblocks / 2) {
		code = XFS_ERROR(EINVAL);
		goto error_return;
		}
		}

		if (fa->fsx_extsize % size) {

Admin message