Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TLP 1.6 Beta] Call for testers #700

Closed
linrunner opened this issue Jul 21, 2023 · 27 comments
Closed

[TLP 1.6 Beta] Call for testers #700

linrunner opened this issue Jul 21, 2023 · 27 comments
Labels
Milestone

Comments

@linrunner
Copy link
Owner

Hi @ALL,

as with every upcoming release I need your help to complete the test coverage.

Test Objectives

Beta Packages
See the download page.

What do I need from the testers?
After installing please start the beta with

sudo tlp start

I'm expecting your unabbreviated outputs of

sudo tlp-stat

on battery and AC (via Gist please).

Bug Reports
If something does not work as expected → please open an issue.

Thanks for testing!

@arvl130
Copy link

arvl130 commented Jul 22, 2023

AC and Battery

@linrunner
Copy link
Owner Author

@arvl130 I am interested in the complete tlp-stat outputs, not only tlp-stat -b.

@arvl130
Copy link

arvl130 commented Jul 22, 2023

@linrunner Fixed now. Forgot I had it aliased.

@LinuxOnTheDesktop
Copy link

@LinuxOnTheDesktop
Copy link

LinuxOnTheDesktop commented Jul 23, 2023

I think that the following bit of configuration info is new.

CAUTION: USB and IEEE1394 disks may fail to mount or data may get corrupted
with APM enabled. Be careful and make sure you have backups of all affected
media before removing 'usb' or 'ieee1394' from the denylist!
Default: "usb ieee1394"

Can you tell me anything under the conditions under which APM can cause these problems? Are particular models of computer at issue?

EDITED to reduce the space taken up by the quote.

@IDeletedSystem64
Copy link

tlp-stat output

@linrunner
Copy link
Owner Author

linrunner commented Jul 23, 2023

@LinuxOnTheDesktop

I think that the following bit of configuration info is new.

Nothing new here. In fact the feature was introduced with 1.4 and it's disabled by default:

I included the warning as a precaution because of a rather old Ubuntu bug that is linked in the above commit. I have not observed any problems with APM on any of my USB disks. Many disks simply do not support APM. After all, you don't have to remove usb from of the denylist if unsure.

@linrunner
Copy link
Owner Author

@IDeletedSystem64 could you add the output on battery power?

@AminBlg
Copy link

AminBlg commented Jul 24, 2023

tlp-stat report

@tisyang
Copy link

tisyang commented Jul 25, 2023

tlp-stat report

My cpu is amd r7-7840hs, kernel 6.4, use 1.6 beta tlp, if choose CPU_DRIVER_OPMODE_ON_AC/BAT = guided, the cpu is locked 400 ~ 900 Mhz. But choose active, cpu frequency is fine.

choose guided in tlp

 sudo cpupower -c 0-15 frequency-info | grep "current CPU" | grep Hz
  current CPU frequency: 547 MHz (asserted by call to kernel)
  current CPU frequency: 544 MHz (asserted by call to kernel)
  current CPU frequency: 549 MHz (asserted by call to kernel)
  current CPU frequency: 541 MHz (asserted by call to kernel)
  current CPU frequency: 546 MHz (asserted by call to kernel)
  current CPU frequency: 549 MHz (asserted by call to kernel)
  current CPU frequency: 548 MHz (asserted by call to kernel)
  current CPU frequency: 547 MHz (asserted by call to kernel)
  current CPU frequency: 548 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 547 MHz (asserted by call to kernel)
  current CPU frequency: 549 MHz (asserted by call to kernel)
  current CPU frequency: 545 MHz (asserted by call to kernel)
  current CPU frequency: 545 MHz (asserted by call to kernel)
  current CPU frequency: 545 MHz (asserted by call to kernel)

choose guided in tlp

sudo cpupower -c 0-15 frequency-info | grep "current CPU" | grep Hz
  current CPU frequency: 1.31 GHz (asserted by call to kernel)
  current CPU frequency: 1.27 GHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 3.20 GHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 1.29 GHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 1.30 GHz (asserted by call to kernel)
  current CPU frequency: 1.54 GHz (asserted by call to kernel)
  current CPU frequency: 1.30 GHz (asserted by call to kernel)
  current CPU frequency: 3.21 GHz (asserted by call to kernel)
  current CPU frequency: 2.80 GHz (asserted by call to kernel)

