Skip to content

Commit bb55ba5

Browse files
jplehrronlieb
authored andcommitted
[OpenMP] Default to single SDMA engine
The actual concurrent use of multiple SDMA engines requires more work in the plugin and how signals/streams are used to chain together data copies. Thus, for the time being, we re-enable the original implementation that let's ROCr decide which engine to use. Change-Id: I4ff78284a5bbbe4a851b465052ff017fa6bc8a7c
1 parent 5b2ce4d commit bb55ba5

File tree

1 file changed

+1
-1
lines changed
  • openmp/libomptarget/plugins-nextgen/amdgpu/src

1 file changed

+1
-1
lines changed

openmp/libomptarget/plugins-nextgen/amdgpu/src/rtl.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2486,7 +2486,7 @@ struct AMDGPUDeviceTy : public GenericDeviceTy, AMDGenericDeviceTy {
24862486
OMPX_StreamBusyWait("LIBOMPTARGET_AMDGPU_STREAM_BUSYWAIT", 2000000),
24872487
OMPX_UseMultipleSdmaEngines(
24882488
// setting default to true here appears to solve random sdma problem
2489-
"LIBOMPTARGET_AMDGPU_USE_MULTIPLE_SDMA_ENGINES", true),
2489+
"LIBOMPTARGET_AMDGPU_USE_MULTIPLE_SDMA_ENGINES", false),
24902490
OMPX_ApuMaps("OMPX_APU_MAPS", false),
24912491
OMPX_DisableUsmMaps("OMPX_DISABLE_USM_MAPS", false),
24922492
OMPX_NoMapChecks("OMPX_DISABLE_MAPS", true),

0 commit comments

Comments
 (0)