Commit 30ca22c7 authored by Patrick J. LoPresti's avatar Patrick J. LoPresti Committed by Joel Becker
Browse files

ext3/ext4: Factor out disk addressability check 



As part of adding support for OCFS2 to mount huge volumes, we need to
check that the sector_t and page cache of the system are capable of
addressing the entire volume.

An identical check already appears in ext3 and ext4.  This patch moves
the addressability check into its own function in fs/libfs.c and
modifies ext3 and ext4 to invoke it.

[Edited to -EINVAL instead of BUG_ON() for bad blocksize_bits -- Joel]

Signed-off-by: default avatarPatrick LoPresti <lopresti@gmail.com>
Cc: linux-ext4@vger.kernel.org
Acked-by: default avatarAndreas Dilger <adilger@dilger.ca>
Signed-off-by: default avatarJoel Becker <joel.becker@oracle.com>
parent 729963a1

fs/ext3/super.c

+2 −2
Original line number Original line Diff line number Diff line
@@ -1849,8 +1849,8 @@ static int ext3_fill_super (struct super_block *sb, void *data, int silent) 
		goto failed_mount;

		goto failed_mount;

	}

	}





	if (le32_to_cpu(es->s_blocks_count) >

	if (generic_check_addressable(sb->s_blocksize_bits,

		    (sector_t)(~0ULL) >> (sb->s_blocksize_bits - 9)) {

				      le32_to_cpu(es->s_blocks_count))) {

		ext3_msg(sb, KERN_ERR,

		ext3_msg(sb, KERN_ERR,

			"error: filesystem is too large to mount safely");

			"error: filesystem is too large to mount safely");

		if (sizeof(sector_t) < 8)

		if (sizeof(sector_t) < 8)

fs/ext4/super.c

+3 −5
Original line number Original line Diff line number Diff line
@@ -2831,15 +2831,13 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) 
	 * Test whether we have more sectors than will fit in sector_t,

	 * Test whether we have more sectors than will fit in sector_t,

	 * and whether the max offset is addressable by the page cache.

	 * and whether the max offset is addressable by the page cache.

	 */

	 */

	if ((ext4_blocks_count(es) >

	ret = generic_check_addressable(sb->s_blocksize_bits,

	     (sector_t)(~0ULL) >> (sb->s_blocksize_bits - 9)) ||

					ext4_blocks_count(es));

	    (ext4_blocks_count(es) >

	if (ret) {

	     (pgoff_t)(~0ULL) >> (PAGE_CACHE_SHIFT - sb->s_blocksize_bits))) {

		ext4_msg(sb, KERN_ERR, "filesystem"

		ext4_msg(sb, KERN_ERR, "filesystem"

			 " too large to mount safely on this system");

			 " too large to mount safely on this system");

		if (sizeof(sector_t) < 8)

		if (sizeof(sector_t) < 8)

			ext4_msg(sb, KERN_WARNING, "CONFIG_LBDAF not enabled");

			ext4_msg(sb, KERN_WARNING, "CONFIG_LBDAF not enabled");

		ret = -EFBIG;

		goto failed_mount;

		goto failed_mount;

	}

	}

fs/libfs.c

+29 −0
Original line number Original line Diff line number Diff line
@@ -913,6 +913,35 @@ int generic_file_fsync(struct file *file, int datasync) 
}

}

EXPORT_SYMBOL(generic_file_fsync);

EXPORT_SYMBOL(generic_file_fsync);





/**

 * generic_check_addressable - Check addressability of file system

 * @blocksize_bits:	log of file system block size

 * @num_blocks:		number of blocks in file system

 *

 * Determine whether a file system with @num_blocks blocks (and a

 * block size of 2**@blocksize_bits) is addressable by the sector_t

 * and page cache of the system.  Return 0 if so and -EFBIG otherwise.

 */

int generic_check_addressable(unsigned blocksize_bits, u64 num_blocks)

{

	u64 last_fs_block = num_blocks - 1;



	if (unlikely(num_blocks == 0))

		return 0;



	if ((blocksize_bits < 9) || (blocksize_bits > PAGE_CACHE_SHIFT))

		return -EINVAL;



	if ((last_fs_block >

	     (sector_t)(~0ULL) >> (blocksize_bits - 9)) ||

	    (last_fs_block >

	     (pgoff_t)(~0ULL) >> (PAGE_CACHE_SHIFT - blocksize_bits))) {

		return -EFBIG;

	}

	return 0;

}

EXPORT_SYMBOL(generic_check_addressable);



/*

/*

 * No-op implementation of ->fsync for in-memory filesystems.

 * No-op implementation of ->fsync for in-memory filesystems.

 */

 */

include/linux/fs.h

+2 −0
Original line number Original line Diff line number Diff line
@@ -2374,6 +2374,8 @@ extern ssize_t simple_write_to_buffer(void *to, size_t available, loff_t *ppos, 




extern int generic_file_fsync(struct file *, int);

extern int generic_file_fsync(struct file *, int);





extern int generic_check_addressable(unsigned, u64);



#ifdef CONFIG_MIGRATION

#ifdef CONFIG_MIGRATION

extern int buffer_migrate_page(struct address_space *,

extern int buffer_migrate_page(struct address_space *,

				struct page *, struct page *);

				struct page *, struct page *);

CRA Git | Maintained and supported by SUSTech CRA and CCSE