Commit 66945294 authored by sjplimp's avatar sjplimp Committed by GitHub
Browse files

Merge pull request #515 from stanmoore1/remove_fences

Remove unnecessary thread fences in Kokkos package
parents 9a7207e3 ed50bd22
Loading
Loading
Loading
Loading
+0 −1
Original line number Diff line number Diff line
@@ -111,7 +111,6 @@ void AngleCharmmKokkos<DeviceType>::compute(int eflag_in, int vflag_in)
      Kokkos::parallel_for(Kokkos::RangePolicy<DeviceType, TagAngleCharmmCompute<0,0> >(0,nanglelist),*this);
    }
  }
  DeviceType::fence();

  if (eflag_global) energy += ev.evdwl;
  if (vflag_global) {
+0 −1
Original line number Diff line number Diff line
@@ -110,7 +110,6 @@ void BondClass2Kokkos<DeviceType>::compute(int eflag_in, int vflag_in)
      Kokkos::parallel_for(Kokkos::RangePolicy<DeviceType, TagBondClass2Compute<0,0> >(0,nbondlist),*this);
    }
  }
  //DeviceType::fence();

  if (eflag_global) energy += ev.evdwl;
  if (vflag_global) {
+0 −1
Original line number Diff line number Diff line
@@ -125,7 +125,6 @@ void BondFENEKokkos<DeviceType>::compute(int eflag_in, int vflag_in)
      Kokkos::parallel_for(Kokkos::RangePolicy<DeviceType, TagBondFENECompute<0,0> >(0,nbondlist),*this);
    }
  }
  DeviceType::fence();

  k_warning_flag.template modify<DeviceType>();
  k_warning_flag.template sync<LMPHostType>();
+0 −1
Original line number Diff line number Diff line
@@ -111,7 +111,6 @@ void BondHarmonicKokkos<DeviceType>::compute(int eflag_in, int vflag_in)
      Kokkos::parallel_for(Kokkos::RangePolicy<DeviceType, TagBondHarmonicCompute<0,0> >(0,nbondlist),*this);
    }
  }
  //DeviceType::fence();

  if (eflag_global) energy += ev.evdwl;
  if (vflag_global) {
+0 −2
Original line number Diff line number Diff line
@@ -63,7 +63,6 @@ double ComputeTempKokkos<DeviceType>::compute_scalar()
    Kokkos::parallel_reduce(Kokkos::RangePolicy<DeviceType, TagComputeTempScalar<1> >(0,nlocal),*this,t_kk);
  else
    Kokkos::parallel_reduce(Kokkos::RangePolicy<DeviceType, TagComputeTempScalar<0> >(0,nlocal),*this,t_kk);
  DeviceType::fence();
  copymode = 0;

  t = t_kk.t0; // could make this more efficient
@@ -118,7 +117,6 @@ void ComputeTempKokkos<DeviceType>::compute_vector()
    Kokkos::parallel_reduce(Kokkos::RangePolicy<DeviceType, TagComputeTempVector<1> >(0,nlocal),*this,t_kk);
  else
    Kokkos::parallel_reduce(Kokkos::RangePolicy<DeviceType, TagComputeTempVector<0> >(0,nlocal),*this,t_kk);
  DeviceType::fence();
  copymode = 0;

  t[0] = t_kk.t0;
Loading