Open jdinan opened 5 years ago
Reported by @minsii:
We notice that you add context error checking for the performance tests. We actually test all unit, perf and app tests for OSHMPI. With this change, the following tests fail with OSHMPI:
FAIL: shmem_bw_put_ctx_perf FAIL: shmem_bw_put_ctx_perf_nb FAIL: shmem_bibw_put_ctx_perf FAIL: shmem_bibw_put_ctx_perf_nb FAIL: shmem_latency_put_ctx_perf FAIL: shmem_latency_put_ctx_perf_nb FAIL: shmem_latency_get_ctx_perf FAIL: shmem_latency_get_ctx_perf_nb
Do you think the default context can be also used in these tests if shmem_ctx_create returns error ?
Reported by @minsii:
We notice that you add context error checking for the performance tests. We actually test all unit, perf and app tests for OSHMPI. With this change, the following tests fail with OSHMPI:
Do you think the default context can be also used in these tests if shmem_ctx_create returns error ?