diff options
author | Darrick J. Wong <djwong@kernel.org> | 2023-09-11 08:39:08 -0700 |
---|---|---|
committer | Darrick J. Wong <djwong@kernel.org> | 2023-09-12 10:31:07 -0700 |
commit | 49813a21ed57895b73ec4ed3b99d4beec931496f (patch) | |
tree | 7a4af498dc4a10ce1245d6e8937be75fe6c9fd31 /fs/xfs/xfs_qm.c | |
parent | 83771c50e42b92de6740a63e152c96c052d37736 (diff) | |
download | linux-49813a21ed57895b73ec4ed3b99d4beec931496f.tar.gz linux-49813a21ed57895b73ec4ed3b99d4beec931496f.tar.bz2 linux-49813a21ed57895b73ec4ed3b99d4beec931496f.zip |
xfs: make inode unlinked bucket recovery work with quotacheck
Teach quotacheck to reload the unlinked inode lists when walking the
inode table. This requires extra state handling, since it's possible
that a reloaded inode will get inactivated before quotacheck tries to
scan it; in this case, we need to ensure that the reloaded inode does
not have dquots attached when it is freed.
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Diffstat (limited to 'fs/xfs/xfs_qm.c')
-rw-r--r-- | fs/xfs/xfs_qm.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/fs/xfs/xfs_qm.c b/fs/xfs/xfs_qm.c index 6abcc34fafd8..7256090c3895 100644 --- a/fs/xfs/xfs_qm.c +++ b/fs/xfs/xfs_qm.c @@ -1160,6 +1160,10 @@ xfs_qm_dqusage_adjust( if (error) return error; + error = xfs_inode_reload_unlinked(ip); + if (error) + goto error0; + ASSERT(ip->i_delayed_blks == 0); if (XFS_IS_REALTIME_INODE(ip)) { @@ -1173,6 +1177,7 @@ xfs_qm_dqusage_adjust( } nblks = (xfs_qcnt_t)ip->i_nblocks - rtblks; + xfs_iflags_clear(ip, XFS_IQUOTAUNCHECKED); /* * Add the (disk blocks and inode) resources occupied by this @@ -1319,8 +1324,10 @@ xfs_qm_quotacheck( flags |= XFS_PQUOTA_CHKD; } + xfs_set_quotacheck_running(mp); error = xfs_iwalk_threaded(mp, 0, 0, xfs_qm_dqusage_adjust, 0, true, NULL); + xfs_clear_quotacheck_running(mp); /* * On error, the inode walk may have partially populated the dquot |