2008-11-06 15:00:54

by Christian Borntraeger

[permalink] [raw]
Subject: EXT4-fs: not enough memory for 522250 flex groups

Hello,

I tried to use ext4 on my s390 system. Converting the old ext3 file systems
(tune2fs -O extents) worked fine. When I tried to create a new ext4 file
system on a logical volume I got the "EXT4-fs: not enough memory for 522250
flex groups" error messageduring mount:


# mke2fs -t ext4 /dev/mapper/space-test
mke2fs 1.41.3 (12-Oct-2008)
Filesystem label=
OS type: Linux
Block size=4096 (log=2)
Fragment size=4096 (log=2)
262144 inodes, 1048576 blocks
52428 blocks (5.00%) reserved for the super user
First data block=0
Maximum filesystem blocks=1073741824
32 block groups
32768 blocks per group, 32768 fragments per group
8192 inodes per group
Superblock backups stored on blocks:
32768, 98304, 163840, 229376, 294912, 819200, 884736
Writing inode tables: done
Creating journal (32768 blocks): done
Writing superblocks and filesystem accounting information: done
This filesystem will be automatically checked every 22 mounts or
180 days, whichever comes first. Use tune2fs -c or -i to override.


# mount -t ext4 /dev/mapper/space-test /mnt/
mount: wrong fs type, bad option, bad superblock on /dev/mapper/space-test,
missing codepage or helper program, or other error
In some cases useful info is found in syslog - try
dmesg | tail or so



# dmesg
[...]
[6923911.715968] EXT4-fs: not enough memory for 522250 flex groups
[6923911.715973] EXT4-fs: unable to initialize flex_bg meta info!



# lvdisplay
--- Logical volume ---
LV Name /dev/space/test
VG Name space
LV UUID FPhFLU-1xTK-XtG4-STmI-u3D5-KhCR-FmlcCO
LV Write Access read/write
LV Status available
# open 0
LV Size 4.00 GB
Current LE 1024
Segments 1
Allocation inherit
Read ahead sectors auto
- currently set to 256
Block device 253:54



# free
total used free shared buffers cached
Mem: 3081168 822972 2258196 0 154784 445420
-/+ buffers/cache: 222768 2858400
Swap: 9615904 0 9615904


# uname -a
Linux 2.6.28-rc2 #64 SMP Thu Nov 6 15:22:04 CET 2008 s390x s390x s390x
GNU/Linux



# git describe
kvm-78-2-g6e6c459


Any ideas?


2008-11-06 15:54:00

by Christian Borntraeger

[permalink] [raw]
Subject: [PATCH]: ext4: fix big endian (was: not enough memory for 522250 flex groups)

Ok, I think I found it:

On big endianess plattforms newly created ext4 file systems cannot be mounted
and show messages like:
[6923911.715968] EXT4-fs: not enough memory for 522250 flex groups
[6923911.715973] EXT4-fs: unable to initialize flex_bg meta info!

We have to access s_reserved_gdb_blocks with le16_to_cpu.

