[Feature] Xbox controller's battery makes it switch to battery profile

[timrosu]

Fill out information requested in this template, without doing so issue will be ignored & closed!

Have you tried?

• Searching through existing/closed issues to see if your bug has already been already submitted? ✅
• If installation failed with auto-cpufreq-installer,have you tried installing auto-cpufreq using Snap package? ❌
• Have you tried configuring auto-cpufreq? ✅
• Have you tried suggestions in troubleshooting section? ✅

Error output:

Add/paste error output in case of failed installation or other failing component

---

System information:

Add/paste output of:

auto-cpufreq --debug

Using settings defined in /etc/auto-cpufreq.conf file

-------------------------------------------------------------------------------

Linux distro: Garuda Linux Soaring BirdOfPrey
Linux kernel: 6.10.0-zen1-2-zen
Processor: Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz
Cores: 12
Architecture: x86_64
Driver: intel_pstate

------------------------------ Current CPU stats ------------------------------

CPU max frequency: 3700 MHz
CPU min frequency: 800 MHz

Core	Usage	Temperature	Frequency
CPU0      8.2%        37 °C       800 MHz
CPU1     14.9%        37 °C       800 MHz
CPU2      3.0%        37 °C       800 MHz
CPU3     14.0%        37 °C       800 MHz
CPU4      6.0%        38 °C       800 MHz
CPU5      9.3%        36 °C       800 MHz
CPU6     10.0%        37 °C       800 MHz
CPU7      6.0%        37 °C       800 MHz
CPU8      4.0%        37 °C       800 MHz
CPU9      8.9%        37 °C       800 MHz
CPU10      5.1%        38 °C       801 MHz
CPU11      7.9%        36 °C       800 MHz

CPU fan speed: 91 RPM

CPU fan speed: 0 RPM

auto-cpufreq version: Version         : 2.3.0-1

Python: 3.12.4
psutil package: 6.0.0
platform package: 1.0.8
click package: 8.1.7
distro package: 1.9.0

Computer type: Desktop
Battery is: discharging

auto-cpufreq system resource consumption:
cpu usage: 0.0 %
memory use: 0.12 %

Total CPU usage: 2.0 %
Total system load: 1.55
Average temp. of all cores: 44.17 °C

Currently using: powersave governor
Currently turbo boost is: off

-------------------------------------------------------------------------------

Also please be descriptive about the issue you're reporting, i.e: what you tried & what's the expected behaviour.

---

Few months ago I have copied default config from readme to my conf file on my desktop pc and only changed turboboost to always. I was playing games in Windows vm with xbox wireless adapter passed to it back then.

Last week I tried gaming natively on Linux and installed xone driver (xone-dkms in aur) which reports battery level of controller. I then tried playing FS22 (pretty CPU intensive) and it worked great (90-130 fps)... until I connected controller and fps dropped to 30. Then I tried connecting it with wire and it worked normally (because controller doesn't report battery usage when plugged). I noticed that cpu frequency was a bit low (800 MHz). Then I went to look at output of auto-cpufreq status and saw that it is using powersave governor and that (controller's) battery was discharging.

In debug output I noticed that program detects computer type. Why doesn't it then just ignore the battery entry in config if desktop type is detected?

I tried connecting my motorcycle helmet intercom to pc and its battery wasn't detected in auto-cpufreq.

Then I looked into /sys/class/power_supply and found that xone has made an entry named gip0.0
Contents of uevent to help you filter it out:

DEVTYPE=power_supply
POWER_SUPPLY_NAME=gip0.0
POWER_SUPPLY_TYPE=Battery
POWER_SUPPLY_STATUS=Discharging
POWER_SUPPLY_CAPACITY_LEVEL=High
POWER_SUPPLY_SCOPE=Device
POWER_SUPPLY_MODEL_NAME=Microsoft Xbox Controller

[PurpleWazard]

huh thats weird but makes sense. for the time being try overriding the Governor in the auto-cpufreq gui. this will keep the cpu at full performance even when on battery.

[AdnanHodzic]

In addition to what @PurpleWazard said, alternatively you can also use --force flag, or config file.

[shadeyg56]

Seems we need more intelligent battery checking here.
We could add gip0.0 to the power supply ignore list that is defined at the top of the file, but it would be nice to have a solution that keeps this from happening again

[timrosu]

@shadeyg56 That would be the best solution. And maybe you could also ignore all batteries if desktop is detected + possible override in conf file (in case of ups).

[timrosu]

@AdnanHodzic I made battery portion of config the same as charger as soon as I found out that auto-cpufreq is impacting performance. But it would be nice to have a way to filter out batteries. I haven't yet tested it with xone driver on laptop, but I suspect it would go into battery mode if I connect a controller to a charging laptop.

[AdnanHodzic]

@timrosu appended "feature", until one of the contributors or I get a chance to take a look at this. If you or anyone else wants to give it a try in implementing this functionality, please give it a try and contribute to the project and you will be credited for your work as part of future release.

[PurpleWazard]

so my idea to fix this is to add an section the the config file where you can ignore specified batteries however then all will be included by default. is this the way this should be inplmented or wanted? or should we just hard code to ingore /sys/class/power_supply/gip* ?

[timrosu]

@PurpleWazard I think it would be best to add option to add batteries to ignore in config file and then show this option in readme. Names of problematic batteries could be included by default in conf file and ignoring enabled.

[PurpleWazard]

@timrosu I submitted a PR wanna try it out

[AdnanHodzic]

@timrosu I submitted a PR wanna try it out

Please check if #760 fixes your problem, or if any additional changes need to be made before merging it.

[timrosu]

@PurpleWazard Great. It works.

[timrosu]

@AdnanHodzic I think it's good as it is (apart from spelling mistake in config comment).