PG::should_restart_peering() is called twice in state transitions. This patch avoid duplicate evaluation of PastIntervals::is_new_interval(). Signed-off-by: Kouya Shimura <kouya@xxxxxxxxxxxxxx> --- src/osd/PG.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/osd/PG.cc b/src/osd/PG.cc index 13ed169..3b64257 100644 --- a/src/osd/PG.cc +++ b/src/osd/PG.cc @@ -5761,6 +5761,7 @@ bool PG::should_restart_peering( info.pgid.pgid)) { dout(20) << "new interval newup " << newup << " newacting " << newacting << dendl; + restart_peering = true; return true; } else { return false; -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html