Add a way to disable auto prefer fit for launch.

This change introduces "is APF enabled" system properties, and
uses them as trigger conditions for APF nodes / pre-APF nodes.

By default APF nodes are active, and pre-APF nodes are inactive,
which maintains the situation before this change.

A future experiment will flip "is APF enabled" system properties,
deactivating APF nodes and activating pre-APF nodes. For more
info see go/pixel-perf-apf-experiment.

Bug: 365855872
Test: set vendor.powerhal.apf_enabled=false, vendor.powerhal.apf_disabled=true, observe auto_prefer_fit
procfs value is never "1"
Flag: NONE powerhint change does not support flags
Change-Id: I919ad3ffe5f496df75bcb492bd3b4fa2285df65c
Signed-off-by: Dmitry Skiba <dskiba@google.com>
This commit is contained in:
Dmitry Skiba 2024-11-27 19:15:44 +00:00 committed by Michael Bestas
parent f466d06608
commit 00f64bf09b
No known key found for this signature in database
2 changed files with 33 additions and 8 deletions

View file

@ -437,6 +437,11 @@ PRODUCT_COPY_FILES += \
PRODUCT_COPY_FILES += \ PRODUCT_COPY_FILES += \
device/google/tegu/bluetooth/audio_set_configurations.json:$(TARGET_COPY_OUT_VENDOR)/etc/aidl/le_audio/aidl_audio_set_configurations.json device/google/tegu/bluetooth/audio_set_configurations.json:$(TARGET_COPY_OUT_VENDOR)/etc/aidl/le_audio/aidl_audio_set_configurations.json
# Enable APF by default
PRODUCT_VENDOR_PROPERTIES += \
vendor.powerhal.apf_disabled=false \
vendor.powerhal.apf_enabled=true
#Component Override for Pixel Troubleshooting App #Component Override for Pixel Troubleshooting App
PRODUCT_COPY_FILES += \ PRODUCT_COPY_FILES += \
device/google/tegu/tegu-component-overrides.xml:$(TARGET_COPY_OUT_VENDOR)/etc/sysconfig/tegu-component-overrides.xml device/google/tegu/tegu-component-overrides.xml:$(TARGET_COPY_OUT_VENDOR)/etc/sysconfig/tegu-component-overrides.xml

View file

@ -1088,6 +1088,13 @@
"Duration": 5000, "Duration": 5000,
"Value": "512" "Value": "512"
}, },
{
"PowerHint": "LAUNCH",
"Node": "TAUClampBoost",
"Duration": 5000,
"Value": "765",
"EnableProperty": "vendor.powerhal.apf_disabled"
},
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "FGUClampBoost", "Node": "FGUClampBoost",
@ -1162,7 +1169,8 @@
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "TAAutoPreferFit", "Node": "TAAutoPreferFit",
"Duration": 5000, "Duration": 5000,
"Value": "1" "Value": "1",
"EnableProperty": "vendor.powerhal.apf_enabled"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
@ -1170,35 +1178,47 @@
"Duration": 5000, "Duration": 5000,
"Value": "0xff" "Value": "0xff"
}, },
{
"PowerHint": "LAUNCH",
"Node": "TAPreferredIdleMaskMid",
"Duration": 5000,
"Value": "0xff",
"EnableProperty": "vendor.powerhal.apf_disabled"
},
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "TAUclampMinOnNiceEnable", "Node": "TAUclampMinOnNiceEnable",
"Duration": 5000, "Duration": 5000,
"Value": "1" "Value": "1",
"EnableProperty": "vendor.powerhal.apf_enabled"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "TAUclampMinOnNiceHighValue", "Node": "TAUclampMinOnNiceHighValue",
"Duration": 5000, "Duration": 5000,
"Value": "765" "Value": "765",
"EnableProperty": "vendor.powerhal.apf_enabled"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "TAUclampMinOnNiceHighPrio", "Node": "TAUclampMinOnNiceHighPrio",
"Duration": 5000, "Duration": 5000,
"Value": "110" "Value": "110",
"EnableProperty": "vendor.powerhal.apf_enabled"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "TAUclampMinOnNiceMidValue", "Node": "TAUclampMinOnNiceMidValue",
"Duration": 5000, "Duration": 5000,
"Value": "590" "Value": "590",
"EnableProperty": "vendor.powerhal.apf_enabled"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",
"Node": "TAUclampMinOnNiceMidPrio", "Node": "TAUclampMinOnNiceMidPrio",
"Duration": 5000, "Duration": 5000,
"Value": "139" "Value": "139",
"EnableProperty": "vendor.powerhal.apf_enabled"
}, },
{ {
"PowerHint": "LAUNCH", "PowerHint": "LAUNCH",