diff --git a/mindspore/ccsrc/ps/core/server_node.cc b/mindspore/ccsrc/ps/core/server_node.cc
index 40a117435f..c10ba5ebe6 100644
--- a/mindspore/ccsrc/ps/core/server_node.cc
+++ b/mindspore/ccsrc/ps/core/server_node.cc
@@ -150,7 +150,6 @@ bool ServerNode::Stop() {
 }
 
 bool ServerNode::Finish(const uint32_t &timeout) {
-  std::lock_guard<std::mutex> lock(finish_mutex_);
   if (is_already_finished_) {
     MS_LOG(INFO) << "Server node already finish!";
     return true;
diff --git a/mindspore/ccsrc/ps/core/worker_node.cc b/mindspore/ccsrc/ps/core/worker_node.cc
index d4138c0751..774220aaa5 100644
--- a/mindspore/ccsrc/ps/core/worker_node.cc
+++ b/mindspore/ccsrc/ps/core/worker_node.cc
@@ -81,7 +81,6 @@ bool WorkerNode::Stop() {
 }
 
 bool WorkerNode::Finish(const uint32_t &timeout) {
-  std::lock_guard<std::mutex> lock(finish_mutex_);
   if (is_already_finished_) {
     MS_LOG(INFO) << "Worker node already finish!";
     return true;