Signed-off-by: Christian Borntraeger <[email protected]>
---
fs/ext4/super.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Index: kvm/fs/ext4/super.c
===================================================================
--- kvm.orig/fs/ext4/super.c
+++ kvm/fs/ext4/super.c
@@ -1455,7 +1455,7 @@ static int ext4_fill_flex_info(struct su

/* We allocate both existing and potentially added groups */
flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
- ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
+ (le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks + 1) <<
EXT4_DESC_PER_BLOCK_BITS(sb))) /
groups_per_flex;
sbi->s_flex_groups = kzalloc(flex_group_count *

2008-11-06 16:06:57

by Andreas Schwab

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian (was: not enough memory for 522250 flex groups)

Christian Borntraeger <[email protected]> writes:

> Index: kvm/fs/ext4/super.c
> ===================================================================
> --- kvm.orig/fs/ext4/super.c
> +++ kvm/fs/ext4/super.c
> @@ -1455,7 +1455,7 @@ static int ext4_fill_flex_info(struct su
>
> /* We allocate both existing and potentially added groups */
> flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
> - ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
> + (le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks + 1) <<

You need to put the "+ 1" after the swap.

Andreas.

--
Andreas Schwab, SuSE Labs, [email protected]
SuSE Linux Products GmbH, Maxfeldstra?e 5, 90409 N?rnberg, Germany
PGP key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
"And now for something completely different."

2008-11-06 16:06:25

by Peter Staubach

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

Christian Borntraeger wrote:
> Ok, I think I found it:
>
> On big endianess plattforms newly created ext4 file systems cannot be mounted
> and show messages like:
> [6923911.715968] EXT4-fs: not enough memory for 522250 flex groups
> [6923911.715973] EXT4-fs: unable to initialize flex_bg meta info!
>
> We have to access s_reserved_gdb_blocks with le16_to_cpu.
>
> Signed-off-by: Christian Borntraeger <[email protected]>
> ---
> fs/ext4/super.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Index: kvm/fs/ext4/super.c
> ===================================================================
> --- kvm.orig/fs/ext4/super.c
> +++ kvm/fs/ext4/super.c
> @@ -1455,7 +1455,7 @@ static int ext4_fill_flex_info(struct su
>
> /* We allocate both existing and potentially added groups */
> flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
> - ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
> + (le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks + 1) <<
>

I suspect that you want to do the le16_to_cpu() and _then_
add the 1. Otherwise, adding 1 to a different byte order
value won't do quite what is expected or hoped for...

ps

> EXT4_DESC_PER_BLOCK_BITS(sb))) /
> groups_per_flex;
> sbi->s_flex_groups = kzalloc(flex_group_count *
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>


2008-11-06 16:17:03

by Christian Borntraeger

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

Am Donnerstag, 6. November 2008 schrieb Peter Staubach:
> > /* We allocate both existing and potentially added groups */
> > flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
> > - ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
> > + (le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks + 1) <<
> >
>
> I suspect that you want to do the le16_to_cpu() and _then_
> add the 1. Otherwise, adding 1 to a different byte order
> value won't do quite what is expected or hoped for...

errm. Right.
Something like this maybe?

[PATCH]: ext4: fix big endian access for flex groups

On big endianess plattforms newly created ext4 file systems cannot be mounted
and show messages like:
[6923911.715968] EXT4-fs: not enough memory for 522250 flex groups
[6923911.715973] EXT4-fs: unable to initialize flex_bg meta info!

We have to access s_reserved_gdb_blocks with le16_to_cpu.

Signed-off-by: Christian Borntraeger <[email protected]>
---
fs/ext4/super.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

Index: kvm/fs/ext4/super.c
===================================================================
--- kvm.orig/fs/ext4/super.c
+++ kvm/fs/ext4/super.c
@@ -1455,8 +1455,8 @@ static int ext4_fill_flex_info(struct su

/* We allocate both existing and potentially added groups */
flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
- ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
- EXT4_DESC_PER_BLOCK_BITS(sb))) /
+ ((le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks) +
+ 1) << EXT4_DESC_PER_BLOCK_BITS(sb))) /
groups_per_flex;
sbi->s_flex_groups = kzalloc(flex_group_count *
sizeof(struct flex_groups), GFP_KERNEL);

2008-11-06 16:33:15

by Aneesh Kumar K.V

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Thu, Nov 06, 2008 at 05:15:53PM +0100, Christian Borntraeger wrote:
> Am Donnerstag, 6. November 2008 schrieb Peter Staubach:
> > > /* We allocate both existing and potentially added groups */
> > > flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
> > > - ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
> > > + (le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks + 1) <<
> > >
> >
> > I suspect that you want to do the le16_to_cpu() and _then_
> > add the 1. Otherwise, adding 1 to a different byte order
> > value won't do quite what is expected or hoped for...
>
> errm. Right.
> Something like this maybe?
>
> [PATCH]: ext4: fix big endian access for flex groups
>
> On big endianess plattforms newly created ext4 file systems cannot be mounted
> and show messages like:
> [6923911.715968] EXT4-fs: not enough memory for 522250 flex groups
> [6923911.715973] EXT4-fs: unable to initialize flex_bg meta info!
>
> We have to access s_reserved_gdb_blocks with le16_to_cpu.
>
> Signed-off-by: Christian Borntraeger <[email protected]>

There is already a patch in the patchqueue that fix this
http://repo.or.cz/w/ext4-patch-queue.git?a=blob;f=convert-to-host-order-in-ext4_fill_flex_info;h=ebe7efb3ee78b6cab4db49391f4a005d78a26f03;hb=HEAD


-aneesh

