From 18a6b4dd9fcab92cf8aa258b9081802eba9d1823 Mon Sep 17 00:00:00 2001 From: Compute-Runtime-Validation Date: Thu, 22 Dec 2022 21:39:43 +0100 Subject: [PATCH] Revert "fix(zebin): add missing ULT for global/const buffers scenario" This reverts commit a526b912624e7e4169ecad5e9dc7976576d35323. Signed-off-by: Compute-Runtime-Validation --- .../zebin_decoder_tests.cpp | 45 ------------------- 1 file changed, 45 deletions(-) diff --git a/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp b/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp index a554005f37..eb23e4dc40 100644 --- a/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp +++ b/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp @@ -7004,51 +7004,6 @@ kernels: EXPECT_EQ(NEO::DecodeError::InvalidBinary, err); } -TEST(PopulateKernelDescriptor, givenGlobalBufferAndConstBufferWhenPopulatingKernelDescriptorThenMakeThemStateful) { - NEO::ConstStringRef zeinfo = R"===( -kernels: - - name : some_kernel - execution_env: - simd_size: 8 - payload_arguments: - - arg_type: global_base - offset: 0 - size: 8 - bti_value: 0 - - arg_type: const_base - offset: 8 - size: 8 - bti_value: 1 -... -)==="; - NEO::ProgramInfo programInfo; - ZebinTestData::ValidEmptyProgram zebin; - zebin.appendSection(NEO::Elf::SHT_PROGBITS, NEO::Elf::SectionsNamesZebin::textPrefix.str() + "some_kernel", {}); - std::string errors, warnings; - auto elf = NEO::Elf::decodeElf(zebin.storage, errors, warnings); - ASSERT_NE(nullptr, elf.elfFileHeader) << errors << " " << warnings; - - NEO::Yaml::YamlParser parser; - bool parseSuccess = parser.parse(zeinfo, errors, warnings); - ASSERT_TRUE(parseSuccess) << errors << " " << warnings; - - NEO::ZebinSections zebinSections; - auto extractErr = NEO::extractZebinSections(elf, zebinSections, errors, warnings); - ASSERT_EQ(NEO::DecodeError::Success, extractErr) << errors << " " << warnings; - - auto &kernelNode = *parser.createChildrenRange(*parser.findNodeWithKeyDfs("kernels")).begin(); - auto res = NEO::populateKernelDescriptor(programInfo, elf, zebinSections, parser, kernelNode, errors, warnings); - EXPECT_EQ(NEO::DecodeError::Success, res); - EXPECT_TRUE(warnings.empty()); - EXPECT_TRUE(errors.empty()); - - const auto &kernelInfo = *programInfo.kernelInfos.rbegin(); - EXPECT_EQ(2u, kernelInfo->kernelDescriptor.payloadMappings.bindingTable.numEntries); - EXPECT_EQ(128u, kernelInfo->kernelDescriptor.payloadMappings.bindingTable.tableOffset); - ASSERT_EQ(192u, kernelInfo->heapInfo.SurfaceStateHeapSize); - ASSERT_NE(nullptr, kernelInfo->heapInfo.pSsh); -} - TEST(PopulateInlineSamplers, GivenInvalidSamplerIndexThenPopulateInlineSamplersFails) { NEO::KernelDescriptor kd; std::string errors, warnings;