refactor: remove not needed code

Signed-off-by: Mateusz Jablonski <[email protected]>

Author: JablonskiMateusz

level_zero/core/source/cmdlist/cmdlist_hw.inl

@@ -293,8 +293,7 @@ ze_result_t CommandListCoreFamily<gfxCoreFamily>::initialize(Device *device, NEO
                                              getCsr(false) &&
                                              getCsr(false)->isAnyDirectSubmissionEnabled() &&
                                              !neoDevice->getExecutionEnvironment()->areMetricsEnabled() &&
-                                             neoDevice->getMemoryManager()->isLocalMemorySupported(neoDevice->getRootDeviceIndex()) &&
-                                             productHelper.isFlatRingBufferSupported();
+                                             neoDevice->getMemoryManager()->isLocalMemorySupported(neoDevice->getRootDeviceIndex());
 
     if (NEO::debugManager.flags.DirectSubmissionFlatRingBuffer.get() != -1) {
         createSecondaryCmdBufferInHostMem &= !!NEO::debugManager.flags.DirectSubmissionFlatRingBuffer.get();

level_zero/core/test/unit_tests/sources/cmdlist/test_cmdlist_5.cpp

@@ -881,7 +881,7 @@ HWTEST_F(CommandListCreate, givenImmediateCopyOnlySingleTileDirectSubmissionComm
     std::unique_ptr<L0::CommandList> commandList(CommandList::createImmediate(productFamily, device, &desc, false, NEO::EngineGroupType::copy, returnValue));
     ASSERT_NE(nullptr, commandList);
 
-    auto flatRingSupported = device->getHwInfo().featureTable.flags.ftrLocalMemory && device->getProductHelper().isFlatRingBufferSupported();
+    auto flatRingSupported = device->getHwInfo().featureTable.flags.ftrLocalMemory;
     EXPECT_EQ(reinterpret_cast<CmdContainerMock *>(&commandList->getCmdContainer())->secondaryCommandStreamForImmediateCmdList.get() != nullptr, flatRingSupported);
     if (flatRingSupported) {
         EXPECT_TRUE(MemoryPoolHelper::isSystemMemoryPool(reinterpret_cast<CmdContainerMock *>(&commandList->getCmdContainer())->secondaryCommandStreamForImmediateCmdList->getGraphicsAllocation()->getMemoryPool()));
@@ -906,7 +906,7 @@ HWTEST_F(CommandListCreate, givenMetricsImmediateCopyOnlySingleTileDirectSubmiss
 }
 
 HWTEST2_F(CommandListCreate, givenSecondaryCommandStreamForImmediateCmdListWhenCheckAvailableSpaceThenSwapCommandStreams, MatchAny) {
-    if (!device->getHwInfo().featureTable.flags.ftrLocalMemory || !device->getProductHelper().isFlatRingBufferSupported()) {
+    if (!device->getHwInfo().featureTable.flags.ftrLocalMemory) {
         GTEST_SKIP();
     }
     DebugManagerStateRestore restorer;
@@ -939,7 +939,7 @@ HWTEST2_F(CommandListCreate, givenSecondaryCommandStreamForImmediateCmdListWhenC
 }
 
 HWTEST2_F(CommandListCreate, givenNoSecondaryCommandStreamForImmediateCmdListWhenCheckAvailableSpaceThenNotSwapCommandStreams, MatchAny) {
-    if (!device->getHwInfo().featureTable.flags.ftrLocalMemory || !device->getProductHelper().isFlatRingBufferSupported()) {
+    if (!device->getHwInfo().featureTable.flags.ftrLocalMemory) {
         GTEST_SKIP();
     }
     DebugManagerStateRestore restorer;

shared/source/os_interface/product_helper.h

@@ -126,7 +126,6 @@ class ProductHelper {
     virtual bool is3DPipelineSelectWARequired() const = 0;
     virtual bool isStorageInfoAdjustmentRequired() const = 0;
     virtual bool isBlitterForImagesSupported() const = 0;
-    virtual bool isFlatRingBufferSupported() const = 0;
     virtual bool isPageFaultSupported() const = 0;
     virtual bool isKmdMigrationSupported() const = 0;
     virtual bool isDisableScratchPagesSupported() const = 0;

shared/source/os_interface/product_helper.inl

@@ -552,11 +552,6 @@ bool ProductHelperHw<gfxProduct>::isThreadArbitrationPolicyReportedWithScm() con
     return ProductHelperHw<gfxProduct>::getScmPropertyThreadArbitrationPolicySupport();
 }
 
-template <PRODUCT_FAMILY gfxProduct>
-bool ProductHelperHw<gfxProduct>::isFlatRingBufferSupported() const {
-    return true;
-}
-
 template <PRODUCT_FAMILY gfxProduct>
 bool ProductHelperHw<gfxProduct>::isCooperativeEngineSupported(const HardwareInfo &hwInfo) const {
     return false;

shared/source/os_interface/product_helper_hw.h

@@ -97,7 +97,6 @@ class ProductHelperHw : public ProductHelper {
     bool isGrfNumReportedWithScm() const override;
     bool isThreadArbitrationPolicyReportedWithScm() const override;
     bool isCopyBufferRectSplitSupported() const override;
-    bool isFlatRingBufferSupported() const override;
     bool isCooperativeEngineSupported(const HardwareInfo &hwInfo) const override;
     bool isTimestampWaitSupportedForEvents() const override;
     bool isTilePlacementResourceWaRequired(const HardwareInfo &hwInfo) const override;

shared/test/unit_test/os_interface/product_helper_tests.cpp

@@ -447,10 +447,6 @@ HWTEST2_F(ProductHelperTest, givenProductHelperWhenCallUseGemCreateExtInAllocate
     EXPECT_FALSE(productHelper->useGemCreateExtInAllocateMemoryByKMD());
 }
 
-HWTEST2_F(ProductHelperTest, givenProductHelperWhenCallIsFlatRingBufferSupportedThenTrueIsReturned, IsAtMostXeHpgCore) {
-    EXPECT_TRUE(productHelper->isFlatRingBufferSupported());
-}
-
 HWTEST_F(ProductHelperTest, givenProductHelperWhenAskedIfBlitterForImagesIsSupportedThenFalseIsReturned) {
 
     EXPECT_FALSE(productHelper->isBlitterForImagesSupported());