From: Bob Peterson Date: Thu, 12 Sep 2019 17:54:27 +0000 (-0400) Subject: gfs2: clear buf_in_tr when ending a transaction in sweep_bh_for_rgrps X-Git-Tag: v4.19.77~30 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e0c1e6e55bca02aa3962ea1134e462197120964b;p=platform%2Fkernel%2Flinux-rpi.git gfs2: clear buf_in_tr when ending a transaction in sweep_bh_for_rgrps commit f0b444b349e33ae0d3dd93e25ca365482a5d17d4 upstream. In function sweep_bh_for_rgrps, which is a helper for punch_hole, it uses variable buf_in_tr to keep track of when it needs to commit pending block frees on a partial delete that overflows the transaction created for the delete. The problem is that the variable was initialized at the start of function sweep_bh_for_rgrps but it was never cleared, even when starting a new transaction. This patch reinitializes the variable when the transaction is ended, so the next transaction starts out with it cleared. Fixes: d552a2b9b33e ("GFS2: Non-recursive delete") Cc: stable@vger.kernel.org # v4.12+ Signed-off-by: Bob Peterson Signed-off-by: Andreas Gruenbacher Signed-off-by: Greg Kroah-Hartman --- diff --git a/fs/gfs2/bmap.c b/fs/gfs2/bmap.c index d14d71d..52fecce 100644 --- a/fs/gfs2/bmap.c +++ b/fs/gfs2/bmap.c @@ -1630,6 +1630,7 @@ out_unlock: brelse(dibh); up_write(&ip->i_rw_mutex); gfs2_trans_end(sdp); + buf_in_tr = false; } gfs2_glock_dq_uninit(rd_gh); cond_resched();