mirror of
https://github.com/intel/compute-runtime.git
synced 2026-01-05 09:09:04 +08:00
refactor: Simplify fence selection in ULLS
Related-To: NEO-14642 Signed-off-by: Lukasz Jobczyk <lukasz.jobczyk@intel.com>
This commit is contained in:
committed by
Compute-Runtime-Automation
parent
21f6bb8889
commit
455209aadc
@@ -69,11 +69,7 @@ DirectSubmissionHw<GfxFamily, Dispatcher>::DirectSubmissionHw(const DirectSubmis
|
||||
detectGpuHang = !!debugManager.flags.DirectSubmissionDetectGpuHang.get();
|
||||
}
|
||||
|
||||
if (hwInfo->capabilityTable.isIntegratedDevice) {
|
||||
miMemFenceRequired = false;
|
||||
} else {
|
||||
miMemFenceRequired = productHelper.isGlobalFenceInDirectSubmissionRequired(*hwInfo);
|
||||
}
|
||||
miMemFenceRequired = productHelper.isGlobalFenceInDirectSubmissionRequired(*hwInfo);
|
||||
|
||||
if (debugManager.flags.DirectSubmissionInsertExtraMiMemFenceCommands.get() != -1) {
|
||||
miMemFenceRequired = debugManager.flags.DirectSubmissionInsertExtraMiMemFenceCommands.get();
|
||||
|
||||
@@ -20,7 +20,7 @@ bool ProductHelperHw<gfxProduct>::isBlitterForImagesSupported() const {
|
||||
|
||||
template <>
|
||||
bool ProductHelperHw<gfxProduct>::isGlobalFenceInCommandStreamRequired(const HardwareInfo &hwInfo) const {
|
||||
return true;
|
||||
return !hwInfo.capabilityTable.isIntegratedDevice;
|
||||
}
|
||||
|
||||
template <>
|
||||
|
||||
@@ -20,7 +20,7 @@ bool ProductHelperHw<gfxProduct>::isBlitterForImagesSupported() const {
|
||||
|
||||
template <>
|
||||
bool ProductHelperHw<gfxProduct>::isGlobalFenceInCommandStreamRequired(const HardwareInfo &hwInfo) const {
|
||||
return true;
|
||||
return !hwInfo.capabilityTable.isIntegratedDevice;
|
||||
}
|
||||
|
||||
template <>
|
||||
|
||||
@@ -681,7 +681,7 @@ XE2_HPG_CORETEST_F(ProductHelperTestXe2HpgCore, givenProductHelperWhenCallUseGem
|
||||
|
||||
XE2_HPG_CORETEST_F(ProductHelperTestXe2HpgCore, givenProductHelperWhenAskingForGlobalFenceSupportThenReturnTrue) {
|
||||
const auto &productHelper = getHelper<ProductHelper>();
|
||||
EXPECT_TRUE(productHelper.isGlobalFenceInCommandStreamRequired(*defaultHwInfo));
|
||||
EXPECT_EQ(productHelper.isGlobalFenceInCommandStreamRequired(*defaultHwInfo), !defaultHwInfo->capabilityTable.isIntegratedDevice);
|
||||
}
|
||||
|
||||
XE2_HPG_CORETEST_F(ProductHelperTestXe2HpgCore, givenProductHelperWhenAskingForCooperativeEngineSupportThenReturnTrue) {
|
||||
|
||||
@@ -696,7 +696,7 @@ XE3_CORETEST_F(ProductHelperTestXe3Core, givenProductHelperWhenIsBlitterForImage
|
||||
|
||||
XE3_CORETEST_F(ProductHelperTestXe3Core, givenProductHelperWhenAskingForGlobalFenceSupportThenReturnTrue) {
|
||||
auto &productHelper = getHelper<ProductHelper>();
|
||||
EXPECT_TRUE(productHelper.isGlobalFenceInCommandStreamRequired(*defaultHwInfo));
|
||||
EXPECT_EQ(productHelper.isGlobalFenceInCommandStreamRequired(*defaultHwInfo), !defaultHwInfo->capabilityTable.isIntegratedDevice);
|
||||
}
|
||||
|
||||
XE3_CORETEST_F(ProductHelperTestXe3Core, givenProductHelperWhenCallDeferMOCSToPatThenTrueIsReturned) {
|
||||
|
||||
Reference in New Issue
Block a user