diff --git a/opencl/source/command_queue/hardware_interface_base.inl b/opencl/source/command_queue/hardware_interface_base.inl index 0897a014ac..55c0f3350d 100644 --- a/opencl/source/command_queue/hardware_interface_base.inl +++ b/opencl/source/command_queue/hardware_interface_base.inl @@ -107,7 +107,7 @@ void HardwareInterface::dispatchWalker( auto numSupportedDevices = commandQueue.getGpgpuCommandStreamReceiver().getOsContext().getNumSupportedDevices(); TimestampPacketHelper::programCsrDependencies(*commandStream, csrDependencies, numSupportedDevices); - dsh->align(HardwareCommandsHelper::alignInterfaceDescriptorData); + dsh->align(EncodeStates::alignInterfaceDescriptorData); uint32_t interfaceDescriptorIndex = 0; const size_t offsetInterfaceDescriptorTable = dsh->getUsed(); diff --git a/opencl/source/device_queue/device_queue_hw_bdw_plus.inl b/opencl/source/device_queue/device_queue_hw_bdw_plus.inl index f8d46da9e3..6ce83a723f 100644 --- a/opencl/source/device_queue/device_queue_hw_bdw_plus.inl +++ b/opencl/source/device_queue/device_queue_hw_bdw_plus.inl @@ -130,7 +130,7 @@ void DeviceQueueHw::addMediaStateClearCmds() { addDcFlushToPipeControlWa(pipeControl); - PreambleHelper::programVFEState(&slbCS, device->getHardwareInfo(), 0, 0, device->getSharedDeviceInfo().maxFrontEndThreads, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&slbCS, device->getHardwareInfo(), 0u, 0, device->getSharedDeviceInfo().maxFrontEndThreads, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); } template diff --git a/opencl/source/helpers/hardware_commands_helper.h b/opencl/source/helpers/hardware_commands_helper.h index 1d174f4c12..7bcb3aa7c3 100644 --- a/opencl/source/helpers/hardware_commands_helper.h +++ b/opencl/source/helpers/hardware_commands_helper.h @@ -143,9 +143,6 @@ struct HardwareCommandsHelper : public PerThreadDataHelper { static void programCacheFlushAfterWalkerCommand(LinearStream *commandStream, const CommandQueue &commandQueue, const Kernel *kernel, uint64_t postSyncAddress); - static const size_t alignInterfaceDescriptorData = 64 * sizeof(uint8_t); - static const uint32_t alignIndirectStatePointer = 64 * sizeof(uint8_t); - static bool doBindingTablePrefetch(); static bool inlineDataProgrammingRequired(const Kernel &kernel); diff --git a/opencl/source/helpers/hardware_commands_helper_base.inl b/opencl/source/helpers/hardware_commands_helper_base.inl index 70ec885376..2e300dbdb4 100644 --- a/opencl/source/helpers/hardware_commands_helper_base.inl +++ b/opencl/source/helpers/hardware_commands_helper_base.inl @@ -46,13 +46,14 @@ size_t HardwareCommandsHelper::getSizeRequiredDSH( ? patchInfo.samplerStateArray->Offset - patchInfo.samplerStateArray->BorderColorOffset : 0; - borderColorSize = alignUp(borderColorSize + alignIndirectStatePointer - 1, alignIndirectStatePointer); + borderColorSize = alignUp(borderColorSize + EncodeStates::alignIndirectStatePointer - 1, + EncodeStates::alignIndirectStatePointer); totalSize += borderColorSize + additionalSizeRequiredDsh(); DEBUG_BREAK_IF(!(totalSize >= kernel.getDynamicStateHeapSize() || kernel.getKernelInfo().isVmeWorkload)); - return alignUp(totalSize, alignInterfaceDescriptorData); + return alignUp(totalSize, EncodeStates::alignInterfaceDescriptorData); } template diff --git a/opencl/source/kernel/kernel.cpp b/opencl/source/kernel/kernel.cpp index bf18aa1341..a77310a5a8 100644 --- a/opencl/source/kernel/kernel.cpp +++ b/opencl/source/kernel/kernel.cpp @@ -828,15 +828,6 @@ void Kernel::resizeSurfaceStateHeap(void *pNewSsh, size_t newSshSize, size_t new localBindingTableOffset = newBindingTableOffset; } -uint32_t Kernel::getScratchSizeValueToProgramMediaVfeState(int scratchSize) { - scratchSize >>= MemoryConstants::kiloByteShiftSize; - uint32_t valueToProgram = 0; - while (scratchSize >>= 1) { - valueToProgram++; - } - return valueToProgram; -} - cl_int Kernel::setArg(uint32_t argIndex, size_t argSize, const void *argVal) { cl_int retVal = CL_SUCCESS; bool updateExposedKernel = true; diff --git a/opencl/source/kernel/kernel.h b/opencl/source/kernel/kernel.h index 8249baddd9..30c8802b8a 100644 --- a/opencl/source/kernel/kernel.h +++ b/opencl/source/kernel/kernel.h @@ -204,7 +204,6 @@ class Kernel : public BaseObject<_cl_kernel> { Program *getProgram() const { return program; } - static uint32_t getScratchSizeValueToProgramMediaVfeState(int scratchSize); uint32_t getScratchSize() { return kernelInfo.patchInfo.mediavfestate ? kernelInfo.patchInfo.mediavfestate->PerThreadScratchSpace : 0; } diff --git a/opencl/test/unit_test/command_queue/dispatch_walker_tests.cpp b/opencl/test/unit_test/command_queue/dispatch_walker_tests.cpp index 99d4abcf2e..d131708e63 100644 --- a/opencl/test/unit_test/command_queue/dispatch_walker_tests.cpp +++ b/opencl/test/unit_test/command_queue/dispatch_walker_tests.cpp @@ -885,7 +885,7 @@ HWCMDTEST_F(IGFX_GEN8_CORE, DispatchWalkerTest, GivenMultipleKernelsWhenDispatch // create Indirect DSH heap auto &indirectHeap = pCmdQ->getIndirectHeap(IndirectHeap::DYNAMIC_STATE, 8192); - indirectHeap.align(HardwareCommandsHelper::alignInterfaceDescriptorData); + indirectHeap.align(EncodeStates::alignInterfaceDescriptorData); auto dshBeforeMultiDisptach = indirectHeap.getUsed(); HardwareInterface::dispatchWalker( diff --git a/opencl/test/unit_test/command_queue/enqueue_kernel_1_tests.cpp b/opencl/test/unit_test/command_queue/enqueue_kernel_1_tests.cpp index 32fd2fda46..374a0c34aa 100644 --- a/opencl/test/unit_test/command_queue/enqueue_kernel_1_tests.cpp +++ b/opencl/test/unit_test/command_queue/enqueue_kernel_1_tests.cpp @@ -6,6 +6,7 @@ */ #include "shared/source/helpers/pause_on_gpu_properties.h" +#include "shared/source/helpers/preamble.h" #include "shared/test/unit_test/cmd_parse/hw_parse.h" #include "shared/test/unit_test/helpers/debug_manager_state_restore.h" @@ -461,7 +462,7 @@ HWCMDTEST_F(IGFX_GEN8_CORE, EnqueueKernelTest, givenSecondEnqueueWithTheSameScra MockKernelWithInternals mockKernel(*pClDevice); mockKernel.kernelInfo.patchInfo.mediavfestate = &mediaVFEstate; - auto sizeToProgram = Kernel::getScratchSizeValueToProgramMediaVfeState(scratchSize); + auto sizeToProgram = PreambleHelper::getScratchSizeValueToProgramMediaVfeState(scratchSize); pCmdQ->enqueueKernel(mockKernel.mockKernel, 1, off, gws, nullptr, 0, nullptr, nullptr); hwParser.parseCommands(*pCmdQ); diff --git a/opencl/test/unit_test/command_queue/enqueue_kernel_2_tests.cpp b/opencl/test/unit_test/command_queue/enqueue_kernel_2_tests.cpp index b2a5d979d3..49ac307bee 100644 --- a/opencl/test/unit_test/command_queue/enqueue_kernel_2_tests.cpp +++ b/opencl/test/unit_test/command_queue/enqueue_kernel_2_tests.cpp @@ -27,8 +27,8 @@ using namespace NEO; struct TestParam2 { - cl_uint ScratchSize; -} TestParamTable2[] = {{1024}, {2048}, {4096}, {8192}, {16384}}; + uint32_t scratchSize; +} TestParamTable2[] = {{1024u}, {2048u}, {4096u}, {8192u}, {16384u}}; struct TestParam { cl_uint globalWorkSizeX; @@ -308,20 +308,20 @@ HWCMDTEST_P(IGFX_GEN8_CORE, EnqueueScratchSpaceTests, GivenKernelRequiringScratc EXPECT_TRUE(csr.getAllocationsForReuse().peekIsEmpty()); SPatchMediaVFEState mediaVFEstate; - auto scratchSize = GetParam().ScratchSize; + auto scratchSize = GetParam().scratchSize; mediaVFEstate.PerThreadScratchSpace = scratchSize; MockKernelWithInternals mockKernel(*pClDevice); mockKernel.kernelInfo.patchInfo.mediavfestate = &mediaVFEstate; - auto sizeToProgram = (scratchSize / MemoryConstants::kiloByte); - auto bitValue = 0u; + uint32_t sizeToProgram = (scratchSize / static_cast(MemoryConstants::kiloByte)); + uint32_t bitValue = 0u; while (sizeToProgram >>= 1) { bitValue++; } - auto valueToProgram = Kernel::getScratchSizeValueToProgramMediaVfeState(scratchSize); + auto valueToProgram = PreambleHelper::getScratchSizeValueToProgramMediaVfeState(scratchSize); EXPECT_EQ(bitValue, valueToProgram); enqueueKernel(mockKernel); @@ -443,20 +443,20 @@ HWTEST_P(EnqueueKernelWithScratch, GivenKernelRequiringScratchWhenItIsEnqueuedWi pDevice->resetCommandStreamReceiver(mockCsr); SPatchMediaVFEState mediaVFEstate; - auto scratchSize = 1024; + uint32_t scratchSize = 1024u; mediaVFEstate.PerThreadScratchSpace = scratchSize; MockKernelWithInternals mockKernel(*pClDevice); mockKernel.kernelInfo.patchInfo.mediavfestate = &mediaVFEstate; - auto sizeToProgram = (scratchSize / MemoryConstants::kiloByte); - auto bitValue = 0u; + uint32_t sizeToProgram = (scratchSize / static_cast(MemoryConstants::kiloByte)); + uint32_t bitValue = 0u; while (sizeToProgram >>= 1) { bitValue++; } - auto valueToProgram = Kernel::getScratchSizeValueToProgramMediaVfeState(scratchSize); + auto valueToProgram = PreambleHelper::getScratchSizeValueToProgramMediaVfeState(scratchSize); EXPECT_EQ(bitValue, valueToProgram); enqueueKernel(mockKernel); diff --git a/opencl/test/unit_test/helpers/hardware_commands_helper_tests.cpp b/opencl/test/unit_test/helpers/hardware_commands_helper_tests.cpp index 78c76e6c69..812e9fda18 100644 --- a/opencl/test/unit_test/helpers/hardware_commands_helper_tests.cpp +++ b/opencl/test/unit_test/helpers/hardware_commands_helper_tests.cpp @@ -7,6 +7,7 @@ #include "opencl/test/unit_test/helpers/hardware_commands_helper_tests.h" +#include "shared/source/command_container/command_encoder.h" #include "shared/source/helpers/basic_math.h" #include "shared/source/helpers/engine_node_helper.h" #include "shared/source/memory_manager/unified_memory_manager.h" @@ -321,7 +322,7 @@ HWCMDTEST_F(IGFX_GEN8_CORE, HardwareCommandsTest, WhenAllocatingIndirectStateRes auto usedBeforeIOH = ioh.getUsed(); auto usedBeforeSSH = ssh.getUsed(); - dsh.align(HardwareCommandsHelper::alignInterfaceDescriptorData); + dsh.align(EncodeStates::alignInterfaceDescriptorData); size_t IDToffset = dsh.getUsed(); dsh.getSpace(sizeof(INTERFACE_DESCRIPTOR_DATA)); @@ -535,7 +536,7 @@ HWCMDTEST_F(IGFX_GEN8_CORE, HardwareCommandsTest, whenSendingIndirectStateThenKe auto &ioh = cmdQ.getIndirectHeap(IndirectHeap::INDIRECT_OBJECT, 8192); auto &ssh = cmdQ.getIndirectHeap(IndirectHeap::SURFACE_STATE, 8192); - dsh.align(HardwareCommandsHelper::alignInterfaceDescriptorData); + dsh.align(EncodeStates::alignInterfaceDescriptorData); size_t IDToffset = dsh.getUsed(); dsh.getSpace(sizeof(INTERFACE_DESCRIPTOR_DATA)); diff --git a/shared/source/command_container/command_encoder.h b/shared/source/command_container/command_encoder.h index b139815dad..f0d439c436 100644 --- a/shared/source/command_container/command_encoder.h +++ b/shared/source/command_container/command_encoder.h @@ -72,6 +72,7 @@ struct EncodeStates { using SAMPLER_STATE = typename GfxFamily::SAMPLER_STATE; static const uint32_t alignIndirectStatePointer = MemoryConstants::cacheLineSize; + static const size_t alignInterfaceDescriptorData = MemoryConstants::cacheLineSize; static uint32_t copySamplerState(IndirectHeap *dsh, uint32_t samplerStateOffset, diff --git a/shared/source/command_container/command_encoder.inl b/shared/source/command_container/command_encoder.inl index c7773de1c1..8135cff407 100644 --- a/shared/source/command_container/command_encoder.inl +++ b/shared/source/command_container/command_encoder.inl @@ -33,7 +33,7 @@ uint32_t EncodeStates::copySamplerState(IndirectHeap *dsh, auto sizeSamplerState = sizeof(SAMPLER_STATE) * samplerCount; auto borderColorSize = samplerStateOffset - borderColorOffset; - dsh->align(alignIndirectStatePointer); + dsh->align(EncodeStates::alignIndirectStatePointer); auto borderColorOffsetInDsh = static_cast(dsh->getUsed()); auto borderColor = dsh->getSpace(borderColorSize); @@ -312,7 +312,7 @@ template void *EncodeDispatchKernel::getInterfaceDescriptor(CommandContainer &container, uint32_t &iddOffset) { if (container.nextIddInBlock == container.getNumIddPerBlock()) { - container.getIndirectHeap(HeapType::DYNAMIC_STATE)->align(HardwareCommandsHelper::alignInterfaceDescriptorData); + container.getIndirectHeap(HeapType::DYNAMIC_STATE)->align(EncodeStates::alignInterfaceDescriptorData); container.setIddBlock(container.getHeapSpaceAllowGrow(HeapType::DYNAMIC_STATE, sizeof(INTERFACE_DESCRIPTOR_DATA) * container.getNumIddPerBlock())); container.nextIddInBlock = 0; diff --git a/shared/source/helpers/preamble.h b/shared/source/helpers/preamble.h index a3165b03ac..71435e51b7 100644 --- a/shared/source/helpers/preamble.h +++ b/shared/source/helpers/preamble.h @@ -38,7 +38,7 @@ struct PreambleHelper { static void addPipeControlBeforeVfeCmd(LinearStream *pCommandStream, const HardwareInfo *hwInfo, aub_stream::EngineType engineType); static uint64_t programVFEState(LinearStream *pCommandStream, const HardwareInfo &hwInfo, - int scratchSize, + uint32_t scratchSize, uint64_t scratchAddress, uint32_t maxFrontEndThreads, aub_stream::EngineType engineType, @@ -60,6 +60,7 @@ struct PreambleHelper { static size_t getPerDssBackedBufferCommandsSize(const HardwareInfo &hwInfo); static size_t getCmdSizeForPipelineSelect(const HardwareInfo &hwInfo); static size_t getSemaphoreDelayCommandSize(); + static uint32_t getScratchSizeValueToProgramMediaVfeState(uint32_t scratchSize); }; template diff --git a/shared/source/helpers/preamble_base.inl b/shared/source/helpers/preamble_base.inl index e64be10250..ba5c5edb50 100644 --- a/shared/source/helpers/preamble_base.inl +++ b/shared/source/helpers/preamble_base.inl @@ -13,8 +13,6 @@ #include "shared/source/helpers/preamble.h" #include "shared/source/helpers/register_offsets.h" -#include "opencl/source/kernel/kernel.h" - #include "hw_cmds.h" #include "reg_configs_common.h" @@ -129,4 +127,14 @@ template void PreambleHelper::programAdditionalFieldsInVfeState(VFE_STATE_TYPE *mediaVfeState, const HardwareInfo &hwInfo) { } +template +uint32_t PreambleHelper::getScratchSizeValueToProgramMediaVfeState(uint32_t scratchSize) { + scratchSize >>= static_cast(MemoryConstants::kiloByteShiftSize); + uint32_t valueToProgram = 0; + while (scratchSize >>= 1) { + valueToProgram++; + } + return valueToProgram; +} + } // namespace NEO diff --git a/shared/source/helpers/preamble_bdw_plus.inl b/shared/source/helpers/preamble_bdw_plus.inl index 2894ea5f22..8923b29b5c 100644 --- a/shared/source/helpers/preamble_bdw_plus.inl +++ b/shared/source/helpers/preamble_bdw_plus.inl @@ -27,7 +27,7 @@ uint32_t PreambleHelper::getUrbEntryAllocationSize() { template uint64_t PreambleHelper::programVFEState(LinearStream *pCommandStream, const HardwareInfo &hwInfo, - int scratchSize, + uint32_t scratchSize, uint64_t scratchAddress, uint32_t maxFrontEndThreads, aub_stream::EngineType engineType, @@ -42,8 +42,8 @@ uint64_t PreambleHelper::programVFEState(LinearStream *pCommandStream cmd.setMaximumNumberOfThreads(maxFrontEndThreads); cmd.setNumberOfUrbEntries(1); cmd.setUrbEntryAllocationSize(PreambleHelper::getUrbEntryAllocationSize()); - cmd.setPerThreadScratchSpace(Kernel::getScratchSizeValueToProgramMediaVfeState(scratchSize)); - cmd.setStackSize(Kernel::getScratchSizeValueToProgramMediaVfeState(scratchSize)); + cmd.setPerThreadScratchSpace(PreambleHelper::getScratchSizeValueToProgramMediaVfeState(scratchSize)); + cmd.setStackSize(PreambleHelper::getScratchSizeValueToProgramMediaVfeState(scratchSize)); uint32_t lowAddress = static_cast(0xFFFFFFFF & scratchAddress); uint32_t highAddress = static_cast(0xFFFFFFFF & (scratchAddress >> 32)); cmd.setScratchSpaceBasePointer(lowAddress); diff --git a/shared/test/unit_test/encoders/test_encode_dispatch_kernel.cpp b/shared/test/unit_test/encoders/test_encode_dispatch_kernel.cpp index 40a9d232ce..cd7b1d823c 100644 --- a/shared/test/unit_test/encoders/test_encode_dispatch_kernel.cpp +++ b/shared/test/unit_test/encoders/test_encode_dispatch_kernel.cpp @@ -363,7 +363,7 @@ HWCMDTEST_F(IGFX_GEN8_CORE, CommandEncodeStatesTest, giveNextIddInBlockZeorWhenD uint32_t dims[] = {2, 1, 1}; std::unique_ptr dispatchInterface(new MockDispatchKernelEncoder()); - cmdContainer->getIndirectHeap(HeapType::DYNAMIC_STATE)->align(HardwareCommandsHelper::alignInterfaceDescriptorData); + cmdContainer->getIndirectHeap(HeapType::DYNAMIC_STATE)->align(EncodeStates::alignInterfaceDescriptorData); cmdContainer->setIddBlock(cmdContainer->getHeapSpaceAllowGrow(HeapType::DYNAMIC_STATE, sizeof(INTERFACE_DESCRIPTOR_DATA) * cmdContainer->getNumIddPerBlock())); cmdContainer->nextIddInBlock = 0; diff --git a/shared/test/unit_test/gen11/test_preamble_gen11.cpp b/shared/test/unit_test/gen11/test_preamble_gen11.cpp index 176df3b04a..49110ea2c9 100644 --- a/shared/test/unit_test/gen11/test_preamble_gen11.cpp +++ b/shared/test/unit_test/gen11/test_preamble_gen11.cpp @@ -58,7 +58,7 @@ GEN11TEST_F(Gen11PreambleVfeState, WaOff) { typedef typename ICLFamily::PIPE_CONTROL PIPE_CONTROL; testWaTable->waSendMIFLUSHBeforeVFE = 0; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); @@ -76,7 +76,7 @@ GEN11TEST_F(Gen11PreambleVfeState, WaOn) { typedef typename ICLFamily::PIPE_CONTROL PIPE_CONTROL; testWaTable->waSendMIFLUSHBeforeVFE = 1; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); diff --git a/shared/test/unit_test/gen12lp/test_preamble_gen12lp.cpp b/shared/test/unit_test/gen12lp/test_preamble_gen12lp.cpp index 6b181dba10..4bc4b2e96f 100644 --- a/shared/test/unit_test/gen12lp/test_preamble_gen12lp.cpp +++ b/shared/test/unit_test/gen12lp/test_preamble_gen12lp.cpp @@ -62,7 +62,7 @@ HWTEST2_F(Gen12LpPreambleVfeState, WaOff, IsTGLLP) { typedef typename FamilyType::PIPE_CONTROL PIPE_CONTROL; testWaTable->waSendMIFLUSHBeforeVFE = 0; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 672u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 672u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); @@ -82,7 +82,7 @@ HWTEST2_F(Gen12LpPreambleVfeState, givenCcsEngineWhenWaIsSetThenAppropriatePipeC testWaTable->waSendMIFLUSHBeforeVFE = 1; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 672u, aub_stream::EngineType::ENGINE_CCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 672u, aub_stream::EngineType::ENGINE_CCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); @@ -101,7 +101,7 @@ HWTEST2_F(Gen12LpPreambleVfeState, givenRcsEngineWhenWaIsSetThenAppropriatePipeC testWaTable->waSendMIFLUSHBeforeVFE = 1; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 672u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 672u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); diff --git a/shared/test/unit_test/gen8/test_preamble_gen8.cpp b/shared/test/unit_test/gen8/test_preamble_gen8.cpp index d0dc8ec240..a5411fab27 100644 --- a/shared/test/unit_test/gen8/test_preamble_gen8.cpp +++ b/shared/test/unit_test/gen8/test_preamble_gen8.cpp @@ -95,7 +95,7 @@ BDWTEST_F(PreambleVfeState, basic) { typedef BDWFamily::PIPE_CONTROL PIPE_CONTROL; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, *defaultHwInfo, 0, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, *defaultHwInfo, 0u, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); diff --git a/shared/test/unit_test/gen9/skl/test_preamble_skl.cpp b/shared/test/unit_test/gen9/skl/test_preamble_skl.cpp index a033071156..1842b5fc94 100644 --- a/shared/test/unit_test/gen9/skl/test_preamble_skl.cpp +++ b/shared/test/unit_test/gen9/skl/test_preamble_skl.cpp @@ -107,7 +107,7 @@ GEN9TEST_F(PreambleVfeState, WaOff) { typedef typename FamilyType::PIPE_CONTROL PIPE_CONTROL; testWaTable->waSendMIFLUSHBeforeVFE = 0; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs); @@ -125,7 +125,7 @@ GEN9TEST_F(PreambleVfeState, WaOn) { typedef typename FamilyType::PIPE_CONTROL PIPE_CONTROL; testWaTable->waSendMIFLUSHBeforeVFE = 1; LinearStream &cs = linearStream; - PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); + PreambleHelper::programVFEState(&linearStream, pPlatform->getClDevice(0)->getHardwareInfo(), 0u, 0, 168u, aub_stream::EngineType::ENGINE_RCS, AdditionalKernelExecInfo::NotApplicable); parseCommands(cs);