From 3fa5504a5d275aaefebd1b597fea1b98171e5f84 Mon Sep 17 00:00:00 2001
From: Compute-Runtime-Validation <compute-runtime-validation@intel.com>
Date: Sat, 21 Sep 2024 03:53:47 +0200
Subject: [PATCH] Revert "performance: Override cacheable for
 timestampPacketTagBuffer"

This reverts commit ec75f8f6868db79bdb0fb4bd8b5d566b0bc9dd5b.

Signed-off-by: Compute-Runtime-Validation <compute-runtime-validation@intel.com>
---
 shared/source/os_interface/product_helper.inl              | 7 +++----
 .../xe2_hpg_core/lnl/product_helper_tests_lnl.cpp          | 3 +--
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/shared/source/os_interface/product_helper.inl b/shared/source/os_interface/product_helper.inl
index c6186bbf80..f0bb1a79b4 100644
--- a/shared/source/os_interface/product_helper.inl
+++ b/shared/source/os_interface/product_helper.inl
@@ -453,8 +453,8 @@ bool ProductHelperHw<gfxProduct>::overridePatToUCAndTwoWayCohForDcFlushMitigatio
     return this->isDcFlushMitigated() &&
            (this->overrideCacheableForDcFlushMitigation(allocationType) ||
             allocationType == AllocationType::timestampPacketTagBuffer ||
-            allocationType == AllocationType::gpuTimestampDeviceBuffer ||
-            allocationType == AllocationType::tagBuffer);
+            allocationType == AllocationType::tagBuffer ||
+            allocationType == AllocationType::gpuTimestampDeviceBuffer);
 }
 
 template <PRODUCT_FAMILY gfxProduct>
@@ -473,8 +473,7 @@ bool ProductHelperHw<gfxProduct>::overrideCacheableForDcFlushMitigation(Allocati
             allocationType == AllocationType::svmCpu ||
             allocationType == AllocationType::svmZeroCopy ||
             allocationType == AllocationType::internalHostMemory ||
-            allocationType == AllocationType::printfSurface ||
-            allocationType == AllocationType::timestampPacketTagBuffer);
+            allocationType == AllocationType::printfSurface);
 }
 
 template <PRODUCT_FAMILY gfxProduct>
diff --git a/shared/test/unit_test/xe2_hpg_core/lnl/product_helper_tests_lnl.cpp b/shared/test/unit_test/xe2_hpg_core/lnl/product_helper_tests_lnl.cpp
index 5c21c91026..719c4cd22b 100644
--- a/shared/test/unit_test/xe2_hpg_core/lnl/product_helper_tests_lnl.cpp
+++ b/shared/test/unit_test/xe2_hpg_core/lnl/product_helper_tests_lnl.cpp
@@ -146,8 +146,7 @@ LNLTEST_F(LnlProductHelper, givenExternalHostPtrWhenMitigateDcFlushThenOverrideC
             allocationType == AllocationType::svmZeroCopy ||
             allocationType == AllocationType::internalHostMemory ||
             allocationType == AllocationType::commandBuffer ||
-            allocationType == AllocationType::printfSurface ||
-            allocationType == AllocationType::timestampPacketTagBuffer) {
+            allocationType == AllocationType::printfSurface) {
             EXPECT_TRUE(productHelper->overrideAllocationCacheable(allocationData));
         } else {
             EXPECT_FALSE(productHelper->overrideAllocationCacheable(allocationData));