Bug #4647
libceph: kill osd request r_con_filling_msg field
% Done:
0%
Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
This field was (I think) needed at one time to avoid
a race between a request with a response message
getting released and a connection the message was
associated with going away.
It is no longer needed though, and it adds a little
weirdness to the code (an optimization in how to
handle it more efficiently), so get rid of it.
History
#1 Updated by Alex Elder almost 10 years ago
- Target version set to v0.62a
By the way, I'm calling this a bug because "cleanup"
doesn't allow me to mark things as "Need Review"
(or something like that).
#2 Updated by Alex Elder almost 10 years ago
- Status changed from In Progress to Fix Under Review
The following patch has been posted for review:
[PATCH 2/9] libceph: drop ceph_osd_request->r_con_filling_msg
#3 Updated by Alex Elder almost 10 years ago
- Status changed from Fix Under Review to Resolved
The following has been committed to the ceph-client
"testing" repository:
ebc6540 libceph: drop ceph_osd_request->r_con_filling_msg