diff options
| author | Greg Farnum <greg@inktank.com> | 2013-09-18 17:49:17 -0700 |
|---|---|---|
| committer | Greg Farnum <greg@inktank.com> | 2013-09-30 14:49:36 -0700 |
| commit | 7f3165b52c7d33eb4417bed5d0f64a5be91f8d36 (patch) | |
| tree | d3d48c954f893bee2dd36e7845600f305d62e8c4 | |
| parent | 76321f8fc8ab5287e1a12473df8337cfd7711f4c (diff) | |
| download | ceph-7f3165b52c7d33eb4417bed5d0f64a5be91f8d36.tar.gz | |
ReplicatedPG: use our already-found iterator instead of going back into map
We have an iterator pointing at the element we want; no need to search
the map again in order to grab the element or remove it.
Signed-off-by: Greg Farnum <greg@inktank.com>
| -rw-r--r-- | src/osd/ReplicatedPG.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc index c826c38c145..88dc150448b 100644 --- a/src/osd/ReplicatedPG.cc +++ b/src/osd/ReplicatedPG.cc @@ -5468,10 +5468,10 @@ void ReplicatedPG::kick_object_context_blocked(ObjectContextRef obc) return; } - list<OpRequestRef>& ls = waiting_for_blocked_object[soid]; + list<OpRequestRef>& ls = p->second; dout(10) << __func__ << " " << soid << " requeuing " << ls.size() << " requests" << dendl; requeue_ops(ls); - waiting_for_blocked_object.erase(soid); + waiting_for_blocked_object.erase(p); } SnapSetContext *ReplicatedPG::create_snapset_context(const object_t& oid) |