From 0074216f27114ab7e077bb7c68a83eafef89b042 Mon Sep 17 00:00:00 2001
From: Cheng Chang <chengcha@google.com>
Date: Tue, 5 Nov 2024 10:03:58 +0000
Subject: [PATCH] gps: Enable coredump report for user ROM

In the user ROM, coredump file won't upload to server. Only for crash info statistic.

flag: EXEMPT the function has been verified at userdebug ROM.
Bug: 378004800
Bug: 377446770
Test: b/378004800 abtd to check sepolicy
Test: b/377446770#comment1 verified the coredump function on user ROM.
Change-Id: I917f494eac9e5a185dc503880d4f7d695e8651f3
---
 location/gnssd/Android.bp           | 19 ++++---------------
 location/gnssd/gnss_release.mk      |  2 ++
 location/gnssd/release/gps_user.cfg |  2 +-
 3 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/location/gnssd/Android.bp b/location/gnssd/Android.bp
index 1774bd7..f7fb192 100644
--- a/location/gnssd/Android.bp
+++ b/location/gnssd/Android.bp
@@ -42,14 +42,9 @@ cc_prebuilt_binary {
     // Bypass because libhidltransport is deprecated
     // Bypass because libhwbinder is deprecated
     check_elf_files: false,
-    product_variables: {
-        debuggable: {
-            required: [
-                "libsighandler",
-            ],
-        },
-    },
-
+    required: [
+        "libsighandler",
+    ],
     init_rc: ["init.gnss.rc"],
     vintf_fragments: ["android.hardware.gnss@default.xml"],
 }
@@ -86,14 +81,8 @@ cc_prebuilt_binary {
     },
     // Bypass because libsitril-gps is Android.mk module
     check_elf_files: false,
-    product_variables: {
-        debuggable: {
-            required: [
-                "libsighandler",
-            ],
-        },
-    },
     required: [
         "libcustomgnss",
+        "libsighandler",
     ],
 }
diff --git a/location/gnssd/gnss_release.mk b/location/gnssd/gnss_release.mk
index 452d7d9..3e606fb 100644
--- a/location/gnssd/gnss_release.mk
+++ b/location/gnssd/gnss_release.mk
@@ -1,3 +1,5 @@
+# Enable coredump funtcion for all the ROM.
+$(call soong_config_set_bool,sighandler,coredump,true)
 # Enable pixel gnss hal service
 -include vendor/google/gnss/aidl_service/pixel_gnss_hal.mk
 
diff --git a/location/gnssd/release/gps_user.cfg b/location/gnssd/release/gps_user.cfg
index 64aaa63..6dd8e4a 100644
--- a/location/gnssd/release/gps_user.cfg
+++ b/location/gnssd/release/gps_user.cfg
@@ -1,7 +1,7 @@
 GlueLayer_ToolConfigSelection=3
 debug_console=0
 debug_enable=0
-mem_dump_to_node=0
+mem_dump_to_node=1
 GlueLayer_IsGedKeyExist=1
 GlueLayer_EnableGnssCfgInterface=1
 ism_stream_enable=1