Hi!
On Mon, 2011-06-13 at 00:46 +0200, Yury V. Zaytsev wrote:
> I have refreshed patch against the latest "next" branch since it no
> longer properly applied and hopefully it can now be included!
I would highly appreciate if the trivial patch that nonetheless fixes an
important issue to my mind posted last month could be looked at.
Is there anything else I could do to help with the inclusion? Sorry if I
am bumping the thread too fast, but I see other patches flowing in which
makes me think that I've done something wrong; if this is the case
please do let me know, I am willing to rectify it.
Thanks!
--
Sincerely yours,
Yury V. Zaytsev
> From 1f8e5ad235694f2918fb442135619d4988366434 Mon Sep 17 00:00:00 2001
> From: "Yury V. Zaytsev" <yury@xxxxxxxxxx>
> Date: Mon, 13 Jun 2011 00:35:05 +0200
> Subject: [PATCH] mke2fs: check that auto-detected blocksize <= sys_page_size
>
> Block size can be specified manually via the -b option or deduced
> automatically. Unfortunately, the check that it is still smaller than
> the system page size is only performed right after the command line
> options are parsed.
>
> Therefore, if buggy or inappropriately installed/configured hardware
> hints that larger block sizes have to be used, mkfs will silently create
> a file system which can not be mounted on the system in question.
>
> By moving the check beyond the last assignment to blocksize it is now
> ensured, that mkfs will issue a warning even if inappropriate blocksize
> was auto-detected.
>
> The new behavior can be easily tested, by exporting the following
> variables before running mkfs:
>
> export MKE2FS_DEVICE_SECTSIZE=8912
> export MKE2FS_DEVICE_PHYS_SECTSIZE=8912
>
> Signed-off-by: Yury V. Zaytsev <yury@xxxxxxxxxx>
> ---
> misc/mke2fs.c | 24 +++++++++++++-----------
> 1 files changed, 13 insertions(+), 11 deletions(-)
>
> diff --git a/misc/mke2fs.c b/misc/mke2fs.c
> index 5ff3f9f..d699b46 100644
> --- a/misc/mke2fs.c
> +++ b/misc/mke2fs.c
> @@ -1516,17 +1516,6 @@ profile_error:
> ext2fs_close(jfs);
> }
>
> - if (blocksize > sys_page_size) {
> - if (!force) {
> - com_err(program_name, 0,
> - _("%d-byte blocks too big for system (max %d)"),
> - blocksize, sys_page_size);
> - proceed_question();
> - }
> - fprintf(stderr, _("Warning: %d-byte blocks too big for system "
> - "(max %d), forced to continue\n"),
> - blocksize, sys_page_size);
> - }
> if (optind < argc) {
> fs_blocks_count = parse_num_blocks2(argv[optind++],
> fs_param.s_log_block_size);
> @@ -1825,6 +1814,19 @@ profile_error:
>
> blocksize = EXT2_BLOCK_SIZE(&fs_param);
>
> + /* This check should happen beyond the last assignment to blocksize */
> + if (blocksize > sys_page_size) {
> + if (!force) {
> + com_err(program_name, 0,
> + _("%d-byte blocks too big for system (max %d)"),
> + blocksize, sys_page_size);
> + proceed_question();
> + }
> + fprintf(stderr, _("Warning: %d-byte blocks too big for system "
> + "(max %d), forced to continue\n"),
> + blocksize, sys_page_size);
> + }
> +
> lazy_itable_init = 0;
> if (access("/sys/fs/ext4/features/lazy_itable_init", R_OK) == 0)
> lazy_itable_init = 1;
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
