From: Artem Zhmurov <zhmurov@gmail.com>
Date: Sun, 20 Oct 2019 13:30:30 +0000 (+0200)
Subject: Remove unnecessary CUDA stream synchronization calls
X-Git-Url: http://biod.pnpi.spb.ru/gitweb/?a=commitdiff_plain;h=34711e3fccc66a7eac0d2c9540834654f123f165;p=alexxy%2Fgromacs.git

Remove unnecessary CUDA stream synchronization calls

These calls were needed before the synchronization was re-introduced.
With the event-based synchronization in place, they are no longer
necessary.

Change-Id: I9507432b40962cd49e4fc7374f15530c7fbf2ae7
---

diff --git a/src/gromacs/mdtypes/state_propagator_data_gpu_impl_gpu.cpp b/src/gromacs/mdtypes/state_propagator_data_gpu_impl_gpu.cpp
index cca10920d1..dacda5fed1 100644
--- a/src/gromacs/mdtypes/state_propagator_data_gpu_impl_gpu.cpp
+++ b/src/gromacs/mdtypes/state_propagator_data_gpu_impl_gpu.cpp
@@ -349,8 +349,6 @@ void StatePropagatorDataGpu::Impl::copyCoordinatesFromGpu(gmx::ArrayRef<gmx::RVe
     GMX_ASSERT(commandStream != nullptr, "No stream is valid for copying positions with given atom locality.");
 
     copyFromDevice(h_x, d_x_, d_xSize_, atomLocality, commandStream);
-    // TODO: Remove When event-based synchronization is introduced
-    gpuStreamSynchronize(commandStream);
     // Note: unlike copyCoordinatesToGpu this is not used in OpenCL, and the conditional is not needed.
     xReadyOnHost_[atomLocality].markEvent(commandStream);
 }
@@ -374,8 +372,6 @@ void StatePropagatorDataGpu::Impl::copyVelocitiesToGpu(const gmx::ArrayRef<const
     GMX_ASSERT(commandStream != nullptr, "No stream is valid for copying velocities with given atom locality.");
 
     copyToDevice(d_v_, h_v, d_vSize_, atomLocality, commandStream);
-    // TODO: Remove When event-based synchronization is introduced
-    gpuStreamSynchronize(commandStream);
     vReadyOnDevice_[atomLocality].markEvent(commandStream);
 }
 
@@ -393,8 +389,6 @@ void StatePropagatorDataGpu::Impl::copyVelocitiesFromGpu(gmx::ArrayRef<gmx::RVec
     GMX_ASSERT(commandStream != nullptr, "No stream is valid for copying velocities with given atom locality.");
 
     copyFromDevice(h_v, d_v_, d_vSize_, atomLocality, commandStream);
-    // TODO: Remove When event-based synchronization is introduced
-    gpuStreamSynchronize(commandStream);
     vReadyOnHost_[atomLocality].markEvent(commandStream);
 }