Skip to content

Commit 367c8f2

Browse files
authored
Improve safety of macOS available RAM calcuations at suggestion of Ansariel and Beq (#2423)
1 parent 600ec6f commit 367c8f2

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

indra/llcommon/llmemory.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ void LLMemory::updateMemoryInfo()
129129
{
130130
// Our Windows definition of PagefileUsage is documented by Microsoft as "the total amount of
131131
// memory that the memory manager has committed for a running process", which is rss.
132-
sAllocatedPageSizeInKB = U32Bytes(info.resident_size);
132+
sAllocatedPageSizeInKB = U32Kilobytes::convert(U64Bytes(info.resident_size));
133133

134134
// Activity Monitor => Inspect Process => Real Memory Size appears to report resident_size
135135
// Activity monitor => main window memory column appears to report phys_footprint, which spot checks as at least 30% less.
@@ -139,7 +139,7 @@ void LLMemory::updateMemoryInfo()
139139
// reported for the app by the Memory Monitor in Instruments.' It is still about 8% bigger than phys_footprint.
140140
//
141141
// (On Windows, we use WorkingSetSize.)
142-
sAllocatedMemInKB = U32Bytes(info.resident_size - info.reusable);
142+
sAllocatedMemInKB = U32Kilobytes::convert(U64Bytes(info.resident_size - info.reusable));
143143
}
144144
else
145145
{

0 commit comments

Comments
 (0)