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

The module ec_sys is not present in kernels higher than 5.11.2 #3

Closed
lestcape opened this issue Jul 11, 2023 · 4 comments
Closed

The module ec_sys is not present in kernels higher than 5.11.2 #3

lestcape opened this issue Jul 11, 2023 · 4 comments

Comments

@lestcape
Copy link

Please see:
https://bugzilla.redhat.com/show_bug.cgi?id=1943318

Or this proposed workaround, that use a kernel module to recover the functionality:
YoyPa/isw#265 (comment)

@lestcape
Copy link
Author

lestcape commented Jul 11, 2023

@lestcape I have already read through those resources as I ran into that error when i tried to update Ubuntu to 22.04. Using the above commands worked to restore access to the EC. As for the other distros i cannot comment as i have not tested them.

If Ubuntu re-added the module somehow (probably just they compiled the kernel with an option to enable the module), i don' t know. In Debian 12 with the Linux kernel 6.1:

Captura desde 2023-07-11 10-56-28

@snowyoneill
Copy link
Owner

snowyoneill commented Jul 12, 2023

@lestcape yeah this code was only tested against Ubuntu 20.04 with an earlier kernel before they removed the access to the EC. If you follow the steps you posted and install acpi_ec then you can update the EC_IO_FILE variable in ecwrite.py to point to the new path and then the program should work.

And im 100% behind on the idea of releasing the program as a deb or package of some sort. I have not had a chance to look into that but willing to work on it with you.

@lestcape
Copy link
Author

Yes, this should be the idea, but it need to be tested and if is working then modify the app to allow both uri. If one fail then we should use the another and if both fails, the app should show a message and do not crash, but currently the app crash.

By the way, seem to be that acpi_ec is compatible with secure boot, but msr-tools is not compatible. Should be good to also have compatibility with secure boot from both and apparently this is probably possible if we fork the msr-tools and modify it. See: https://github.com/intel/intel-cmt-cat/wiki/UEFI-Secure-Boot-Compatibility. Also should be good to have support for the turbo button. My turbo button works with the Jafar kernel module: https://github.com/JafarAkhondali/acer-predator-turbo-and-rgb-keyboard-linux-module. Is the only thing that aparently really work with this module and my laptop.

Unfortunately i have not time for now to do all that things. But i promised that was going to test your fork and well, I'm just reporting what I could see. There are more problems really, but they are related to Qt and I really don't know much about pyQt.

@snowyoneill
Copy link
Owner

@lestcape closing this issue. I have updated the code to point to the acpi_ec path if it exists. I've also updated the README to reflect that msr-tools and undervolt are now optional. Without installing these packages the app should still work, you will simply no longer see the voltage read outs, historical chart or be able to make changes to the CPU offsets.

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

No branches or pull requests

2 participants