Message-Id: <20180615203513.5729-1-artem.blagodarenko@gmail.com>
Date:   Fri, 15 Jun 2018 23:35:13 +0300
From:   Artem Blagodarenko <artem.blagodarenko@...il.com>
To:     linux-ext4@...r.kernel.org
Cc:     adilger.kernel@...ger.ca, alexey.lyashkov@...il.com,
        Andreas Dilger <andreas.dilger@...el.com>,
        Kalpak Shah <kalpak@...sterfs.com>
Subject: [PATCH] e2fsck: handle preallocation for large PAGE_SIZE

From: Andreas Dilger <andreas.dilger@...el.com>

Fix handling of block preallocation support in cases where the kernel
PAGE_SIZE is larger than the filesystem blocksize.

Signed-off-by: Kalpak Shah <kalpak@...sterfs.com>
Signed-off-by: Andreas Dilger <andreas.dilger@...el.com>
---
 e2fsck/pass1.c             |  11 ++++++++---
 tests/f_eofblocks/expect.1 |   6 ++----
 tests/f_eofblocks/expect.2 |   2 +-
 tests/f_eofblocks/image.gz | Bin 1364 -> 1372 bytes
 4 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/e2fsck/pass1.c b/e2fsck/pass1.c
index f6623a6f..1d60cc88 100644
--- a/e2fsck/pass1.c
+++ b/e2fsck/pass1.c
@@ -3443,10 +3443,15 @@ static void check_blocks(e2fsck_t ctx, struct problem_context *pctx,
 
 		size = EXT2_I_SIZE(inode);
 		if ((pb.last_init_lblock >= 0) &&
-		    /* allow allocated blocks to end of PAGE_SIZE */
+		    /* if size is smaller than expected by the block count,
+		     * allow allocated blocks to end of PAGE_SIZE.
+		     * last_init_lblock is the last in-use block, so it is
+		     * the minimum expected file size, but +1 because it is
+		     * the base-zero block number and not the block count. */
 		    (size < (__u64)pb.last_init_lblock * fs->blocksize) &&
-		    (pb.last_init_lblock / blkpg * blkpg != pb.last_init_lblock ||
-		     size < (__u64)(pb.last_init_lblock & ~(blkpg-1)) *
+		    ((pb.last_init_lblock + 1) / blkpg * blkpg !=
+		     (pb.last_init_lblock + 1) ||
+		     size < (__u64)(pb.last_init_lblock & ~(blkpg - 1)) *
 		     fs->blocksize))
 			bad_size = 3;
 		else if (!(extent_fs && (inode->i_flags & EXT4_EXTENTS_FL)) &&
diff --git a/tests/f_eofblocks/expect.1 b/tests/f_eofblocks/expect.1
index 2e9133e1..34222480 100644
--- a/tests/f_eofblocks/expect.1
+++ b/tests/f_eofblocks/expect.1
@@ -1,7 +1,5 @@
 Pass 1: Checking inodes, blocks, and sizes
-Inode 30, i_size is 2048, should be 4096.  Fix? yes
-
-Inode 31, i_size is 2048, should be 4096.  Fix? yes
+Inode 31, i_size is 2048, should be 6144.  Fix? yes
 
 Pass 2: Checking directory structure
 Pass 3: Checking directory connectivity
@@ -9,5 +7,5 @@ Pass 4: Checking reference counts
 Pass 5: Checking group summary information
 
 test_filesys: ***** FILE SYSTEM WAS MODIFIED *****
-test_filesys: 31/56 files (29.0% non-contiguous), 83/400 blocks
+test_filesys: 31/56 files (29.0% non-contiguous), 85/400 blocks
 Exit status is 1
diff --git a/tests/f_eofblocks/expect.2 b/tests/f_eofblocks/expect.2
index 2a2bca5c..0cce3148 100644
--- a/tests/f_eofblocks/expect.2
+++ b/tests/f_eofblocks/expect.2
@@ -3,5 +3,5 @@ Pass 2: Checking directory structure
 Pass 3: Checking directory connectivity
 Pass 4: Checking reference counts
 Pass 5: Checking group summary information
-test_filesys: 31/56 files (29.0% non-contiguous), 83/400 blocks
+test_filesys: 31/56 files (29.0% non-contiguous), 85/400 blocks
 Exit status is 0
diff --git a/tests/f_eofblocks/image.gz b/tests/f_eofblocks/image.gz
index 2586a6a7670ad609ba7e66911a8f7a8f4df96321..e1177b3a7d6f33ce135fe91c7b6d11bbb1ee09a3 100644
GIT binary patch
literal 1372
zcmeH^Z!p^j7{_gHWq)pV>s%4!*j5`dL&d)l)1f=FR9$4$E~jY_5rXTqM1E?z%@ET~
zC?TU&hAs^g6(yT0>Cr@5;w~-emPlNYREYQ`zdy5=z1!Qp+xNwD_x*nEp6BlQV(Xlp
zkH0oee9@...+94bY^|hxN20koJu5YR=A+da`a+}fKmJVwRX@...&LE@...3j+<Du6
z^6S78&drIw@...SC;FSS>-N&$c^OyzN_PS#JfUvk)De&33i*U#!OI}OQnoO~*SK^>
zx|k-kz-p+W($M*W>8$&QY0UcaaUQL**psV`^JHuhZ%vTEoz?LWyrjpvn-{}4<{eFD
z#F`qDsaMz1<`nH-=dSIKCOysAEAiBz1<$%7bFtvwLn7xqGD|L>yk*<?Vz>zXgCR({
zVOCNE`}*w`d_A9Iu~Lkj{f?0$ox+IoX7JHm@...!VPAB9Z*n6hj)T)PP`TcQl|8-R
z#JjkueM$a?!(+C`pNg4FiP|h2e7LGTe&T{l)_{;p9S|;)g@...th5-5;q}`r)>5Zq
zkWsTFA#-OL(W)(ZVSwF|f$v=xWVO&r=VE6JuCUxqK-mkP{CU0X?*7yeMI&*!XI}}P
z?AxTWi<PF@...NdEu4TZN>nCXTV;I^&NdM|Ie0*Ct_!%^YX>yod*q56#T^FAcQ@dI
zC;D)EzXcEtj35`?ca!6Ow>b?0|2CcgU32d{RJr#k^E~B{kki_<S>^DYSXw$Ci@|M3
z=V4~4ij9f{A2c)|>@m`fBuN|47%`VaJH9P4#4{g1`e~`VaGR<1@...ns=g{C-bNV9
zkof$o{8h}V0Yrwy4-vA$RtNiS+(dU1SIWK2gpbr})F_F-M|n15XDPsNTg8`(OST%g
zD7ARd>)udBKuEb%T~^Q!HOW>y!a{t2N7q1@30g@...ufj_t<#1m`Fu1YIAtHPLU$h
zV8#@...BiyliF+&$^_vgPS%-keXXp;F8Vw`KaHGBK%g3k+GNjM6dfcPrxDgtZ0iqz
ze~Gj{Dy#u?5w8UaWt4;DXo-{?71{t27I@...ulpJdP>Jzx`1e{0%$~eeL5y5H6B@$
z;RbdfKRuzfRT_~E!>(hr9yvfegIunG5_P=hD3nwS!f9)YxuMa&ET-{PP&6+f6~hLw
zy^b35tF*%3?DVF&G0EtZI_sw^YCnc@...?>F!`{V_(@?#hk<}VWl4J8^o}&p2R(bR
zF$87b4+>NP<lQ3GQ3aOoPhcPU=O}8)R6{L(IsiLHJJ*R$Lt(WBAYlqA#T)IPXVFOy
zpsU_EIY1w>io9%jv)6wy@7U!(k~2wqYx+C^*cuHhM(((R&sG1w`ENb|MI(n&#=kK+
I#2s?@...BH;s5{u

delta 1029
zcmV+g1p52j3e*aJABzYGwEKil0t0DnVP|Ck?cGg?Rb?0l@N@4>A*1;@>Wq$B5N#~K
zpfpXwDN;z7McWL_kL*JYvLGWIX(WQ87VV-)s+C|wdn<~B7C{BkqK%uPMP#W9jkrDc
zo^$AqbLY%^(ZZbH51u&>?>*yx-s5@...R%u%Yf->L)068wJht8<K{RH#<A9Z!p(U#
z&wJ<6*vLmw^{ml&=(k$yygx3Cjg+UBpEomVwD_}!>sj`E9N+z9=>FMf9{lFx55Ir<
zhZp-kyZq}|z5l?WE1ylR^3`7+`TF7K)(?O2((gat{=l1SUTvN|t9Y$(_MdrNO`X1?
z-l$z}J+1+N&3m!A_HS*i|G})@8p#c2+uw490O0!52!A%tH5xB;@d?0nq2781uGLq?
zG4Jd3Rjcx6`2MV~{?x%|4z1jK@...q^JnFFy|#1hp6sTwUzcUGi~iBISvIHaV|;GW
z?~3txWgp}7i+*>EFDUzU`Ey9K-xK2t%Ra^z75(0S7++lWF+NcAkFCvrZ_(;wd`Z#o
zi}9spALBzs|9FfqEBhE<Ui44I_=@Py)}vqNdjr?MnU}5q^F8_c2d2?40RihDu>Jw-
zAF%!b>mRWG0qY;I{sHSBu>OJng&vmx0H&2jqj4njuliwHvI7DD000000000000000
zz_q7;advE^tp76i*DYD={l|X1FgEhj#k}%YdRqS<ahA=<Hpa)kbG31AQ&x+*!CxC!
zX!K}&Skv1(@&2z}@...fcX_XQpEmcOHJ$dqdNxq(KUs(UXI9jWS0VrXqxt@...fAP
zO;OF``1@...p?4&SD&!|Ei8_;(JZ^OB4gHnm-~-Rl|LN0Tcd_2<Zqiaf1oY@...EG
z3HduBcX!l|F6O_!Eq_Br{+xvTeUbZQ)V?m}KiHOkcSZi(sq*6%I~=)3q7HX4|0Qkt
z8!PhXCFCEA+?S({bus^?ZTXw3_Q8bwQ<3|5)Tu7!Kh&1LtRjCwLjLK<{VM8o7xTY=
zr!9YZMSg!m{)NabtMmNF??26-jW#FlF$)v&FGg-zo##I>|K7y>MG5QgugEPcb^a?7
z^A{)NHzK#J)cLPW%paI4e{Pn&9zXxS9o0OR`7<i+|7QM{;#iOS?~aQ6!G!$dk^5fM
z@...cCe6R4Eq_r({*r|J6Omh1=lPFozy7_Ed0*o7Uz(7AGIGmGnSXQ5FDTZ3PxP<V
zCeI(5DnEQ1xn-r!|CL(W{Ff!<e;2uBb)Nsk{MCuqe|f_Gb0(hqPF%_>Y5q~$i|b#S
z?0sxEez0@...b*4&Xbq&N}7Mv*2MX5s>lbk@...0k^}_|;NS5#ODu1?06+!+NxVE=

-- 
2.14.3

Powered by blists - more mailing lists