linux-ext4 - Re: + ext3-fix-online-resize-bug.patch added to -mm tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20080602172237.1bfdeb12.akpm@linux-foundation.org>
Date:	Mon, 2 Jun 2008 17:22:37 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	jbacik@...hat.com, adilger@....com, linux-ext4@...r.kernel.org
Subject: Re: + ext3-fix-online-resize-bug.patch added to -mm tree

On Mon, 02 Jun 2008 17:17:22 -0700
akpm@...ux-foundation.org wrote:

> 
> The patch titled
>      ext3: fix online resize bug
> has been added to the -mm tree.  Its filename is
>      ext3-fix-online-resize-bug.patch
> 
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
> 
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
> 
> See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
> out what to do about this
> 
> The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
> 
> ------------------------------------------------------
> Subject: ext3: fix online resize bug
> From: Josef Bacik <jbacik@...hat.com>
> 
> There is a bug when we are trying to verify that the reserve inode's
> double indirect blocks point back to the primary gdt blocks.  The fix is
> obvious, we need to mod the gdb count by the addr's per block.  You can
> verify this with the following test case
> 
> dd if=/dev/zero of=disk1 seek=1024 count=1 bs=100M
> losetup /dev/loop1 disk1
> pvcreate /dev/loop1
> vgcreate loopvg1 /dev/loop1
> lvcreate -l 100%VG loopvg1 -n looplv1
> mkfs.ext3 -J size=64 -b 1024 /dev/loopvg1/looplv1
> mount /dev/loopvg1/looplv1 /mnt/loop
> dd if=/dev/zero of=disk2 seek=1024 count=1 bs=50M
> losetup /dev/loop2 disk2
> pvcreate /dev/loop2
> vgextend loopvg1 /dev/loop2
> lvextend -l 100%VG /dev/loopvg1/looplv1
> resize2fs /dev/loopvg1/looplv1
> 
> without this patch the resize2fs fails, with it the resize2fs succeeds.
> 

Could I please gather some reviews and ackings on this?

Also, do we think it is 2.6.25.x material?

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html