Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-05-10 20:27:45 +00:00
Code Issues Projects Releases Packages Wiki Activity

[NTOS:Mm] Bail out in MmTrimUserMemory, when all LRU pages have been looped through

Browse source
This commit is contained in:
Timo Kreuzer 2023-04-08 19:27:15 +03:00
parent 5f014e421b
commit d7de53b6d5
1 changed files with 23 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
ntoskrnl/mm/balance.c
View file

@ -138,14 +138,15 @@ MiTrimMemoryConsumer(ULONG Consumer, ULONG InitialTarget)
NTSTATUS NTSTATUS
MmTrimUserMemory(ULONG Target, ULONG Priority, PULONG NrFreedPages) MmTrimUserMemory(ULONG Target, ULONG Priority, PULONG NrFreedPages)
{ {
PFN_NUMBER CurrentPage; PFN_NUMBER FirstPage, CurrentPage;
NTSTATUS Status; NTSTATUS Status;
(*NrFreedPages) = 0; (*NrFreedPages) = 0;
DPRINT1("MM BALANCER: %s\n", Priority ? "Paging out!" : "Removing access bit!"); DPRINT1("MM BALANCER: %s\n", Priority ? "Paging out!" : "Removing access bit!");
CurrentPage = MmGetLRUFirstUserPage(); FirstPage = MmGetLRUFirstUserPage();
CurrentPage = FirstPage;
while (CurrentPage != 0 && Target > 0) while (CurrentPage != 0 && Target > 0)
{ {
if (Priority) if (Priority)
@ -156,6 +157,10 @@ MmTrimUserMemory(ULONG Target, ULONG Priority, PULONG NrFreedPages)
DPRINT("Succeeded\n"); DPRINT("Succeeded\n");
Target--; Target--;
(*NrFreedPages)++; (*NrFreedPages)++;
if (CurrentPage == FirstPage)
{
FirstPage = 0;
}
} }
} }
else else
@ -245,8 +250,14 @@ MmTrimUserMemory(ULONG Target, ULONG Priority, PULONG NrFreedPages)
/* Nobody accessed this page since the last time we check. Time to clean up */ /* Nobody accessed this page since the last time we check. Time to clean up */
Status = MmPageOutPhysicalAddress(CurrentPage); Status = MmPageOutPhysicalAddress(CurrentPage);
if (NT_SUCCESS(Status))
{
if (CurrentPage == FirstPage)
{
FirstPage = 0;
}
}
// DPRINT1("Paged-out one page: %s\n", NT_SUCCESS(Status) ? "Yes" : "No"); // DPRINT1("Paged-out one page: %s\n", NT_SUCCESS(Status) ? "Yes" : "No");
(void)Status;
} }
/* Done for this page. */ /* Done for this page. */
@ -254,6 +265,15 @@ MmTrimUserMemory(ULONG Target, ULONG Priority, PULONG NrFreedPages)
} }
CurrentPage = MmGetLRUNextUserPage(CurrentPage, TRUE); CurrentPage = MmGetLRUNextUserPage(CurrentPage, TRUE);
if (FirstPage == 0)
{
FirstPage = CurrentPage;
}
else if (CurrentPage == FirstPage)
{
DPRINT1("We are back at the start, abort!\n");
return STATUS_SUCCESS;
}
} }
if (CurrentPage) if (CurrentPage)