summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Juan Yescas <jyescas@google.com> 2023-05-09 12:08:02 -0700
committer Juan Yescas <jyescas@google.com> 2023-05-09 22:19:05 +0000
commit34f1eab423dcf6ab29d01b84bf3b99621a75e2e8 (patch)
tree841b96dde4ca5509ed794e3dc00172431a97853d
parent29fd4cf2d8404a87591dfb0154fd0f91f20083df (diff)
Return TotalMemory size when ddr size is not present
When the ddr size is not present or it is invalid, getAdvertisedMem() must return getTotalMemory() size. Bug: 278977257 Test: atest frameworks/base/core/tests/coretests/src/android/os/ProcessTest.java Merged-In: I2323d79d869a4c58eb29d0cae351144f357b867d Change-Id: I2323d79d869a4c58eb29d0cae351144f357b867d
-rw-r--r--core/java/android/os/Process.java2
-rw-r--r--core/tests/coretests/src/android/os/ProcessTest.java1
2 files changed, 2 insertions, 1 deletions
diff --git a/core/java/android/os/Process.java b/core/java/android/os/Process.java
index bf3d52d358ed..04525e8b8ff7 100644
--- a/core/java/android/os/Process.java
+++ b/core/java/android/os/Process.java
@@ -1359,7 +1359,7 @@ public class Process {
String formatSize = MemoryProperties.memory_ddr_size().orElse("0KB");
long memSize = FileUtils.parseSize(formatSize);
- if (memSize == Long.MIN_VALUE) {
+ if (memSize <= 0) {
return FileUtils.roundStorageSize(getTotalMemory());
}
diff --git a/core/tests/coretests/src/android/os/ProcessTest.java b/core/tests/coretests/src/android/os/ProcessTest.java
index 52846dfbb14b..b2ffdc035e8b 100644
--- a/core/tests/coretests/src/android/os/ProcessTest.java
+++ b/core/tests/coretests/src/android/os/ProcessTest.java
@@ -73,6 +73,7 @@ public class ProcessTest extends TestCase {
}
public void testGetAdvertisedMem() {
+ assertTrue(Process.getAdvertisedMem() > 0);
assertTrue(Process.getTotalMemory() <= Process.getAdvertisedMem());
}
}