]> git.baikalelectronics.ru Git - kernel.git/commit
xfs: revert "xfs: actually bump warning counts when we send warnings"
authorEric Sandeen <sandeen@redhat.com>
Tue, 26 Apr 2022 03:35:26 +0000 (13:35 +1000)
committerDave Chinner <david@fromorbit.com>
Tue, 26 Apr 2022 03:35:26 +0000 (13:35 +1000)
commit8710a9e275c22f0088833da7c4dfe6d0289e8129
tree0ffc0f36874bf9946530c062fca9c060cfbc0d74
parentbd541b8d4f92df224a87c283c486fea1366f4381
xfs: revert "xfs: actually bump warning counts when we send warnings"

This reverts commit 44992ec23fa5c676d0053a2cb49b9ef0b7f681a9.

XFS quota has had the concept of a "quota warning limit" since
the earliest Irix implementation, but a mechanism for incrementing
the warning counter was never implemented, as documented in the
xfs_quota(8) man page. We do know from the historical archive that
it was never incremented at runtime during quota reservation
operations.

With this commit, the warning counter quickly increments for every
allocation attempt after the user has crossed a quote soft
limit threshold, and this in turn transitions the user to hard
quota failures, rendering soft quota thresholds and timers useless.
This was reported as a regression by users.

Because the intended behavior of this warning counter has never been
understood or documented, and the result of this change is a regression
in soft quota functionality, revert this commit to make soft quota
limits and timers operable again.

Fixes: 44992ec23fa5 ("xfs: actually bump warning counts when we send warnings)
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
fs/xfs/xfs_trans_dquot.c