diff --git a/level_zero/core/source/gfx_core_helpers/l0_gfx_core_helper_skl_to_tgllp.inl b/level_zero/core/source/gfx_core_helpers/l0_gfx_core_helper_skl_to_tgllp.inl index 17022e00eb..92a52384d9 100644 --- a/level_zero/core/source/gfx_core_helpers/l0_gfx_core_helper_skl_to_tgllp.inl +++ b/level_zero/core/source/gfx_core_helpers/l0_gfx_core_helper_skl_to_tgllp.inl @@ -11,7 +11,7 @@ namespace L0 { template bool L0GfxCoreHelperHw::platformSupportsCmdListHeapSharing() const { - return true; + return false; } template diff --git a/level_zero/core/source/kernel/kernel_imp.cpp b/level_zero/core/source/kernel/kernel_imp.cpp index af4967e975..2ab0c4cf0e 100644 --- a/level_zero/core/source/kernel/kernel_imp.cpp +++ b/level_zero/core/source/kernel/kernel_imp.cpp @@ -113,7 +113,13 @@ ze_result_t KernelImmutableData::initialize(NEO::KernelInfo *kernelInfo, Device kernelDescriptor->payloadMappings.implicitArgs.simdSize, kernelDescriptor->kernelAttributes.simdSize); } - if (NEO::KernelDescriptor::isBindlessAddressingKernel(kernelInfo->kernelDescriptor)) { + if (kernelInfo->heapInfo.surfaceStateHeapSize != 0) { + this->surfaceStateHeapSize = kernelInfo->heapInfo.surfaceStateHeapSize; + surfaceStateHeapTemplate.reset(new uint8_t[surfaceStateHeapSize]); + + memcpy_s(surfaceStateHeapTemplate.get(), surfaceStateHeapSize, + kernelInfo->heapInfo.pSsh, surfaceStateHeapSize); + } else if (NEO::KernelDescriptor::isBindlessAddressingKernel(kernelInfo->kernelDescriptor)) { auto &gfxCoreHelper = deviceImp->getNEODevice()->getGfxCoreHelper(); auto surfaceStateSize = static_cast(gfxCoreHelper.getRenderSurfaceStateSize()); @@ -121,12 +127,6 @@ ze_result_t KernelImmutableData::initialize(NEO::KernelInfo *kernelInfo, Device UNRECOVERABLE_IF(kernelInfo->kernelDescriptor.kernelAttributes.numArgsStateful != kernelInfo->kernelDescriptor.getBindlessOffsetToSurfaceState().size()); surfaceStateHeapTemplate.reset(new uint8_t[surfaceStateHeapSize]); - } else if (kernelInfo->heapInfo.surfaceStateHeapSize != 0) { - this->surfaceStateHeapSize = kernelInfo->heapInfo.surfaceStateHeapSize; - surfaceStateHeapTemplate.reset(new uint8_t[surfaceStateHeapSize]); - - memcpy_s(surfaceStateHeapTemplate.get(), surfaceStateHeapSize, - kernelInfo->heapInfo.pSsh, surfaceStateHeapSize); } if (kernelInfo->heapInfo.dynamicStateHeapSize != 0) { diff --git a/level_zero/core/test/unit_tests/gen11/test_l0_gfx_core_helper_gen11.cpp b/level_zero/core/test/unit_tests/gen11/test_l0_gfx_core_helper_gen11.cpp index de255bba8f..402cc020a2 100644 --- a/level_zero/core/test/unit_tests/gen11/test_l0_gfx_core_helper_gen11.cpp +++ b/level_zero/core/test/unit_tests/gen11/test_l0_gfx_core_helper_gen11.cpp @@ -16,9 +16,9 @@ namespace ult { using L0GfxCoreHelperTestGen11 = Test; -GEN11TEST_F(L0GfxCoreHelperTestGen11, GivenGen11WhenCheckingL0HelperForCmdListHeapSharingSupportThenReturnTrue) { +GEN11TEST_F(L0GfxCoreHelperTestGen11, GivenGen11WhenCheckingL0HelperForCmdListHeapSharingSupportThenReturnFalse) { auto &l0GfxCoreHelper = getHelper(); - EXPECT_TRUE(l0GfxCoreHelper.platformSupportsCmdListHeapSharing()); + EXPECT_FALSE(l0GfxCoreHelper.platformSupportsCmdListHeapSharing()); } GEN11TEST_F(L0GfxCoreHelperTestGen11, GivenGen11WhenCheckingL0HelperForStateComputeModeTrackingSupportThenReturnFalse) { diff --git a/level_zero/core/test/unit_tests/gen12lp/test_l0_gfx_core_helper_gen12lp.cpp b/level_zero/core/test/unit_tests/gen12lp/test_l0_gfx_core_helper_gen12lp.cpp index 8c98cd6c7c..95772507b4 100644 --- a/level_zero/core/test/unit_tests/gen12lp/test_l0_gfx_core_helper_gen12lp.cpp +++ b/level_zero/core/test/unit_tests/gen12lp/test_l0_gfx_core_helper_gen12lp.cpp @@ -23,10 +23,10 @@ HWTEST_EXCLUDE_PRODUCT(L0GfxCoreHelperTest, givenBitmaskWithAttentionBitsForHalf using L0GfxCoreHelperTestGen12Lp = Test; -GEN12LPTEST_F(L0GfxCoreHelperTestGen12Lp, GivenGen12LpWhenCheckingL0HelperForCmdListHeapSharingSupportThenReturnTrue) { +GEN12LPTEST_F(L0GfxCoreHelperTestGen12Lp, GivenGen12LpWhenCheckingL0HelperForCmdListHeapSharingSupportThenReturnFalse) { auto &l0GfxCoreHelper = getHelper(); - EXPECT_TRUE(l0GfxCoreHelper.platformSupportsCmdListHeapSharing()); + EXPECT_FALSE(l0GfxCoreHelper.platformSupportsCmdListHeapSharing()); } GEN12LPTEST_F(L0GfxCoreHelperTestGen12Lp, GivenGen12LpWhenCheckingL0HelperForStateComputeModeTrackingSupportThenReturnFalse) { diff --git a/level_zero/core/test/unit_tests/gen9/test_l0_gfx_core_helper_gen9.cpp b/level_zero/core/test/unit_tests/gen9/test_l0_gfx_core_helper_gen9.cpp index a715f5029e..dd9dc50b47 100644 --- a/level_zero/core/test/unit_tests/gen9/test_l0_gfx_core_helper_gen9.cpp +++ b/level_zero/core/test/unit_tests/gen9/test_l0_gfx_core_helper_gen9.cpp @@ -16,9 +16,9 @@ namespace ult { using L0GfxCoreHelperTestGen9 = Test; -GEN9TEST_F(L0GfxCoreHelperTestGen9, GivenGen9WhenCheckingL0HelperForCmdListHeapSharingSupportThenReturnTrue) { +GEN9TEST_F(L0GfxCoreHelperTestGen9, GivenGen9WhenCheckingL0HelperForCmdListHeapSharingSupportThenReturnFalse) { auto &l0GfxCoreHelper = getHelper(); - EXPECT_TRUE(l0GfxCoreHelper.platformSupportsCmdListHeapSharing()); + EXPECT_FALSE(l0GfxCoreHelper.platformSupportsCmdListHeapSharing()); } GEN9TEST_F(L0GfxCoreHelperTestGen9, GivenGen9WhenCheckingL0HelperForStateComputeModeTrackingSupportThenReturnFalse) { diff --git a/shared/source/command_container/command_encoder.inl b/shared/source/command_container/command_encoder.inl index 2661ecb87c..e9a29277d9 100644 --- a/shared/source/command_container/command_encoder.inl +++ b/shared/source/command_container/command_encoder.inl @@ -739,7 +739,6 @@ size_t EncodeDispatchKernel::getSizeRequiredDsh(const KernelDescriptor & size = alignUp(size, INTERFACE_DESCRIPTOR_DATA::SAMPLERSTATEPOINTER_ALIGN_SIZE); if (additionalDshSize > 0) { - size = alignUp(size, EncodeStates::alignInterfaceDescriptorData); size += additionalDshSize; size = alignUp(size, EncodeDispatchKernel::getDefaultDshAlignment()); }