fix: iterate over each indirect allocation

Related-To: GSD-9450

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

Author: smorek-intel

shared/source/memory_manager/unified_memory_manager.cpp

@@ -743,7 +743,6 @@ void SVMAllocsManager::makeIndirectAllocationsResident(CommandStreamReceiver &co
     std::unique_lock<std::shared_mutex> lock(mtx);
     bool parseAllAllocations = false;
     auto entry = indirectAllocationsResidency.find(&commandStreamReceiver);
-    TaskCountType previousCounter = 0;
     if (entry == indirectAllocationsResidency.end()) {
         parseAllAllocations = true;
 
@@ -755,14 +754,13 @@ void SVMAllocsManager::makeIndirectAllocationsResident(CommandStreamReceiver &co
     } else {
         if (this->allocationsCounter > entry->second.latestResidentObjectId) {
             parseAllAllocations = true;
-            previousCounter = entry->second.latestResidentObjectId;
             entry->second.latestResidentObjectId = this->allocationsCounter;
         }
         entry->second.latestSentTaskCount = taskCount;
     }
     if (parseAllAllocations) {
         auto currentCounter = this->allocationsCounter.load();
-        for (auto allocationId = static_cast<uint32_t>(previousCounter + 1); allocationId <= currentCounter; allocationId++) {
+        for (auto allocationId = 1u; allocationId <= currentCounter; allocationId++) {
             makeResidentForAllocationsWithId(allocationId, commandStreamReceiver);
         }
     }

shared/test/unit_test/memory_manager/unified_memory_manager_tests.cpp

@@ -581,7 +581,7 @@ TEST_F(SVMLocalMemoryAllocatorTest, givenInternalAllocationWhenNewAllocationIsCr
     // now call with task count 2, first allocation shouldn't be modified
     svmManager->makeIndirectAllocationsResident(*csr, 2u);
 
-    EXPECT_FALSE(graphicsAllocation->gpuAllocations.getDefaultGraphicsAllocation()->isResident(csr->getOsContext().getContextId()));
+    EXPECT_TRUE(graphicsAllocation->gpuAllocations.getDefaultGraphicsAllocation()->isResident(csr->getOsContext().getContextId()));
     EXPECT_TRUE(graphicsAllocation2->gpuAllocations.getDefaultGraphicsAllocation()->isResident(csr->getOsContext().getContextId()));
 
     svmManager->freeSVMAlloc(ptr);