search

JamesCJ60 / Universal-x86-Tuning-Utility

Unlock the full potential of your Intel/AMD based device.
GNU General Public License v3.0
1.54k stars 38 forks source link

[BUG] Skin Temperature Limit (apu-skin-temp) value cannot be set properly #68

Closed Snuupy closed 1 year ago

Snuupy commented 1 year ago

Describe the bug

Skin Temperature Limit (apu-skin-temp) is set to a nonsense value

To Reproduce Steps to reproduce the behaviour:

  1. Go to custom presets
  2. Set skin temp limit to something reasonable, such as 60C (or manually via ryzenadj --apu-skin-temp 60
  3. Apply, run ryzenadj --info
  4. See: | STT LIMIT APU | 0.407 | apu-skin-temp |

Expected behaviour

apu-skin-temp should be set to 60.

Screenshots

N/A

Desktop (please complete the following information):

JamesCJ60 commented 1 year ago

Hi, I have not been able to reproduce this issue on my end, so this is possibly just a reporting issue.

Snuupy commented 1 year ago

Not a reporting issue, but it's because the ThinkPad T14 AMD BIOS keeps reverting the TDP changes back which causes throttling. I have had to run this instead in a loop: while (1) {C:\path\ryzenadj-win64\ryzenadj -a 25000 -b 25000 -c 25000 --apu-skin-temp 60 --tctl-temp=86; sleep 60}

JamesCJ60 commented 1 year ago

Auto reapply in the settings menu exists for this reason.

Snuupy commented 9 months ago

Auto reapply in the settings menu exists for this reason.

You are correct, I did not grok/understand the flags to a sufficient degree (and also the BIOS resetting the limits every "once in a while" is very confusing/opaque/hard to debug

Thank you for all the releases and allowing me to set my laptop's TDPs! It's a shame Lenovo sets such low limits by default.