From ab9faee1930f564f3cf2708f848d2c1490d1ccad Mon Sep 17 00:00:00 2001
From: Paulsu Su <paulsu@google.com>
Date: Mon, 13 Jan 2025 17:36:23 -0800
Subject: [PATCH] powerhint: Mask hints in AA and DISPLAY_IDLE

This CL adds a powerhint for AA mode to mask storming hint. DISPLAY_IDLE masks DISPLAY_UPDATE_IMMINENT and CPU_LOAD_RESET when Android Auto property is set. When navigation in Android Auto, the two hint with action to boost memlat is costly.

Bug: 380188115
Change-Id: I825a56ac32a0ecdc8b441af9cb64e81b1d04f9a6
Test: Local test with desktop_head_unit_internal
Flag: EXEMPT powerhint change
---
 perf/powerhint-caiman.json | 28 ++++++++++++++++++++++++++++
 perf/powerhint-komodo.json | 28 ++++++++++++++++++++++++++++
 perf/powerhint-tokay.json  | 28 ++++++++++++++++++++++++++++
 3 files changed, 84 insertions(+)

diff --git a/perf/powerhint-caiman.json b/perf/powerhint-caiman.json
index 3670723a..a860b204 100644
--- a/perf/powerhint-caiman.json
+++ b/perf/powerhint-caiman.json
@@ -993,6 +993,16 @@
       "WriteOnly": true,
       "ResetOnInit": false
     },
+    {
+      "Name": "AAModeProperty",
+      "Path": "vendor.powerhal.mode.aa",
+      "Values": [
+        "1",
+        "0"
+      ],
+      "ResetOnInit": true,
+      "Type": "Property"
+    },
     {
       "Name": "KswapdUclaimMin",
       "Path": "/sys/kernel/vendor_mm/kswapd_uclamp_min",
@@ -3213,6 +3223,12 @@
       "Duration": 0,
       "Value": "1"
     },
+    {
+      "PowerHint": "AUTOMOTIVE_PROJECTION",
+      "Node": "AAModeProperty",
+      "Duration": 0,
+      "Value": "1"
+    },
     {
       "PowerHint": "DISPLAY_IDLE",
       "Node": "FGPreferIdle",
@@ -3254,6 +3270,18 @@
       "Duration": 0,
       "Value": "170"
     },
+    {
+      "PowerHint": "DISPLAY_IDLE",
+      "Type": "MaskHint",
+      "Value":"DISPLAY_UPDATE_IMMINENT",
+      "EnableProperty": "vendor.powerhal.mode.aa"
+    },
+    {
+      "PowerHint": "DISPLAY_IDLE",
+      "Type": "MaskHint",
+      "Value":"CPU_LOAD_RESET",
+      "EnableProperty": "vendor.powerhal.mode.aa"
+    },
     {
       "PowerHint": "DISPLAY_UPDATE_IMMINENT",
       "Node": "CPULittleClusterDownRateLimitUs",
diff --git a/perf/powerhint-komodo.json b/perf/powerhint-komodo.json
index c7512d89..2a0fbef1 100644
--- a/perf/powerhint-komodo.json
+++ b/perf/powerhint-komodo.json
@@ -993,6 +993,16 @@
       "WriteOnly": true,
       "ResetOnInit": false
     },
+    {
+      "Name": "AAModeProperty",
+      "Path": "vendor.powerhal.mode.aa",
+      "Values": [
+        "1",
+        "0"
+      ],
+      "ResetOnInit": true,
+      "Type": "Property"
+    },
     {
       "Name": "KswapdUclaimMin",
       "Path": "/sys/kernel/vendor_mm/kswapd_uclamp_min",
@@ -3213,6 +3223,12 @@
       "Duration": 0,
       "Value": "1"
     },
+    {
+      "PowerHint": "AUTOMOTIVE_PROJECTION",
+      "Node": "AAModeProperty",
+      "Duration": 0,
+      "Value": "1"
+    },
     {
       "PowerHint": "DISPLAY_IDLE",
       "Node": "FGPreferIdle",
@@ -3254,6 +3270,18 @@
       "Duration": 0,
       "Value": "170"
     },
+    {
+      "PowerHint": "DISPLAY_IDLE",
+      "Type": "MaskHint",
+      "Value":"DISPLAY_UPDATE_IMMINENT",
+      "EnableProperty": "vendor.powerhal.mode.aa"
+    },
+    {
+      "PowerHint": "DISPLAY_IDLE",
+      "Type": "MaskHint",
+      "Value":"CPU_LOAD_RESET",
+      "EnableProperty": "vendor.powerhal.mode.aa"
+    },
     {
       "PowerHint": "DISPLAY_UPDATE_IMMINENT",
       "Node": "CPULittleClusterDownRateLimitUs",
diff --git a/perf/powerhint-tokay.json b/perf/powerhint-tokay.json
index 985e4d63..87e86d51 100644
--- a/perf/powerhint-tokay.json
+++ b/perf/powerhint-tokay.json
@@ -993,6 +993,16 @@
       "WriteOnly": true,
       "ResetOnInit": false
     },
+    {
+      "Name": "AAModeProperty",
+      "Path": "vendor.powerhal.mode.aa",
+      "Values": [
+        "1",
+        "0"
+      ],
+      "ResetOnInit": true,
+      "Type": "Property"
+    },
     {
       "Name": "KswapdUclaimMin",
       "Path": "/sys/kernel/vendor_mm/kswapd_uclamp_min",
@@ -3213,6 +3223,12 @@
       "Duration": 0,
       "Value": "1"
     },
+    {
+      "PowerHint": "AUTOMOTIVE_PROJECTION",
+      "Node": "AAModeProperty",
+      "Duration": 0,
+      "Value": "1"
+    },
     {
       "PowerHint": "DISPLAY_IDLE",
       "Node": "FGPreferIdle",
@@ -3254,6 +3270,18 @@
       "Duration": 0,
       "Value": "170"
     },
+    {
+      "PowerHint": "DISPLAY_IDLE",
+      "Type": "MaskHint",
+      "Value":"DISPLAY_UPDATE_IMMINENT",
+      "EnableProperty": "vendor.powerhal.mode.aa"
+    },
+    {
+      "PowerHint": "DISPLAY_IDLE",
+      "Type": "MaskHint",
+      "Value":"CPU_LOAD_RESET",
+      "EnableProperty": "vendor.powerhal.mode.aa"
+    },
     {
       "PowerHint": "DISPLAY_UPDATE_IMMINENT",
       "Node": "CPULittleClusterDownRateLimitUs",