Add additional pipe control arguments check

Signed-off-by: Rafal Maziejuk <[email protected]>

Author: RafalMaziejukIntel

opencl/test/unit_test/helpers/hw_helper_tests.cpp

@@ -1350,3 +1350,9 @@ HWTEST_F(HwHelperTest, givenRcsOrCcsEnabledWhenQueryingGpgpuEngineCountThenRetur
     hwInfo.featureTable.ftrRcsNode = true;
     EXPECT_EQ(4u, HwHelper::getGpgpuEnginesCount(hwInfo));
 }
+
+using isXeHpCoreOrBelow = IsAtMostProduct<IGFX_XE_HP_SDV>;
+HWTEST2_F(HwHelperTest, givenXeHPAndBelowPlatformWhenCheckingIfAdditionalPipeControlArgsAreRequiredThenReturnFalse, isXeHpCoreOrBelow) {
+    auto &hwHelper = HwHelper::get(renderCoreFamily);
+    EXPECT_FALSE(hwHelper.additionalPipeControlArgsRequired());
+}

shared/source/helpers/hw_helper.h

@@ -151,6 +151,7 @@ class HwHelper {
     virtual size_t getSingleTimestampPacketSize() const = 0;
     virtual void applyAdditionalCompressionSettings(Gmm &gmm, bool isNotCompressed) const = 0;
     virtual void applyRenderCompressionFlag(Gmm &gmm, uint32_t isRenderCompressed) const = 0;
+    virtual bool additionalPipeControlArgsRequired() const = 0;
 
     static uint32_t getSubDevicesCount(const HardwareInfo *pHwInfo);
     static uint32_t getGpgpuEnginesCount(const HardwareInfo &hwInfo);
@@ -383,6 +384,8 @@ class HwHelperHw : public HwHelper {
 
     void applyRenderCompressionFlag(Gmm &gmm, uint32_t isRenderCompressed) const override;
 
+    bool additionalPipeControlArgsRequired() const override;
+
   protected:
     LocalMemoryAccessMode getDefaultLocalMemoryAccessMode(const HardwareInfo &hwInfo) const override;
 

shared/source/helpers/hw_helper_bdw_plus.inl

@@ -134,4 +134,9 @@ inline void MemorySynchronizationCommands<GfxFamily>::setPipeControlExtraPropert
 template <typename GfxFamily>
 bool MemorySynchronizationCommands<GfxFamily>::isPipeControlWArequired(const HardwareInfo &hwInfo) { return false; }
 
+template <typename GfxFamily>
+bool HwHelperHw<GfxFamily>::additionalPipeControlArgsRequired() const {
+    return false;
+}
+
 } // namespace NEO

shared/source/xe_hp_core/hw_helper_xe_hp_core.cpp

@@ -204,6 +204,11 @@ void MemorySynchronizationCommands<Family>::setCacheFlushExtraProperties(PipeCon
     args.hdcPipelineFlush = true;
 }
 
+template <>
+bool HwHelperHw<Family>::additionalPipeControlArgsRequired() const {
+    return false;
+}
+
 template class HwHelperHw<Family>;
 template class FlatBatchBufferHelperHw<Family>;
 template struct MemorySynchronizationCommands<Family>;