If set kernel parameter amd_pstate=guided, CPU frequency is fine too.

@linrunner
Copy link
Owner Author

linrunner commented Jul 25, 2023

@tisyang seems the kernel driver doesn't like switching at runtime. TLP cannot do anything about it.

ps. you may try configuring the frequencies

CPU_SCALING_MIN_FREQ_ON_AC=400000
CPU_SCALING_MAX_FREQ_ON_AC=5293000
CPU_SCALING_MIN_FREQ_ON_BAT=400000
CPU_SCALING_MAX_FREQ_ON_BAT=5293000

ps2. on my machine switching to guided works, frequencies are not locked.

@tisyang
Copy link

tisyang commented Jul 25, 2023

@tisyang seems the kernel driver doesn't like switching at runtime. TLP cannot do anything about it.

ps. you may try configuring the frequencies

CPU_SCALING_MIN_FREQ_ON_AC=400000
CPU_SCALING_MAX_FREQ_ON_AC=5293000
CPU_SCALING_MIN_FREQ_ON_BAT=400000
CPU_SCALING_MAX_FREQ_ON_BAT=5293000

ps2. on my machine switching to guided works, frequencies are not locked.

configuring frequencies does not work for guided mode, cpu frequency is still locked, it maybe a kernel or bios bug. Anyway, I will use active mode.

@siddhpant
Copy link

siddhpant commented Jul 25, 2023

On AMD Ryzen 5500U with kernel 6.3 and amd-pstate=active (tlp-stat report), when I set CPU_SCALING_GOVERNOR_ON_* to performance, the EPP preference is set to performance regardless of CPU_ENERGY_PERF_POLICY_ON_* setting. This does not happen when the governor is powersave.

Manually setting the EPP preference results in the error bash: line 1: echo: write error: Device or resource busy. This does not happen when the governor is powersave.

@linrunner
Copy link
Owner Author

linrunner commented Jul 25, 2023

@siddhpant same on my machine (with 6.4.4). However, I think it is "works as designed" of the driver. I have read about this behavior elsewhere, I mean with intel_pstate.

Full throttle and brake at the same time doesn't seem to be provided. Makes sense, doesn't it?

ps. answer added to the FAQ.

@siddhpant
Copy link

@siddhpant same on my machine (with 6.4.4). However, I think it is "works as designed" of the driver. I have read about this behavior elsewhere, I mean with intel_pstate.

Full throttle and brake at the same time doesn't seem to be provided. Makes sense, doesn't it?

Yeah, that makes sense, thanks.

Just googled for confirmation, seems like it is the case on laptops [1], but on other hardware like EPYC, balance_performance is available [2] (but is probably none of our concern, and probably will work anyways).

Thanks for the release!


[1] https://www.phoronix.com/review/amd-pstate-epp-ryzen-mobile

[2] https://www.phoronix.com/review/linux-63-amd-epyc-epp

@linrunner
Copy link
Owner Author

linrunner commented Jul 25, 2023

@siddhpant

[1] https://www.phoronix.com/review/amd-pstate-epp-ryzen-mobile
the powersave mode with ACPI CPUFreq and passive AMD P-State was very slow.

I can only shake my head. If you make strange comparisons, you get strange results.

It has always been known that acpi-cpufreq/powersave slows down the system to a crawl instead of saving power. That's why ondemand has been recommended for ages, and schedutil nowadays. acpi-cpufreq/powersave is no valid candidate for comparisons with *_pstate/powersave.

@siddhpant
Copy link

siddhpant commented Jul 25, 2023

@linrunner Yes, I agree. It is an apples to oranges comparison. The only reason to do it is for completeness' sake (which is probably why I land up there when I search for p-state).

@linrunner
Copy link
Owner Author

@ALL thanks for your contributions so far.

@tisyang
Copy link

tisyang commented Jul 26, 2023

@linrunner

I found out, if

CPU_DRIVER_OPMODE_ON_AC=guided

CPU_SCALING_GOVERNOR_ON_AC=performance

The mode will be guided and CPU frequency is fine.

But

CPU_DRIVER_OPMODE_ON_AC=guided

CPU_SCALING_GOVERNOR_ON_AC=powersave

