Bug #21566
OSDService::recovery_need_sleep read+updated without locking
% Done:
0%
Source:
Tags:
Backport:
luminous
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
OSD
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
Unless I'm misreading this, OSD::do_recovery() is invoked from the ShardedOpQueue without holding any locks on global OSD state. And then we both read and edit the recovery_needs_sleep member without holding the recovery_request_lock or anything else that prevents it from being handled by multiple threads?`
Related issues
History
#1 Updated by Neha Ojha over 6 years ago
- Status changed from New to Fix Under Review
https://github.com/ceph/ceph/pull/18022 should take care of this.
#2 Updated by Kefu Chai over 6 years ago
- Status changed from Fix Under Review to Pending Backport
#3 Updated by Nathan Cutler over 6 years ago
- Backport set to luminous
#4 Updated by Nathan Cutler over 6 years ago
- Copied to Backport #21697: luminous: OSDService::recovery_need_sleep read+updated without locking added
#5 Updated by Nathan Cutler about 6 years ago
- Status changed from Pending Backport to Resolved