From 0fed542401ff180b3a609ed90f8717a759aa4c34 Mon Sep 17 00:00:00 2001 From: Rick Yiu Date: Wed, 24 May 2023 03:26:00 +0000 Subject: [PATCH 1/3] Restore uclamp.max for bg groups These values could indicate if cpu is running important tasks. Bug: 201260585 Test: build pass Change-Id: I222ba118f63d8c1a3285c882eb4e5940e666a02d --- conf/init.gs201.rc | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/conf/init.gs201.rc b/conf/init.gs201.rc index bec1a4fd..7c2d17b3 100644 --- a/conf/init.gs201.rc +++ b/conf/init.gs201.rc @@ -740,6 +740,12 @@ on property:sys.boot_completed=1 # Set bg group throttle write /proc/vendor_sched/ug_bg_group_throttle 308 + # Set uclamp.max for some groups, which could indicate cpu importance used in scheduling + write /proc/vendor_sched/bg_uclamp_max 512 + write /proc/vendor_sched/sysbg_uclamp_max 512 + write /proc/vendor_sched/ota_uclamp_max 512 + write /proc/vendor_sched/dex2oat_uclamp_max 615 + # Set PMU freq limit parameters write /sys/devices/system/cpu/cpufreq/policy0/sched_pixel/lcpi_threshold 2 write /sys/devices/system/cpu/cpufreq/policy0/sched_pixel/spc_threshold 70 From 2a83a22a6dfa1177c66a37a6d065c32f7ca7185d Mon Sep 17 00:00:00 2001 From: Donnie Pollitz Date: Wed, 17 May 2023 09:24:36 +0200 Subject: [PATCH 2/3] Make sure nsp storage has the right owner/group Background: * Not all secure storage files had the correct ownership transitions. Bug: 280325952 Test: Changed permissions and confirmed user transitions Change-Id: Ic76f312ec954e8afbbe95b3843356b8f92a1d4a7 Signed-off-by: Donnie Pollitz --- conf/init.gs201.rc | 1 + 1 file changed, 1 insertion(+) diff --git a/conf/init.gs201.rc b/conf/init.gs201.rc index 7c2d17b3..51ab29b3 100644 --- a/conf/init.gs201.rc +++ b/conf/init.gs201.rc @@ -904,6 +904,7 @@ on post-fs-data symlink /mnt/vendor/persist/ss /data/vendor/ss/persist chown root system /data/vendor/ss/persist chmod 0770 /data/vendor/ss/persist + chown system system /data/vendor/ss/persist/nsp restart storageproxyd From b67c8e14231de5e66813f44532eef4b43e497c21 Mon Sep 17 00:00:00 2001 From: Sam Dubey Date: Wed, 24 May 2023 10:40:48 +0000 Subject: [PATCH 3/3] Revert "Make sure nsp storage has the right owner/group" Revert submission 23280559-nsp Reason for revert: b/284107307 Reverted changes: /q/submissionid:23280559-nsp Change-Id: Ieaed6e8e130c1a443c48981d160dc20e06c2449b --- conf/init.gs201.rc | 1 - 1 file changed, 1 deletion(-) diff --git a/conf/init.gs201.rc b/conf/init.gs201.rc index 51ab29b3..7c2d17b3 100644 --- a/conf/init.gs201.rc +++ b/conf/init.gs201.rc @@ -904,7 +904,6 @@ on post-fs-data symlink /mnt/vendor/persist/ss /data/vendor/ss/persist chown root system /data/vendor/ss/persist chmod 0770 /data/vendor/ss/persist - chown system system /data/vendor/ss/persist/nsp restart storageproxyd