From: Li Dongyang Date: Thu, 20 Jul 2023 12:50:12 +0000 (+1000) Subject: ext2fs: make sure we have at least EXT2_FIRST_INO + 1 inodes X-Git-Tag: v1.47.1-rc1~72^2~3 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=196cd1224ccaf5ca76540fc0a6238695d4476ca9;p=thirdparty%2Fe2fsprogs.git ext2fs: make sure we have at least EXT2_FIRST_INO + 1 inodes When creating a small fs with 100 1k blocks, mke2fs fails with: Creating filesystem with 100 1k blocks and 8 inodes Allocating group tables: done Writing inode tables: done ext2fs_mkdir: Could not allocate inode in ext2 filesystem while creating /lost+found Increase s_inodes_per_group with a step of 8 to make sure we have at least EXT2_FIRST_INO + 1 inodes. Change-Id: Ib885735641dfa0ed9c6f6a4a1f9afec291673126 Signed-off-by: Li Dongyang Reviewed-by: Andreas Dilger Link: https://lore.kernel.org/r/20230720125012.641504-1-dongyangli@ddn.com Signed-off-by: Theodore Ts'o --- diff --git a/lib/ext2fs/initialize.c b/lib/ext2fs/initialize.c index edd692bb..e96f3cab 100644 --- a/lib/ext2fs/initialize.c +++ b/lib/ext2fs/initialize.c @@ -307,13 +307,6 @@ retry: else set_field(s_inodes_count, ext2fs_blocks_count(super) / i); - /* - * Make sure we have at least EXT2_FIRST_INO + 1 inodes, so - * that we have enough inodes for the filesystem(!) - */ - if (super->s_inodes_count < EXT2_FIRST_INODE(super)+1) - super->s_inodes_count = EXT2_FIRST_INODE(super)+1; - /* * There should be at least as many inodes as the user * requested. Figure out how many inodes per group that @@ -375,6 +368,15 @@ ipg_retry: } super->s_inodes_count = super->s_inodes_per_group * fs->group_desc_count; + /* + * Make sure we have at least EXT2_FIRST_INO + 1 inodes, so + * that we have enough inodes for the filesystem(!) + */ + if (super->s_inodes_count < EXT2_FIRST_INODE(super)+1) { + ipg += 8; + goto ipg_retry; + } + super->s_free_inodes_count = super->s_inodes_count; /*