Skip to content

Commit 62b3e07

Browse files
bcheng0127igcbot
authored andcommitted
Add platforms to avoid dst/src overlap
Send dst src operand overlap may cause dead lock due to SBID dependence
1 parent 189ecc2 commit 62b3e07

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

IGC/Compiler/CISACodeGen/CISABuilder.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5245,11 +5245,6 @@ namespace IGC
52455245
SaveOption(vISA_noSendSrcDstOverlap, true);
52465246
}
52475247

5248-
if (m_program->m_Platform->WaDisableSendSrcDstOverlap())
5249-
{
5250-
SaveOption(vISA_noSendSrcDstOverlap, true);
5251-
}
5252-
52535248
// Set to stitch all functions to all kernels in a VISABuidler
52545249
SaveOption(vISA_noStitchExternFunc, false);
52555250

IGC/Compiler/CISACodeGen/Platform.hpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1545,7 +1545,8 @@ bool WaDisableSendSrcDstOverlap() const
15451545
{
15461546
return (!IGC_IS_FLAG_ENABLED(DisableSendSrcDstOverlapWA)) &&
15471547
(m_SkuTable.FtrWddm2Svm != 0 || m_platformInfo.eRenderCoreFamily == IGFX_GEN10_CORE ||
1548-
m_platformInfo.eRenderCoreFamily == IGFX_GEN11_CORE);
1548+
m_platformInfo.eRenderCoreFamily == IGFX_GEN11_CORE ||
1549+
m_platformInfo.eProductFamily >= IGFX_PVC);
15491550

15501551
}
15511552

visa/HWCaps.inc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -334,8 +334,8 @@ bool has8ByteA64Gather() const { return getPlatform() != GENX_TGLLP; }
334334
bool WaDisableSendSrcDstOverlap() const {
335335
return getOption(vISA_noSendSrcDstOverlap) ||
336336
(m_options->getTarget() == VISA_CM && getPlatform() >= GENX_SKL &&
337-
getPlatform() < GENX_TGLLP) ||
338-
getPlatform() == GENX_ICLLP;
337+
getPlatform() < GENX_TGLLP) || getPlatform() == GENX_ICLLP ||
338+
getPlatform() >= Xe_MTL;
339339
}
340340

341341
bool isXeLP() const { return getPlatform() == GENX_TGLLP; }

0 commit comments

Comments
 (0)