FireDynamics / ARTSS

ARTSS is an Accelerator-based Real Time Smoke Simulator, which specialises in fast calculation of smoke propagation
MIT License
11 stars 7 forks source link

Unnecessary copyin() commands #174

Open LinhWuerzburger opened 3 years ago

LinhWuerzburger commented 3 years ago

see #70

LinhWuerzburger commented 3 years ago

by adding

#ifdef _OPENACC
   #include "accel.h"
#endif

and

#ifdef _OPENACC
    acc_present_dump();
#endif

image_2021-10-31_17-36-18 image_2021-10-31_17-36-07

presentcounter in present table dump of OpenACC raises by 1

LinhWuerzburger commented 2 years ago

Depending on which constructor is called the copyin is necessary. Although in case it was already copied the copyin overwrites the new values on the GPU with the old values of the CPU.