diff options
| author |   Darrick J. Wong <djwong@kernel.org> | 2021-06-07 09:34:50 -0700 |
|---|---|---|
| committer | Darrick J. Wong <djwong@kernel.org> | 2021-06-08 09:30:20 -0700 |
| commit | 7975e465af6b46e9d0eaf94f764922dc92b28d9c (patch) | |
| tree | d1e710e2217d64fc9ec0e39605f80854ccb1295f /fs/xfs/xfs_health.c | |
| parent | xfs: only reset incore inode health state flags when reclaiming an inode (diff) | |
| download | linux-dev-7975e465af6b46e9d0eaf94f764922dc92b28d9c.tar.xz linux-dev-7975e465af6b46e9d0eaf94f764922dc92b28d9c.zip | |
xfs: drop IDONTCACHE on inodes when we mark them sick
When we decide to mark an inode sick, clear the DONTCACHE flag so that
the incore inode will be kept around until memory pressure forces it out
of memory. This increases the chances that the sick status will be
caught by someone compiling a health report later on.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Diffstat (limited to 'fs/xfs/xfs_health.c')
| -rw-r--r-- | fs/xfs/xfs_health.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/fs/xfs/xfs_health.c b/fs/xfs/xfs_health.c index 5de3195f6cb2..eb10eacabc8f 100644 --- a/fs/xfs/xfs_health.c +++ b/fs/xfs/xfs_health.c @@ -229,6 +229,15 @@ xfs_inode_mark_sick( ip->i_sick |= mask; ip->i_checked |= mask; spin_unlock(&ip->i_flags_lock); + + /* + * Keep this inode around so we don't lose the sickness report. Scrub + * grabs inodes with DONTCACHE assuming that most inode are ok, which + * is not the case here. + */ + spin_lock(&VFS_I(ip)->i_lock); + VFS_I(ip)->i_state &= ~I_DONTCACHE; + spin_unlock(&VFS_I(ip)->i_lock); } /* Mark parts of an inode healed. */ |