Skip to content

Commit

Permalink
bug fix, test=develop
Browse files Browse the repository at this point in the history
  • Loading branch information
sandyhouse committed Jun 9, 2021
1 parent 2b56b1b commit 1f92cc5
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions paddle/fluid/operators/collective/barrier_op.cu.cc
Original file line number Diff line number Diff line change
Expand Up @@ -43,12 +43,10 @@ class BarrierOpCUDAKernel : public framework::OpKernel<T> {
ncclRedOp_t nccl_red_type = ncclSum;
PADDLE_ENFORCE_CUDA_SUCCESS(platform::dynload::ncclAllReduce(
sendbuff, recvbuff, numel, dtype, nccl_red_type, comm->comm(), stream));
auto comm_stream =
platform::NCCLCommContext::Instance().Get(rid, place)->stream();
#ifdef PADDLE_WITH_RCCL
PADDLE_ENFORCE_CUDA_SUCCESS(hipStreamSynchronize(comm_stream));
PADDLE_ENFORCE_CUDA_SUCCESS(hipStreamSynchronize(stream));
#else
PADDLE_ENFORCE_CUDA_SUCCESS(cudaStreamSynchronize(comm_stream));
PADDLE_ENFORCE_CUDA_SUCCESS(cudaStreamSynchronize(stream));
#endif
#else
PADDLE_THROW(platform::errors::Unavailable(
Expand Down

1 comment on commit 1f92cc5

@paddle-bot-old
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congratulation! Your pull request passed all required CI. You could ask reviewer(s) to approve and merge. 🎉

Please sign in to comment.