From f6de1c9ad06a9d625cc1cbc8b95a6c066bac0966 Mon Sep 17 00:00:00 2001 From: Niu Chong Date: Mon, 10 Sep 2018 15:48:27 +0800 Subject: [PATCH] fix(actor): bug: do not send consumed ctrl regst msg to producer rightly (#1222) Former-commit-id: 592227a1ef50795ccdd3b44db0befb6e5f22f522 --- oneflow/core/actor/actor.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/oneflow/core/actor/actor.cpp b/oneflow/core/actor/actor.cpp index 2706d3198b..e5d5f48d09 100644 --- a/oneflow/core/actor/actor.cpp +++ b/oneflow/core/actor/actor.cpp @@ -452,8 +452,8 @@ void Actor::AsyncSendCtrlRegstMsgToProducer() { CHECK_GE(returned_regst_num, 1); CHECK_GE(reg_deq.size(), returned_regst_num); - while (returned_regst_num--) { - Regst* regst = reg_deq.front(); + for (size_t i = 0; i < returned_regst_num; ++i) { + Regst* regst = reg_deq.at(i); AsyncSendMsg(ActorMsg::BuildRegstMsgToProducer(actor_id_, regst->producer_actor_id(), regst)); regst_desc_ids.push_back(regst->regst_desc_id()); } -- GitLab