Skip to content
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

nautilus: rgw_file: advance_mtime() takes RGWFileHandle::mutex unconditionally #29801

Merged
merged 1 commit into from Aug 21, 2019

Conversation

@smithfarm
Copy link
Contributor

commented Aug 21, 2019

master tracker: https://tracker.ceph.com/issues/41173
backport of master PR: #29560
backport tracker: https://tracker.ceph.com/issues/41263


AFAICT this is a must-have for 14.2.3 because it fixes a regression introduced by #29410 which has already been merged (?)

rgw_file: dont deadlock in advance_mtime()
Fixes: https://tracker.ceph.com/issues/41173

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
(cherry picked from commit 0d3338c)
@mattbenjamin
Copy link
Contributor

left a comment

lgtm

@yuriw
yuriw approved these changes Aug 21, 2019

@yuriw yuriw merged commit e157074 into ceph:nautilus Aug 21, 2019

2 of 4 checks passed

Docs: build check Docs: failed with errors
Details
Signed-off-by one or more commits in this PR are not signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
@cbodley

This comment has been minimized.

Copy link
Contributor

commented Aug 21, 2019

thanks @smithfarm @yuriw!

@smithfarm smithfarm deleted the smithfarm:wip-41263-nautilus branch Aug 23, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.