- Remove stats monitoring for sub-sensors since they don't denote a
particular hotspot.
- Remove stats monitoring for wingboard configs.
- For vskin-legacy, vskin-model, vskin-speaker improve the resolution of
buckets by splitting range [49, 55] into [49, 51, 53, 55]
Bug: 341872098
Test: Push config and verify lshal
Change-Id: Iabff63922c283a595fa7ec2dfa8af539629d4064
- generated legacy formulas with DVT and CUJ data
- removed 'battery' thermistor from sensor combination list
- removed 'LINEAR' formulas
Bug: 342227291
Test: verified thermal hal service starts properly with it
Change-Id: Iff59072d432ff98afe852295745cebd81ec73d15
Signed-off-by: Jikai Ma <Jikai@google.com>
With previous samples being used in vt estimation, the sampling period
between vt estimation calls impact vt accuracy. Currently vt model is
using default cache interval of 2 seconds. This change sets the cache
interval to 7 seconds explicitly to match with passive delay
Bug: 341967304
Test: Check vt estimation invoke calls with frequency thermalservice dump
Change-Id: Ic6a38939b2bec88a66c87517e6efa548566ea877
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Due to defalut passive delay of 2 seconds being used for the VT sensor,
previous samples are ignored at 7 seconds intervals. This change sets
explicit passive_delay for VT sensor to make sure we are not ignoring
previous samples and apply step_ratio
Bug: 342257359
Test: Check VIRTUAL-SKIN-MODEL-UPDATED values at run time
Change-Id: I18a9e4913b8153b8d2c3a313451338a9f1b6ca0c
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Bug: 297083757
Test: Tested on CM4 boot to home and with thermal recorder
Change-Id: Ia6071a8c70e19cebeab909ca6c3e36138c93badc
Signed-off-by: Ramya Subramanian <rsubr@google.com>
Enabling step ration for vt model would dampen the vt which seems
to be very sensitive to some of the thermistor changes
Bug: 338284524
Test: Build and boot on device. Monitor Virtual Temperature
Change-Id: Ia45f683e3bef9dc1becf74c683c408ffdecf6e48
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Reduce temperature and charging throttling overshoots by fine-tuning PID
parameters.
Bug: 303547723
Test: boot and charging thermal throtlting worked properly.
Change-Id: Icaeef76998347167976a1ef3606f417db3c5a380
For mmWAVE test cases, legacy equation is under predicting heavily
resulting in virtua-skin switching to vt legacy even though the
model output was more accurate. This change increases the threshold
to address the same
Bug: 338284524
Test: Build and boot on device
Change-Id: I1c7a14a1d9042a2370c30490969d29294fc95f79
Signed-off-by: Sayanna Chandula <sayanna@google.com>
- Update VT models trained with dvt and cuj data
- Use backup sensor during undersampling
- Remove battery thermistor from VT model prediction
Bug: 337512118
Bug: 338596757
Test: Build and boot on device. Check VT through dumpsys
adb shell dumpsys thermalservice
Change-Id: Id26238f7baf58e23ef5e1c771d2ee6cf8968ec8e
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Replacec cdevceiling and hardlimit information with freuqencies in
thermal config
Bug: 336698561
Test: Flash Build and boot to home, run adb dumpsys android.hardware.thermal.IThermal/default, check the cdevceiling and hardlimit number for gpu
Change-Id: If5d105208ea055230dbbf4b6ecbac799b04a18db
Signed-off-by: Peter (YM) <peterym@google.com>
Expose virtual charge sensors to which throttling is binded to and
hide those which don't have.
Unhide:
- VIRTUAL-SKIN-CHARGE-WIRED
- VIRTUAL-SKIN-CHARGE-PERSIST
Hide:
- VIRTUAL-SKIN-CHARGE
Bug: 336193005
Test: Verify dumpsys
Change-Id: I4d83f6af17fccb7de8d053c9d981163c9dde0ea3
Bug: 320714468
Test: side load to device and successfully start thermal hal service.
Change-Id: I8189d7d7c94d590cc06ac82893d6b3dfd730585a
Signed-off-by: Jikai Ma <jikai@google.com>
- Remove backup sensor and enable under sampling
- Clamp VT_MODEL prediction based on training set max_surface
- Use VT_MODEL_LEGACY_DIFF to decide on switch
- Enable smooth transition with weights
Bug: 336265406
Test: Build and boot on device. Test smooth transition
Change-Id: If30a87a24b12b0af5527d9c05be7a14bb06dcfac
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Test: Boot to home and flash on device
Bug: 330780045
Change-Id: I428e14f354a5897601cbcaf8f7bbbeb6a0cc5d8f
Signed-off-by: Peter (YM) <peterym@google.com>
- add tflite model files
- add lstm model sensors to thermal config
- add lstm model sensor to virtual skin prediction
Bug: 308179047
Test: build and validate thermal service
Change-Id: Iaff613d75b3766bfbdf40b391e453a94172872a3
Signed-off-by: Jikai Ma <jikai@google.com>
Reduce threshold to 7C to gather more traces to analyze
Bug: 333110670
Test: Build and boot on device
Change-Id: I2a9ff381055fedaea18791fc030832a8b0877105
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Given that we have very few traces/events uploaded for 12C threshold,
this change reduces the threshold to 10C to help collect more data
to analyze infield abnormalities.
Bug: 333110670
Test: Build and boot on device
Change-Id: Icd3357e543bfd612b856d843a7cf56486361ed81
Signed-off-by: Sayanna Chandula <sayanna@google.com>
- Models prepared with both PD and Cert data from EVT1.1 tests
- Used leaky_relu activation
Bug: 332971287
Test: Build and boot on device
Change-Id: I07fea17ab21d6efd66d1d74629886051ea92aa94
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Remove the below power rails as they're going to be replaced with
different ones for better ODPM coverage.
- S8S_VDD_G3D_L2
- L9S_GNSS_CORE
Bug: 331556280
Test: Push config and verify thermal logs
Change-Id: Ic5c6cb7c4ddae258c89cbe3cb50a104f7044e016
Change the rule to identify wireless charge according to the latest
hda definition from bms driver pa/2736243
Bug: 324267311
Test: boot to home
Change-Id: Ice9b38201cbbe868784ac389e799cc24e8f8b7ff
We already have the CPU scheduler optimization for game mode to sustain
the power and perf, so remove the CPU thermal policy at lower
temperature to avoid the interfence on CPU scheduler.
Bug: 329697728
Test: run genshin
Change-Id: Ib0d47b3de7b3550a3af4db37ef062f8c5d2ea262
Latest ML models have been trained for latest HW and are no longer
valid for proto devices. We can use legacy equation prepared for
proto devices and remove any stats upload to avoid polluting stats.
Bug: 320714468
Test: Build and boot on proto devices. Check Virtual Skin output
Change-Id: Ic80934f0df80a423d8ba2cb4d1ac4b45999803fe
Signed-off-by: Sayanna Chandula <sayanna@google.com>
Update tflite models with metadata containing input data ranges.
Enable input data validation and backup sesor.
Add offsets based off temperature ranges.
Bug: 319167815
Bug: 328789394
Test: Build and boot on device. Test backup sensor and offsets
Change-Id: Icdc0c1a1fe026613e4a1716747335e60aca4edf9
Signed-off-by: Sayanna Chandula <sayanna@google.com>
- vs formulas generated from EVT 1.1 data
Bug: 320714468
Test: push to device and boot to home
Change-Id: Ife745ac7640ffd284404212706c9bdfdb0500f33
Signed-off-by: Jikai Ma <jikai@google.com>
(cherry picked from commit 859dc39412)
Inefficient OPPs were removed for the LITTLE cores from em table.
This change is to update the cdev ceilings in thermal policy to reflect
the EM changes
Bug: 329876112
Test: Build and boot on device.
Change-Id: I8b9ce12fc22dce0374d2a68a947bd598fd26f936
Signed-off-by: Ramya Subramanian <rsubr@google.com>
tz_temp callback will invoke thermal_zone_device_update and return the cached temp from thermal core. However, during device resume, the cached temp may be reset to INVALID number before being returned.
So, change the reading path to upstream temp_show which will return the temperature from thermistor driver directly.
Bug: 326252555
Test: adb shell dumpsys thermalservice
Change-Id: I82cf81d940585c4827256804bc9c9397f4b54ccf