Remove O(n) lookup in command container destructor

Signed-off-by: Szymon Morek <[email protected]>

Author: smorek-intel

shared/source/command_container/cmdcontainer.cpp

@@ -254,9 +254,8 @@ void CommandContainer::handleCmdBufferAllocations(size_t startIndex) {
             if (isHandleFenceCompletionRequired) {
                 this->device->getMemoryManager()->handleFenceCompletion(cmdBufferAllocations[i]);
             }
-            if (!this->reusableAllocationList->peekContains(*cmdBufferAllocations[i])) {
-                reusableAllocationList->pushFrontOne(*cmdBufferAllocations[i]);
-            }
+
+            reusableAllocationList->pushFrontOne(*cmdBufferAllocations[i]);
         } else {
             this->device->getMemoryManager()->freeGraphicsMemory(cmdBufferAllocations[i]);
         }

shared/test/unit_test/command_container/command_container_tests.cpp

@@ -879,20 +879,6 @@ TEST_F(CommandContainerTest, givenCmdContainerWhenReuseExistingCmdBufferWithoutA
     allocList.freeAllGraphicsAllocations(pDevice);
 }
 
-TEST_F(CommandContainerTest, givenCmdContainerWhenDestroyCommandContainerThenAllocationListFilledCorrectly) {
-    auto cmdContainer = std::make_unique<CommandContainer>();
-    AllocationsList allocList;
-    cmdContainer->initialize(pDevice, &allocList, false);
-    auto alloc = cmdContainer->getCmdBufferAllocations()[0];
-    allocList.pushFrontOne(*alloc);
-    cmdContainer.reset();
-
-    EXPECT_TRUE(allocList.peekContains(*alloc));
-    EXPECT_EQ(allocList.peekHead()->countThisAndAllConnected(), 1u);
-
-    allocList.freeAllGraphicsAllocations(pDevice);
-}
-
 HWTEST_F(CommandContainerTest, givenCmdContainerWhenReuseExistingCmdBufferWithAllocationInListAndCsrTaskCountLowerThanAllocationThenReturnNullptr) {
     auto cmdContainer = std::make_unique<CommandContainer>();
     auto &csr = pDevice->getUltCommandStreamReceiver<FamilyType>();