The mode will be guided, but CPU frequency will locked in 400~1000Mhz. This maybe not TLP's bug.

tlp-stat report

My cpu is amd r7-7840hs, kernel 6.4, use 1.6 beta tlp, if choose CPU_DRIVER_OPMODE_ON_AC/BAT = guided, the cpu is locked 400 ~ 900 Mhz. But choose active, cpu frequency is fine.

choose guided in tlp

 sudo cpupower -c 0-15 frequency-info | grep "current CPU" | grep Hz
  current CPU frequency: 547 MHz (asserted by call to kernel)
  current CPU frequency: 544 MHz (asserted by call to kernel)
  current CPU frequency: 549 MHz (asserted by call to kernel)
  current CPU frequency: 541 MHz (asserted by call to kernel)
  current CPU frequency: 546 MHz (asserted by call to kernel)
  current CPU frequency: 549 MHz (asserted by call to kernel)
  current CPU frequency: 548 MHz (asserted by call to kernel)
  current CPU frequency: 547 MHz (asserted by call to kernel)
  current CPU frequency: 548 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 547 MHz (asserted by call to kernel)
  current CPU frequency: 549 MHz (asserted by call to kernel)
  current CPU frequency: 545 MHz (asserted by call to kernel)
  current CPU frequency: 545 MHz (asserted by call to kernel)
  current CPU frequency: 545 MHz (asserted by call to kernel)

choose guided in tlp

sudo cpupower -c 0-15 frequency-info | grep "current CPU" | grep Hz
  current CPU frequency: 1.31 GHz (asserted by call to kernel)
  current CPU frequency: 1.27 GHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 3.20 GHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 1.29 GHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 400 MHz (asserted by call to kernel)
  current CPU frequency: 1.30 GHz (asserted by call to kernel)
  current CPU frequency: 1.54 GHz (asserted by call to kernel)
  current CPU frequency: 1.30 GHz (asserted by call to kernel)
  current CPU frequency: 3.21 GHz (asserted by call to kernel)
  current CPU frequency: 2.80 GHz (asserted by call to kernel)

If set kernel parameter amd_pstate=guided, CPU frequency is fine too.

@linrunner
Copy link
Owner Author

linrunner commented Jul 26, 2023

@tisyang

This maybe not TLP's bug.

Right. It's a driver issue. I would prefer to discuss this over in #630 (comment). I copied your post and have written my reply there.

@siddhpant
Copy link

siddhpant commented Jul 29, 2023

Debian testing finally got 6.4 today, so I am now able to test guided mode, and switching between modes.

Relevant TLP Settings:

CPU_DRIVER_OPMODE_ON_AC=active
CPU_DRIVER_OPMODE_ON_BAT=guided

CPU_SCALING_GOVERNOR_ON_AC=performance
CPU_SCALING_GOVERNOR_ON_BAT=conservative

CPU_SCALING_MIN_FREQ_ON_BAT=400000
CPU_SCALING_MAX_FREQ_ON_BAT=1800000

The mode changes from active to guided and vice-versa successfully upon switching off/on the AC connection.

@lectrode
Copy link

lectrode commented Aug 5, 2023

Regarding #587 , the original issue (USB ports stop working) still affects TLP 1.6 (installed tlp-git from AUR on Manjaro). That issue also effects integrated webcam and bluetooth (on internal USB hub).

The AHCI_RUNTIME_PM_ON_BAT=on workaround still works in 1.6.

Without the workaround, bluetooth stops working after a maximum of ~15 minutes. With the workaround, bluetooth stayed on for the 1.5 hours or so i tested it (on laptop battery the whole time).

tlp stat gist

@linrunner
Copy link
Owner Author

@self Relnotes: #587 only partly solved

@linrunner
Copy link
Owner Author

Thanks to all testers so far.

Is there anything else to report - positive or negative - from ongoing operation?

@linrunner
Copy link
Owner Author

@lectrode #587 has been removed from the changelog and FAQ adapted.

@linrunner
Copy link
Owner Author

Beta test is completed and TLP 1.6.0 is out. Have fun!

Sincere thanks to all testers! 👍

@linrunner
Copy link
Owner Author

Hi @ALL : TLP 1.7 Beta 1 is out. Testers wanted -> #760

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants