From bebfd12a8c5771d29936145ebd66f857b1402bda Mon Sep 17 00:00:00 2001
From: foreman
Date: Mon, 16 Sep 2019 16:50:21 -0400
Subject: [PATCH] P4 to Git Change 1998944 by
vsytchen@vsytchen-remote-ocl-win10 on 2019/09/16 16:39:08
SWDEV-193973 - Only enable image pitch workaround for gfx10.1
ReviewBoardURL = http://ocltc.amd.com/reviews/r/17993/diff/
Affected files ...
... //depot/stg/opencl/drivers/opencl/runtime/device/pal/palsettings.cpp#98 edit
---
rocclr/runtime/device/pal/palsettings.cpp | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/rocclr/runtime/device/pal/palsettings.cpp b/rocclr/runtime/device/pal/palsettings.cpp
index d364b5f784..d9a3bd99ad 100644
--- a/rocclr/runtime/device/pal/palsettings.cpp
+++ b/rocclr/runtime/device/pal/palsettings.cpp
@@ -212,6 +212,10 @@ bool Settings::create(const Pal::DeviceProperties& palProp,
enableWave32Mode_ = GPU_ENABLE_WAVE32_MODE;
}
lcWavefrontSize64_ = !enableWave32Mode_;
+ if (palProp.gfxLevel == Pal::GfxIpLevel::GfxIp10_1) {
+ // GFX10.1 HW doesn't support custom pitch. Enable double copy workaround
+ imageBufferWar_ = GPU_IMAGE_BUFFER_WAR;
+ }
// Fall through to AI (gfx9) ...
case Pal::AsicRevision::Vega20:
// Enable HW P2P path for Vega20+. Runtime still relies on KMD/PAL for support
@@ -337,11 +341,6 @@ bool Settings::create(const Pal::DeviceProperties& palProp,
return false;
}
- if (gfx10Plus_) {
- // GFX10 HW doesn't support custom pitch. Enable double copy workaround
- imageBufferWar_ = GPU_IMAGE_BUFFER_WAR;
- }
-
// Image DMA must be disabled if SDMA is disabled
imageDMA_ &= !disableSdma_;