New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
jewel: cephfs: Journaler may execute on_safe contexts prematurely #16192
Conversation
This asserted that flush_pos would be ahead of safe_pos after calling _flush. However, this is not guaranteed to be the case because prezeroing might prevent us from flushing right now. Signed-off-by: John Spray <john.spray@redhat.com> (cherry picked from commit 58ec1c6)
Signed-off-by: "Yan, Zheng" <zyan@redhat.com> (cherry picked from commit 41618d8)
Hmm, I can't remember how bad the issue was that needed the followup commit: @ukernel ? |
it's better to also cherry pick 146383a. |
The whole series fix an issue in purge queue. we haven't back-ported purge queue to jewel. So another option is reverting #15468 |
the bug can cause damaged mds. it should be blocker for 10.2.8. |
Since we tend towards not backporting features, and the purge queue sounds like a feature, I propose the following course of action:
Alternatively, we could cherry-pick 146383a into this PR, merge it, and cut a 10.2.9 with this PR? |
See the discussion in ceph#16192 Signed-off-by: Nathan Cutler <ncutler@suse.com>
http://tracker.ceph.com/issues/20536 was rejected |
See the discussion in ceph#16192 Signed-off-by: Nathan Cutler <ncutler@suse.com>
See the discussion in ceph#16192 Signed-off-by: Nathan Cutler <ncutler@suse.com>
http://tracker.ceph.com/issues/20536