From 7064db2a19b23b9f18e39b1dda1fb967a2275fec Mon Sep 17 00:00:00 2001 From: Christopher Ferris Date: Fri, 19 Jul 2024 21:38:52 +0000 Subject: [PATCH] Replace PRODUCT_DISABLE_SCUDO with MALLOC_LOW_MEMORY. The PRODUCT_DISABLE_SCUDO only removes scudo from the list of sanitizers. It doesn't change the allocator to be used. Instead use MALLOC_LOW_MEMORY := true, which causes a device to use a low memory version of the allocator. Bug: 318756974 Test: Treehugger. Change-Id: Idaf57cfa4fa723ce8d7f285b2bad8ec29059f0df --- target/product/go_defaults_common.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/target/product/go_defaults_common.mk b/target/product/go_defaults_common.mk index 5218f29c0a..fd4047a65b 100644 --- a/target/product/go_defaults_common.mk +++ b/target/product/go_defaults_common.mk @@ -37,9 +37,9 @@ PRODUCT_ART_TARGET_INCLUDE_DEBUG_BUILD := false # leave less information available via JDWP. PRODUCT_MINIMIZE_JAVA_DEBUG_INFO := true -# Disable Scudo outside of eng builds to save RAM. +# Use the low memory allocator outside of eng builds to save RSS. ifneq (,$(filter eng, $(TARGET_BUILD_VARIANT))) - PRODUCT_DISABLE_SCUDO := true + MALLOC_LOW_MEMORY := true endif # Add the system properties.