]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
mds: revert standby-replay trimming changes
authorPatrick Donnelly <pdonnell@redhat.com>
Thu, 10 Nov 2022 13:22:35 +0000 (08:22 -0500)
committerPatrick Donnelly <pdonnell@redhat.com>
Wed, 29 Nov 2023 13:36:21 +0000 (08:36 -0500)
commit435747b250c928f30a39e1dfa66eccdf5d213a3c
tree37278105a6ea7f4da0a0de2855567b22868eba28
parentb4a94af5441a68588ad0a5541fba95e42d1d7241
mds: revert standby-replay trimming changes

Revert "mds: do not trim the inodes from the lru list in standby_replay"
Revert "mds: trim cache during standby replay"
This reverts commit 79bb44c1b9f1715378a9550a81984e949e454ff4.
This reverts commit c0fe25bb2a87856c1281eddcb4da2efe0d7fbf75.

standby-replay daemons were changed to keep minimal metadata from the journal
in cache but the original intent of standby-replay was to have a cache that's
as warm as the rank itself.  This reverts the two commits which changed that
behavior.

Part of these reason for this is that the new rapid cache trimming behavior was
not correct at all. The trimming loop would break when it runs into a dentry
with non-null linkage. This would nearly always be the case.  It was thought
that this was a problem introduced by [2] as MDCache::standby_trim_segment has
a different trim check [4] but the original issue (tracker 48673) is as old as
[1], indicating the problem predates [2].

So, this commit reverts all of that. I have lingering suspicions that the
standby-replay daemon is not pinning some dentries properly which causes [5]
but this did not show up unless the MDS was rapidly evicting some dentries.
More research needs done there.

[1] c0fe25bb2a87856c1281eddcb4da2efe0d7fbf75
[2] 79bb44c1b9f1715378a9550a81984e949e454ff4
[3] https://github.com/ceph/ceph/blob/84fba097049ec4f72549588eaacc64f30c7a88a8/src/mds/MDCache.cc#L6816-L6820
[4] https://github.com/ceph/ceph/blob/84fba097049ec4f72549588eaacc64f30c7a88a8/src/mds/MDCache.cc#L7476-L7481
[5] https://tracker.ceph.com/issues/50246

Fixes: https://tracker.ceph.com/issues/48673
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 589e59af11e40164695ca13f4ce4f2bc140b18b8)
src/mds/MDCache.cc