2008-11-06 16:38:46

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Thu, Nov 06, 2008 at 11:06:25AM -0500, Peter Staubach wrote:
>> Index: kvm/fs/ext4/super.c
>> ===================================================================
>> --- kvm.orig/fs/ext4/super.c
>> +++ kvm/fs/ext4/super.c
>> @@ -1455,7 +1455,7 @@ static int ext4_fill_flex_info(struct su
>> /* We allocate both existing and potentially added groups */
>> flex_group_count = ((sbi->s_groups_count + groups_per_flex - 1) +
>> - ((sbi->s_es->s_reserved_gdt_blocks +1 ) <<
>> + (le16_to_cpu(sbi->s_es->s_reserved_gdt_blocks + 1) <<
>>
>
> I suspect that you want to do the le16_to_cpu() and _then_
> add the 1. Otherwise, adding 1 to a different byte order
> value won't do quite what is expected or hoped for...

Yes. And if someone ran sparse over the code both the initial error
and this varaint would be trivial to spot..


2008-11-06 16:59:04

by Theodore Ts'o

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian (was: not enough memory for 522250 flex groups)

On Thu, Nov 06, 2008 at 04:51:49PM +0100, Christian Borntraeger wrote:
> Ok, I think I found it:
>
> On big endianess plattforms newly created ext4 file systems cannot be mounted
> and show messages like:
> [6923911.715968] EXT4-fs: not enough memory for 522250 flex groups
> [6923911.715973] EXT4-fs: unable to initialize flex_bg meta info!
>
> We have to access s_reserved_gdb_blocks with le16_to_cpu.

Aneesh has already submitted a patch to fix this; I'll be pushing it
to Linus shortly.

- Ted

2008-11-06 17:11:44

by Christian Borntraeger

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

Am Donnerstag, 6. November 2008 schrieb Christoph Hellwig:
> > I suspect that you want to do the le16_to_cpu() and _then_
> > add the 1. Otherwise, adding 1 to a different byte order
> > value won't do quite what is expected or hoped for...
>
> Yes. And if someone ran sparse over the code both the initial error
> and this varaint would be trivial to spot..

I think the problem is, that sparse now requires
__CHECK_ENDIAN__
to check for endianess problems. Seems that lots of people are not aware of
this.

Christian

2008-11-06 17:33:27

by Theodore Ts'o

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Thu, Nov 06, 2008 at 11:38:46AM -0500, Christoph Hellwig wrote:
>
> Yes. And if someone ran sparse over the code both the initial error
> and this varaint would be trivial to spot..
>

That's how Aneesh found it. :-)

- Ted

2008-11-06 17:39:56

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Thu, Nov 06, 2008 at 12:33:27PM -0500, Theodore Tso wrote:
> On Thu, Nov 06, 2008 at 11:38:46AM -0500, Christoph Hellwig wrote:
> >
> > Yes. And if someone ran sparse over the code both the initial error
> > and this varaint would be trivial to spot..
> >
>
> That's how Aneesh found it. :-)

Heh, okay. We really should try to find some annotations that run
sparse with endian checking by default for those parts of the tree
where it makes sense..


2008-11-06 17:48:05

by Alexey Dobriyan

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Thu, Nov 06, 2008 at 12:39:44PM -0500, Christoph Hellwig wrote:
> On Thu, Nov 06, 2008 at 12:33:27PM -0500, Theodore Tso wrote:
> > On Thu, Nov 06, 2008 at 11:38:46AM -0500, Christoph Hellwig wrote:
> > >
> > > Yes. And if someone ran sparse over the code both the initial error
> > > and this varaint would be trivial to spot..
> > >
> >
> > That's how Aneesh found it. :-)
>
> Heh, okay. We really should try to find some annotations that run
> sparse with endian checking by default for those parts of the tree
> where it makes sense..

In fact endian warnings were off by default, because there were too much of
them in networking stack and elsewhere, but situations definitely improved.
Out of head, only drivers/ieee1394/ is not done.

2008-11-08 22:36:09

by Jörn Engel

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Thu, 6 November 2008 12:39:44 -0500, Christoph Hellwig wrote:
>
> Heh, okay. We really should try to find some annotations that run
> sparse with endian checking by default for those parts of the tree
> where it makes sense..

#define __CHECK_ENDIAN__ in some ext4 header. Made life much easier for
me.

Jörn

--
A surrounded army must be given a way out.
-- Sun Tzu

2008-11-10 18:09:42

by Harvey Harrison

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Sat, 2008-11-08 at 23:35 +0100, Jörn Engel wrote:
> On Thu, 6 November 2008 12:39:44 -0500, Christoph Hellwig wrote:
> >
> > Heh, okay. We really should try to find some annotations that run
> > sparse with endian checking by default for those parts of the tree
> > where it makes sense..
>
> #define __CHECK_ENDIAN__ in some ext4 header. Made life much easier for
> me.
>

I'd say that it's getting close to just being able to turn it on by
default. A lot of the really verbose offenders have been annotated
now, drivers/ieee1394, drivers/scsi, drivers/message and some of the
older areas of drivers/net will get noisy, I was going to send an RFC
for 2.6.29 and continue to chip away at the output.

Harvey

2008-11-10 19:32:11

by Jörn Engel

[permalink] [raw]
Subject: Re: [PATCH]: ext4: fix big endian

On Mon, 10 November 2008 10:09:38 -0800, Harvey Harrison wrote:
>
> I'd say that it's getting close to just being able to turn it on by
> default. A lot of the really verbose offenders have been annotated
> now, drivers/ieee1394, drivers/scsi, drivers/message and some of the
> older areas of drivers/net will get noisy, I was going to send an RFC
> for 2.6.29 and continue to chip away at the output.

My argument for the define is that it doesn't hurt much if endian checks
are enabled globally, but helps a lot if they are not. So until it is
clear that endian checks will be enabled, it may still be a good idea.

Jörn

--
It's not whether you win or lose, it's how you place the blame.
-- unknown