[nvptx, libgomp] Don't launch with num_workers == 0
authorTom de Vries <tdevries@suse.de>
Wed, 9 Jan 2019 00:07:45 +0000 (00:07 +0000)
committerTom de Vries <vries@gcc.gnu.org>
Wed, 9 Jan 2019 00:07:45 +0000 (00:07 +0000)
When using a compiler build with:
...
+#define PTX_DEFAULT_VECTOR_LENGTH PTX_CTA_SIZE
+#define PTX_MAX_VECTOR_LENGTH PTX_CTA_SIZE
...
and running the libgomp testsuite, we run into an execution failure in
parallel-loop-1.c, due to a cuda launch failure:
...
  nvptx_exec: kernel f6_none_none$_omp_fn$0: launch gangs=480, workers=0, \
    vectors=1024

libgomp: cuLaunchKernel error: invalid argument
...
because workers == 0.

The workers variable is set to 0 here in nvptx_exec:
...
                workers = blocks / actual_vectors;
...
because actual_vectors is 1024, and blocks is 768:
...
cuOccupancyMaxPotentialBlockSize: grid = 10, block = 768
...

Fix this by ensuring that workers is at least one.

2019-01-09  Tom de Vries  <tdevries@suse.de>

* plugin/plugin-nvptx.c (nvptx_exec): Make sure to launch with at least
one worker.

From-SVN: r267746

libgomp/ChangeLog
libgomp/plugin/plugin-nvptx.c

index 120f0874b27dd8d799c6966078bcdebe2fb20a1d..fba0ba0562ac4222605b28390fa8bbc2976fca3b 100644 (file)
@@ -1,3 +1,8 @@
+2019-01-09  Tom de Vries  <tdevries@suse.de>
+
+       * plugin/plugin-nvptx.c (nvptx_exec): Make sure to launch with at least
+       one worker.
+
 2019-01-07  Tom de Vries  <tdevries@suse.de>
 
        * testsuite/libgomp.oacc-c-c++-common/vector-length-128-3.c: Fix
index 572d9ef8d5c3d32cf3180789dd05e8ede14e3601..60553bdf3bd58a31eaf72f3f793e63bae22121de 100644 (file)
@@ -1272,6 +1272,7 @@ nvptx_exec (void (*fn), size_t mapnum, void **hostaddrs, void **devaddrs,
                                      ? vectors
                                      : dims[GOMP_DIM_VECTOR]);
                workers = blocks / actual_vectors;
+               workers = MAX (workers, 1);
              }
 
            for (i = 0; i != GOMP_DIM_MAX; i++)