Hi Peter:

  
> With recent change to e2fsprogs, overhead calculated in user-space increased
Do you know in which version it got changed?
  You can reference https://github.com/tytso/e2fsprogs/commit/59037c5357d39c6d0f14a0aff70e67dc13eafc84

I tested on my platform(arm/x86), found the

df -P -B 1k mntpoint | tail -n1 | awk '{print $2}'

the value is less than size 8*10

change to "7" can pass.

 

There's no strict rule how much the overhead will be and no abnormal for e2fsprogs test,So relax the condition to 70%

to pass the mkfs interface test.

 

Thanks!
----

 




主 题:Re: [LTP] [PATCH] mkfs: relax size check
日 期:2022-11-29 18:24
发件人:Petr Vorel
收件人:曾红玲;

Hi zenghongling,

> Number of total data blocks in filesystem reported by statfs
> may be less than current formula of 90%. For example ext4 will
> subtract "s_first_data_block plus internal journal blocks".

> With recent change to e2fsprogs, overhead calculated in user-space increased
Do you know in which version it got changed?
Maybe it'd be worth to mention that to ext4 maintainers,
just to be sure it's not a regression.

> slightly and LTP test started failing: tytso/e2fsprogs
A bit cryptic description :(. You mean probably https://github.com/tytso/e2fsprogs
which is also on
https://git.kernel.org/pub/scm/fs/ext2/e2fsprogs.git/

Kind regards,
Petr

> mkfs01 1 TPASS: 'mkfs -t ext4 /dev/loop0 ' passed.
> mkfs01 2 TFAIL: 'mkfs -t ext4 /dev/loop0 16000' failed, not expected.

> Since there's no strict rule how much the overhead will be,
> as rule of thumb relax the condition to 70%.

> Signed-off-by: zenghongling
> ---
> testcases/commands/mkfs/mkfs01.sh | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)

> diff --git a/testcases/commands/mkfs/mkfs01.sh b/testcases/commands/mkfs/mkfs01.sh
> index 263aa47..a964774 100755
> --- a/testcases/commands/mkfs/mkfs01.sh
> +++ b/testcases/commands/mkfs/mkfs01.sh
> @@ -66,11 +66,11 @@ mkfs_verify_size()
> # 1k-block size should be devided by this argument for ntfs verification.
> if [ "$1" = "ntfs" ]; then
> local rate=1024/512
> - if [ $blocknum -lt "$(($2/$rate*8/10))" ]; then
> + if [ $blocknum -lt "$(($2/$rate*7/10))" ]; then
> return 1
> fi
> else
> - if [ $blocknum -lt "$(($2*8/10))" ]; then
> + if [ $blocknum -lt "$(($2*7/10))" ]; then
> return 1
> fi
> fi