From afcbebef0bb0062ca9e1481e882b0d885d953996 Mon Sep 17 00:00:00 2001 From: Tomasz Biernacik Date: Tue, 11 Mar 2025 11:00:41 +0000 Subject: [PATCH] Revert "performance: configure Xe2 compression" This reverts commit ec7821ed8483eeb4c70e0a6b5bfe10ebbe7661dd. Signed-off-by: Tomasz Biernacik --- .../windows/product_helper_wddm.cpp | 5 ----- .../windows/product_helper_win_tests.cpp | 18 ------------------ 2 files changed, 23 deletions(-) diff --git a/shared/source/os_interface/windows/product_helper_wddm.cpp b/shared/source/os_interface/windows/product_helper_wddm.cpp index ee6589fe75..7ca9dbfc65 100644 --- a/shared/source/os_interface/windows/product_helper_wddm.cpp +++ b/shared/source/os_interface/windows/product_helper_wddm.cpp @@ -9,7 +9,6 @@ #include "shared/source/execution_environment/root_device_environment.h" #include "shared/source/helpers/hw_info.h" #include "shared/source/os_interface/product_helper.h" -#include "shared/source/release_helper/release_helper.h" #include "hw_cmds.h" @@ -19,10 +18,6 @@ int ProductHelper::configureHwInfoWddm(const HardwareInfo *inHwInfo, HardwareInf outHwInfo->capabilityTable.ftrSvm = outHwInfo->featureTable.flags.ftrSVM; this->setCapabilityCoherencyFlag(*outHwInfo, outHwInfo->capabilityTable.ftrSupportsCoherency); outHwInfo->capabilityTable.ftrSupportsCoherency &= inHwInfo->featureTable.flags.ftrL3IACoherency; - auto releaseHelper = rootDeviceEnvironment.getReleaseHelper(); - if (releaseHelper) { - outHwInfo->featureTable.flags.ftrXe2Compression = releaseHelper->getFtrXe2Compression(); - } setupDefaultEngineType(*outHwInfo, rootDeviceEnvironment); setupPreemptionMode(*outHwInfo, rootDeviceEnvironment, true); diff --git a/shared/test/unit_test/os_interface/windows/product_helper_win_tests.cpp b/shared/test/unit_test/os_interface/windows/product_helper_win_tests.cpp index f69618f2a5..d2601c2e2d 100644 --- a/shared/test/unit_test/os_interface/windows/product_helper_win_tests.cpp +++ b/shared/test/unit_test/os_interface/windows/product_helper_win_tests.cpp @@ -12,7 +12,6 @@ #include "shared/source/os_interface/os_interface.h" #include "shared/source/os_interface/product_helper.h" #include "shared/source/os_interface/windows/wddm/wddm.h" -#include "shared/source/release_helper/release_helper.h" #include "shared/test/common/helpers/debug_manager_state_restore.h" #include "shared/test/common/mocks/mock_execution_environment.h" #include "shared/test/common/test_macros/hw_test.h" @@ -66,23 +65,6 @@ TEST_F(ProductHelperTestWindows, givenCorrectParametersWhenConfiguringHwInfoThen EXPECT_EQ(outHwInfo.capabilityTable.ftrSvm, ftrSvm); } -TEST_F(ProductHelperTestWindows, givenCorrectParametersWhenConfiguringHwInfoThenSetXe2CompressionCorrectly) { - - auto releaseHelper = rootDeviceEnvironment->getReleaseHelper(); - if (releaseHelper) { - outHwInfo.featureTable.flags.ftrXe2Compression = !releaseHelper->getFtrXe2Compression(); - } - - int ret = productHelper->configureHwInfoWddm(&pInHwInfo, &outHwInfo, *rootDeviceEnvironment.get()); - ASSERT_EQ(0, ret); - - if (releaseHelper) { - EXPECT_EQ(releaseHelper->getFtrXe2Compression(), outHwInfo.featureTable.flags.ftrXe2Compression); - } else { - EXPECT_EQ(false, outHwInfo.featureTable.flags.ftrXe2Compression); - } -} - TEST_F(ProductHelperTestWindows, givenInstrumentationForHardwareIsEnabledOrDisabledWhenConfiguringHwInfoThenOverrideItUsingHaveInstrumentation) { int ret;