[cig-commits] [commit] devel, master: allow the code to use up to 100% of the memory instead of 92% when using UNDO_ATTENUATION on GPUs because UNDO_ATTENUATION buffers are then stored on another machine (the host) instead of on the GPU itself (4cf4cfc)

[cig_noreply at geodynamics.org]

Repository : https://github.com/geodynamics/specfem3d_globe

On branches: devel,master
Link       : https://github.com/geodynamics/specfem3d_globe/compare/bc58e579b3b0838a0968725a076f5904845437ca...be63f20cbb6f462104e949894dbe205d2398cd7f

>---------------------------------------------------------------

commit 4cf4cfc0a0597bec6fadf52cbd2defe5fec35588
Author: Dimitri Komatitsch <komatitsch at lma.cnrs-mrs.fr>
Date:   Tue Sep 23 12:23:45 2014 +0200

    allow the code to use up to 100% of the memory instead of 92% when using UNDO_ATTENUATION on GPUs because UNDO_ATTENUATION buffers are then stored on another machine (the host) instead of on the GPU itself


>---------------------------------------------------------------

4cf4cfc0a0597bec6fadf52cbd2defe5fec35588
 src/shared/read_compute_parameters.f90 | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/shared/read_compute_parameters.f90 b/src/shared/read_compute_parameters.f90
index b7d1fc4..525cd7b 100644
--- a/src/shared/read_compute_parameters.f90
+++ b/src/shared/read_compute_parameters.f90
@@ -382,13 +382,18 @@
 
   if (MEMORY_INSTALLED_PER_CORE_IN_GB < 0.1d0) &
        stop 'less than 100 MB per core for MEMORY_INSTALLED_PER_CORE_IN_GB does not seem realistic; exiting...'
-  if (MEMORY_INSTALLED_PER_CORE_IN_GB > 200.d0) &
-       stop 'more than 200 GB per core for MEMORY_INSTALLED_PER_CORE_IN_GB does not seem realistic; exiting...'
+!! DK DK the value below will probably need to be increased one day, on future machines
+  if (MEMORY_INSTALLED_PER_CORE_IN_GB > 512.d0) &
+       stop 'more than 512 GB per core for MEMORY_INSTALLED_PER_CORE_IN_GB does not seem realistic; exiting...'
 
   if (PERCENT_OF_MEM_TO_USE_PER_CORE < 50.d0) &
        stop 'less than 50% for PERCENT_OF_MEM_TO_USE_PER_CORE does not seem realistic; exiting...'
-  if (PERCENT_OF_MEM_TO_USE_PER_CORE > 92.d0) &
-       stop 'more than 92% for PERCENT_OF_MEM_TO_USE_PER_CORE is risky; exiting...'
+  if (PERCENT_OF_MEM_TO_USE_PER_CORE > 100.d0) &
+       stop 'more than 100% for PERCENT_OF_MEM_TO_USE_PER_CORE makes no sense; exiting...'
+!! DK DK will need to remove the ".and. .not. GPU_MODE" test here
+!! DK DK if the undo_attenuation buffers are stored on the GPU instead of on the host
+  if (PERCENT_OF_MEM_TO_USE_PER_CORE > 92.d0 .and. .not. GPU_MODE) &
+       stop 'more than 92% for PERCENT_OF_MEM_TO_USE_PER_CORE when not using GPUs is risky; exiting...'
 
   what_we_can_use_in_GB = MEMORY_INSTALLED_PER_CORE_IN_GB * PERCENT_OF_MEM_TO_USE_PER_CORE / 100.d0