xfs: don't deplete the reserve pool when trying to shrink the fs
authorDarrick J. Wong <djwong@kernel.org>
Tue, 13 Jun 2023 01:09:04 +0000 (18:09 -0700)
committerDarrick J. Wong <djwong@kernel.org>
Tue, 13 Jun 2023 01:09:04 +0000 (18:09 -0700)
commit06f3ef6e1705612b88aa0b6991e2ac3b8ed3f8ec
tree1adcc43460eed64372604661834d300aa560f496
parent858fd168a95c5b9669aac8db6c14a9aeab446375
xfs: don't deplete the reserve pool when trying to shrink the fs

Every now and then, xfs/168 fails with this logged in dmesg:

Reserve blocks depleted! Consider increasing reserve pool size.
EXPERIMENTAL online shrink feature in use. Use at your own risk!
Per-AG reservation for AG 1 failed.  Filesystem may run out of space.
Per-AG reservation for AG 1 failed.  Filesystem may run out of space.
Error -28 reserving per-AG metadata reserve pool.
Corruption of in-memory data (0x8) detected at xfs_ag_shrink_space+0x23c/0x3b0 [xfs] (fs/xfs/libxfs/xfs_ag.c:1007).  Shutting down filesystem.

It's silly to deplete the reserved blocks pool just to shrink the
filesystem, particularly since the fs goes down after that.

Fixes: fb2fc1720185 ("xfs: support shrinking unused space in the last AG")
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
fs/xfs/xfs_fsops.c