Skip to content

Commit 494c262

Browse files
wanghaoshuangwangkuiyi
authored andcommitted
Fix average_accumulate_op for parallel executor. (#9852)
1 parent 630943c commit 494c262

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

paddle/fluid/operators/average_accumulates_op.cu

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,14 @@ void GetAccumulators<paddle::platform::CUDADeviceContext>(
2525
auto* in_num_accumulates = ctx.Input<Tensor>("in_num_accumulates");
2626
auto* in_num_updates = ctx.Input<Tensor>("in_num_updates");
2727
auto stream = ctx.cuda_device_context().stream();
28-
memory::Copy(platform::CPUPlace(), old_num_accumulates_,
29-
platform::CUDAPlace(), in_old_num_accumulates->data<int64_t>(),
30-
sizeof(int64_t), stream);
31-
memory::Copy(platform::CPUPlace(), num_accumulates_, platform::CUDAPlace(),
28+
auto cuda_place =
29+
boost::get<platform::CUDAPlace>(in_old_num_accumulates->place());
30+
memory::Copy(platform::CPUPlace(), old_num_accumulates_, cuda_place,
31+
in_old_num_accumulates->data<int64_t>(), sizeof(int64_t),
32+
stream);
33+
memory::Copy(platform::CPUPlace(), num_accumulates_, cuda_place,
3234
in_num_accumulates->data<int64_t>(), sizeof(int64_t), stream);
33-
memory::Copy(platform::CPUPlace(), num_updates_, platform::CUDAPlace(),
35+
memory::Copy(platform::CPUPlace(), num_updates_, cuda_place,
3436
in_num_updates->data<int64_t>(), sizeof(int64_t), stream);
3537
}
3638

@@ -42,14 +44,16 @@ void SetAccumulators<paddle::platform::CUDADeviceContext>(
4244
auto* out_old_num_accumulates = ctx.Output<Tensor>("out_old_num_accumulates");
4345
auto* out_num_accumulates = ctx.Output<Tensor>("out_num_accumulates");
4446
auto* out_num_updates = ctx.Output<Tensor>("out_num_updates");
47+
auto cuda_place =
48+
boost::get<platform::CUDAPlace>(out_old_num_accumulates->place());
4549

46-
memory::Copy(platform::CUDAPlace(), out_old_num_accumulates->data<int64_t>(),
50+
memory::Copy(cuda_place, out_old_num_accumulates->data<int64_t>(),
4751
platform::CPUPlace(), &old_num_accumulates_, sizeof(int64_t),
4852
stream);
49-
memory::Copy(platform::CUDAPlace(), out_num_accumulates->data<int64_t>(),
53+
memory::Copy(cuda_place, out_num_accumulates->data<int64_t>(),
5054
platform::CPUPlace(), &num_accumulates_, sizeof(int64_t),
5155
stream);
52-
memory::Copy(platform::CUDAPlace(), out_num_updates->data<int64_t>(),
56+
memory::Copy(cuda_place, out_num_updates->data<int64_t>(),
5357
platform::CPUPlace(), &num_updates_, sizeof(int64_t), stream);
5458
}
5559

0 commit comments

Comments
 (0)