mirror of
https://github.com/intel/compute-runtime.git
synced 2025-09-15 13:01:45 +08:00
Store single KernelInfo in Kernel
remove root device index from Kernel's methods Related-To: NEO-5001 Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com>
This commit is contained in:

committed by
Compute-Runtime-Automation

parent
ecceddcab6
commit
7098e9c5f2
@ -37,8 +37,8 @@ GEN12LPTEST_F(GpgpuWalkerTests, givenMiStoreRegMemWhenAdjustMiStoreRegMemModeThe
|
||||
|
||||
class MockKernelWithApplicableWa : public MockKernel {
|
||||
public:
|
||||
MockKernelWithApplicableWa(Program *program, const KernelInfoContainer &kernelInfos, ClDevice &clDeviceArg) : MockKernel(program, kernelInfos, clDeviceArg) {}
|
||||
bool requiresWaDisableRccRhwoOptimization(uint32_t rootDeviceIndex) const override {
|
||||
MockKernelWithApplicableWa(Program *program, const KernelInfo &kernelInfos, ClDevice &clDeviceArg) : MockKernel(program, kernelInfos, clDeviceArg) {}
|
||||
bool requiresWaDisableRccRhwoOptimization() const override {
|
||||
return waApplicable;
|
||||
}
|
||||
bool waApplicable = false;
|
||||
@ -138,14 +138,14 @@ GEN12LPTEST_F(HardwareInterfaceTests, GivenKernelWithApplicableWaDisableRccRhwoO
|
||||
using MI_LOAD_REGISTER_IMM = typename FamilyType::MI_LOAD_REGISTER_IMM;
|
||||
|
||||
pKernel->waApplicable = true;
|
||||
auto cmdSize = GpgpuWalkerHelper<FamilyType>::getSizeForWaDisableRccRhwoOptimization(pKernel, rootDeviceIndex);
|
||||
auto cmdSize = GpgpuWalkerHelper<FamilyType>::getSizeForWaDisableRccRhwoOptimization(pKernel);
|
||||
size_t expectedSize = 2 * (sizeof(PIPE_CONTROL) + sizeof(MI_LOAD_REGISTER_IMM));
|
||||
EXPECT_EQ(expectedSize, cmdSize);
|
||||
}
|
||||
|
||||
GEN12LPTEST_F(HardwareInterfaceTests, GivenKernelWithoutApplicableWaDisableRccRhwoOptimizationWhenCalculatingCommandsSizeThenZeroIsReturned) {
|
||||
pKernel->waApplicable = false;
|
||||
auto cmdSize = GpgpuWalkerHelper<FamilyType>::getSizeForWaDisableRccRhwoOptimization(pKernel, rootDeviceIndex);
|
||||
auto cmdSize = GpgpuWalkerHelper<FamilyType>::getSizeForWaDisableRccRhwoOptimization(pKernel);
|
||||
EXPECT_EQ(0u, cmdSize);
|
||||
}
|
||||
|
||||
|
@ -1,5 +1,5 @@
|
||||
/*
|
||||
* Copyright (C) 2019-2020 Intel Corporation
|
||||
* Copyright (C) 2019-2021 Intel Corporation
|
||||
*
|
||||
* SPDX-License-Identifier: MIT
|
||||
*
|
||||
@ -22,7 +22,7 @@ GEN12LPTEST_F(Gen12LpKernelTest, givenKernelWhenCanTransformImagesIsCalledThenRe
|
||||
|
||||
GEN12LPTEST_F(Gen12LpKernelTest, GivenKernelWhenNotUsingSharedObjArgsThenWaDisableRccRhwoOptimizationIsNotRequired) {
|
||||
MockKernelWithInternals kernel(*pClDevice);
|
||||
EXPECT_FALSE(kernel.mockKernel->requiresWaDisableRccRhwoOptimization(rootDeviceIndex));
|
||||
EXPECT_FALSE(kernel.mockKernel->requiresWaDisableRccRhwoOptimization());
|
||||
}
|
||||
|
||||
GEN12LPTEST_F(Gen12LpKernelTest, GivenKernelWhenAtLeastOneArgIsMediaCompressedThenWaDisableRccRhwoOptimizationIsRequired) {
|
||||
@ -56,5 +56,5 @@ GEN12LPTEST_F(Gen12LpKernelTest, GivenKernelWhenAtLeastOneArgIsMediaCompressedTh
|
||||
cl_mem clMem2 = &bufferMediaCompressed;
|
||||
kernel.mockKernel->setArgBuffer(2, sizeof(cl_mem *), &clMem2);
|
||||
|
||||
EXPECT_TRUE(kernel.mockKernel->requiresWaDisableRccRhwoOptimization(rootDeviceIndex));
|
||||
EXPECT_TRUE(kernel.mockKernel->requiresWaDisableRccRhwoOptimization());
|
||||
}
|
||||
|
@ -29,12 +29,11 @@ TGLLPTEST_F(KernelTgllpTests, GivenUseOffsetToSkipSetFFIDGPWorkaroundActiveWhenS
|
||||
|
||||
hwInfo.platform.usRevId = hwHelper.getHwRevIdFromStepping(stepping, hwInfo);
|
||||
auto device = std::make_unique<MockClDevice>(MockDevice::createWithNewExecutionEnvironment<MockDevice>(&hwInfo));
|
||||
auto rootDeviceIndex = device->getRootDeviceIndex();
|
||||
MockKernelWithInternals mockKernelWithInternals{*device};
|
||||
populateKernelDescriptor(mockKernelWithInternals.kernelInfo.kernelDescriptor, threadPayload);
|
||||
|
||||
for (auto isCcsUsed : ::testing::Bool()) {
|
||||
uint64_t kernelStartOffset = mockKernelWithInternals.mockKernel->getKernelStartOffset(false, false, isCcsUsed, rootDeviceIndex);
|
||||
uint64_t kernelStartOffset = mockKernelWithInternals.mockKernel->getKernelStartOffset(false, false, isCcsUsed);
|
||||
|
||||
if (stepping == REVISION_A0 && isCcsUsed) {
|
||||
EXPECT_EQ(defaultKernelStartOffset + additionalOffsetDueToFfid, kernelStartOffset);
|
||||
|
Reference in New Issue
Block a user