Hi all,
Historically, when users ran out of space or quota when trying to write
to the filesystem, XFS didn't try very hard to reclaim space that it
might have speculatively allocated for the purpose of speeding up
front-end filesystem operations (appending writes, cow staging). The
upcoming deferred inactivation series will greatly increase the amount
of allocated space that isn't actively being used to store user data.
Therefore, try to reduce the circumstances where we return EDQUOT or
ENOSPC to userspace by teaching the write paths to try to clear space
and retry the operation one time before giving up.
Previous iterations of this patchset made massive changes to the
codebase, but thanks to the transaction allocation helpers that Brian
pushed for (in the previous patchset) this is mostly no longer
necessary. :)
v2: clean up and rebase against 5.11.
v3: restructure the retry loops per dchinner suggestion
v4: simplify the calling convention of xfs_trans_reserve_quota_nblks
v5: constrain the open-coded 'goto retry' loops to the helpers created
in the previous patchset
v6: move "xfs: try worst case space reservation upfront in
xfs_reflink_remap_extent" to this series, open-code the qretry
helpers in the (now very few) places they are used
v7: rebase chown stuff to use new helpers introduced in quota cleanups
If you're going to start using this mess, you probably ought to just
pull from my git trees, which are linked below.
This is an extraordinary way to destroy everything. Enjoy!
Comments and questions are, as always, welcome.
--D
kernel git tree:
https://git.kernel.org/cgit/linux/kernel/git/djwong/xfs-linux.git/log/?h=reclaim-space-harder-5.12
---
fs/xfs/xfs_file.c | 24 +++----
fs/xfs/xfs_icache.c | 184 +++++++++++++++++++++++++++++++++-----------------
fs/xfs/xfs_icache.h | 7 +-
fs/xfs/xfs_ioctl.c | 2 +
fs/xfs/xfs_reflink.c | 28 +++++++-
fs/xfs/xfs_trace.c | 1
fs/xfs/xfs_trace.h | 42 +++++++++++
fs/xfs/xfs_trans.c | 38 ++++++++++
8 files changed, 244 insertions(+), 82 deletions(-)
