From fa45ed8190edc40e81d8c9711691c8123494fb9b Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Fri, 29 Aug 2014 19:27:34 -0700 Subject: [PATCH] osd/ReplicatedPG: do not evict blocked objects If the object is blocked, that means we are either still promoting it, or we are in the process of doing a blocking flush. In both cases, do not evict it just yet. For the promotion case in particular this can cause a very long op delay because we'll have to restart the promotion, and this can potentially loop indefinitely. For the flush case, flushes are generally only blocking when the user explicitly requests it. In that case, we are not particularly concerned that we may delay the objects eventual eviction as the user is probably controlling that anyway (and even if they aren't, the agent will still try this object again later). Fixes: #9285 Reported-by: Wang, Zhiqiang Signed-off-by: Sage Weil --- src/osd/ReplicatedPG.cc | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc index 27ee3f3dbdf3..937108236ed1 100644 --- a/src/osd/ReplicatedPG.cc +++ b/src/osd/ReplicatedPG.cc @@ -11581,6 +11581,10 @@ bool ReplicatedPG::agent_maybe_evict(ObjectContextRef& obc) dout(20) << __func__ << " skip (watchers) " << obc->obs.oi << dendl; return false; } + if (obc->is_blocked()) { + dout(20) << __func__ << " skip (blocked) " << obc->obs.oi << dendl; + return false; + } if (soid.snap == CEPH_NOSNAP) { int result = _verify_no_head_clones(soid, obc->ssc->snapset); -- 2.47.3