Media/compute-runtime
1
0
Fork 0
mirror of https://github.com/intel/compute-runtime.git synced 2025-09-10 12:53:42 +08:00
Code Issues Packages Projects Releases Wiki Activity

Move hwHelper ownership to RootDeviceEnvironment 5/n

Browse Source 
Related-To: NEO-6853
Signed-off-by: Kamil Kopryk <kamil.kopryk@intel.com>

UseRootDeviceEnvironment getHelper<CoreHelper> for
- isFenceAllocationRequired

Move common implementation to pvc_and_later file.
This commit is contained in:
Kamil Kopryk
2022-11-14 17:52:33 +00:00
committed by Compute-Runtime-Automation
parent 211cc8552a
commit 1bcceb1071
5 changed files with 21 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
opencl/test/unit_test/fixtures/cl_device_fixture.cpp
View File

@ -49,5 +49,6 @@ HelperType &ClDeviceFixture::getHelper() const {
}
template ProductHelper &ClDeviceFixture::getHelper() const;
template CoreHelper &ClDeviceFixture::getHelper() const;
} // namespace NEO

12
opencl/test/unit_test/xe_hpc_core/hw_helper_tests_xe_hpc_core.cpp
View File

@ -590,32 +590,32 @@ XE_HPC_CORETEST_F(HwHelperTestsXeHpcCore, givenHwHelperWhenAskedIfFenceAllocatio
DebugManagerStateRestore dbgRestore;
auto hwInfo = *defaultHwInfo;
auto &helper = HwHelper::get(renderCoreFamily);
auto &coreHelper = getHelper<CoreHelper>();
DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.set(-1);
DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.set(-1);
DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.set(-1);
EXPECT_TRUE(helper.isFenceAllocationRequired(hwInfo));
EXPECT_TRUE(coreHelper.isFenceAllocationRequired(hwInfo));
DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.set(0);
DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.set(0);
DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.set(0);
EXPECT_FALSE(helper.isFenceAllocationRequired(hwInfo));
EXPECT_FALSE(coreHelper.isFenceAllocationRequired(hwInfo));
DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.set(1);
DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.set(0);
DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.set(0);
EXPECT_TRUE(helper.isFenceAllocationRequired(hwInfo));
EXPECT_TRUE(coreHelper.isFenceAllocationRequired(hwInfo));
DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.set(0);
DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.set(1);
DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.set(0);
EXPECT_TRUE(helper.isFenceAllocationRequired(hwInfo));
EXPECT_TRUE(coreHelper.isFenceAllocationRequired(hwInfo));
DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.set(0);
DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.set(0);
DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.set(1);
EXPECT_TRUE(helper.isFenceAllocationRequired(hwInfo));
EXPECT_TRUE(coreHelper.isFenceAllocationRequired(hwInfo));
}
XE_HPC_CORETEST_F(HwHelperTestsXeHpcCore, givenDontProgramGlobalFenceAsMiMemFenceCommandInCommandStreamWhenGettingSizeForAdditionalSynchronizationThenCorrectValueIsReturned) {

6
shared/source/command_stream/command_stream_receiver.cpp
View File

@ -722,8 +722,10 @@ bool CommandStreamReceiver::createWorkPartitionAllocation(const Device &device)
}
bool CommandStreamReceiver::createGlobalFenceAllocation() {
auto hwInfo = executionEnvironment.rootDeviceEnvironments[rootDeviceIndex]->getHardwareInfo();
if (!HwHelper::get(hwInfo->platform.eRenderCoreFamily).isFenceAllocationRequired(*hwInfo)) {
auto &rootDevicEnvironment = *executionEnvironment.rootDeviceEnvironments[rootDeviceIndex].get();
auto &coreHelper = rootDevicEnvironment.getHelper<CoreHelper>();
auto &hwInfo = *rootDevicEnvironment.getHardwareInfo();
if (!coreHelper.isFenceAllocationRequired(hwInfo)) {
return true;
}

10
shared/source/helpers/hw_helper_pvc_and_later.inl
View File

@ -10,6 +10,16 @@
namespace NEO {
template <>
bool HwHelperHw<Family>::isFenceAllocationRequired(const HardwareInfo &hwInfo) const {
if ((DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.get() == 0) &&
(DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.get() == 0) &&
(DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.get() == 0)) {
return false;
}
return true;
}
template <>
bool HwHelperHw<Family>::isCpuImageTransferPreferred(const HardwareInfo &hwInfo) const {
return !hwInfo.capabilityTable.supportsImages;

10
shared/source/xe_hpc_core/hw_helper_xe_hpc_core.cpp
View File

@ -125,16 +125,6 @@ uint32_t HwHelperHw<Family>::getMinimalSIMDSize() {
return 16u;
}
template <>
bool HwHelperHw<Family>::isFenceAllocationRequired(const HardwareInfo &hwInfo) const {
if ((DebugManager.flags.ProgramGlobalFenceAsMiMemFenceCommandInCommandStream.get() == 0) &&
(DebugManager.flags.ProgramGlobalFenceAsPostSyncOperationInComputeWalker.get() == 0) &&
(DebugManager.flags.ProgramGlobalFenceAsKernelInstructionInEUKernel.get() == 0)) {
return false;
}
return true;
}
template <>
uint32_t HwHelperHw<Family>::getMocsIndex(const GmmHelper &gmmHelper, bool l3enabled, bool l1enabled) const {
if (l3enabled) {