From: Bob Peterson Date: Fri, 30 Jul 2021 17:40:25 +0000 (-0500) Subject: gfs2: Mark journal inodes as "don't cache" X-Git-Tag: howlett/maple/20220722_2~2471^2~4 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=1b8550b5de7610027609ef605f85dc29f1d9da82;p=users%2Fjedix%2Flinux-maple.git gfs2: Mark journal inodes as "don't cache" Before this patch, journal inodes were considered regular inodes, which meant that instead of evicting them, function iput_final would just put them on the lru for later processing. If the file system withdrew for whatever reason, the withdraw would never be seen until the inode was evicted, which could be indefinitely. This patch marks all journal inodes as "don't cache" which means function iput_final will evict them immediately, allowing us to properly recover the journal on other cluster nodes. Signed-off-by: Bob Peterson --- diff --git a/fs/gfs2/ops_fstype.c b/fs/gfs2/ops_fstype.c index 5b90d2b7db470..7f8410d8fdc1d 100644 --- a/fs/gfs2/ops_fstype.c +++ b/fs/gfs2/ops_fstype.c @@ -614,6 +614,7 @@ static int gfs2_jindex_hold(struct gfs2_sbd *sdp, struct gfs2_holder *ji_gh) break; } + d_mark_dontcache(jd->jd_inode); spin_lock(&sdp->sd_jindex_spin); jd->jd_jid = sdp->sd_journals++; jip = GFS2_I(jd->jd_inode); diff --git a/fs/gfs2/util.c b/fs/gfs2/util.c index 34087bba88ee5..cf345a86ef67b 100644 --- a/fs/gfs2/util.c +++ b/fs/gfs2/util.c @@ -278,6 +278,7 @@ static void signal_our_withdraw(struct gfs2_sbd *sdp) goto skip_recovery; } sdp->sd_jdesc->jd_inode = inode; + d_mark_dontcache(inode); /* * Now wait until recovery is complete.