tuxedocomputers / tuxedo-keyboard

This repository will no longer get any updates as the code here is now part of tuxedo-drivers https://gitlab.com/tuxedocomputers/development/packages/tuxedo-drivers.
https://gitlab.com/tuxedocomputers/development/packages/tuxedo-drivers
GNU General Public License v3.0
299 stars 93 forks source link

Keyboard backlight is not working after update, returning module verification failed: signature and/or required key missing - tainting kernel #181

Closed yunusarkan closed 11 months ago

yunusarkan commented 1 year ago

My keyboard backlight is not working after updating to the new version, here is the output of sudo dmesg | grep -i keyboard;

[    3.535023] input: AT Translated Set 2 keyboard as /devices/platform/i8042/serio0/input/input0
[    5.092449] hid-generic 0003:1EA7:0066.0002: input,hidraw1: USB HID v1.10 Keyboard [2.4G Mouse] on usb-0000:00:14.0-6/input0
[    7.917997] systemd[1]: Starting Set the console keyboard layout...
[    7.937076] systemd[1]: Finished Set the console keyboard layout.
[    7.948700] tuxedo_keyboard: loading out-of-tree module taints kernel.
[    7.948739] tuxedo_keyboard: module verification failed: signature and/or required key missing - tainting kernel
[    7.949405] tuxedo_keyboard: module init
[    8.082042] tuxedo_keyboard: Registering single zone rgb leds interface failed
[    8.082046] tuxedo_keyboard: Set keyboard backlight mode on CUSTOM
[    8.089586] tuxedo_keyboard: Set keyboard enabled to: 1
[    8.094112] input: TUXEDO Keyboard as /devices/platform/tuxedo_keyboard/input/input25
Matombo commented 1 year ago

Please post Distribution and desktop environment with version numbers Also the version of tuxedo-control-center and tuxedo-keyboard you have currently installed

onurbbr commented 1 year ago

I have this problem too. Keyboard lights work sometimes. Sometimes it doesn't work. I'm using Tuxedo Polaris 17 Gen1. This is dmesg output:

onur@ubuntu:~$ sudo dmesg | grep tuxedo
[sudo] password for onur: 
[    2.306449] tuxedo_keyboard: loading out-of-tree module taints kernel.
[    2.306504] tuxedo_keyboard: module verification failed: signature and/or required key missing - tainting kernel
[    2.314468] tuxedo_keyboard: module init
[    2.945127] input: TUXEDO Keyboard as /devices/platform/tuxedo_keyboard/input/input24

Kernel Version:

onur@ubuntu:~$ uname -a
Linux ubuntu 6.3.3-x64v3-xanmod1 #0~20230517.05b3a98 SMP PREEMPT_DYNAMIC Thu May 18 05:17:33 UTC  x86_64 x86_64 x86_64 GNU/Linux

I used ubuntu 5.19.x kernel. Unfortunately the result is the same.

OS info:

onur@ubuntu:~$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.2 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.2 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

App and Module versions:

TUXEDO Control Center: 2.0.5
TUXEDO Keyboard: 3.2.3
onurbbr commented 1 year ago

When I revert this commit, I can control the keyboard lights. @Matombo

Matombo commented 1 year ago

Ok this is very strange. When you can't control the backlight, what entries do you have in /sys/class/leds?

Matombo commented 1 year ago

Can you please double check that you are not running in one of these bugs?

https://gitlab.gnome.org/GNOME/gnome-settings-daemon/-/issues/743

https://gitlab.gnome.org/GNOME/gnome-settings-daemon/-/issues/744

https://gitlab.gnome.org/GNOME/gnome-settings-daemon/-/issues/745

onurbbr commented 1 year ago

I'm sorry for the late reply. The output is as follows:

onur@ubuntu:~$ ls /sys/class/leds
input4::capslock  input4::numlock  input4::scrolllock  mmc0::  phy0-led  rgb:kbd_backlight

Btw, I tried Linux Mint and Kubuntu. Keyboard works fine. The problem is from GNOME.

Edit: It's stuttering on 5.x kernels. There seems to be no problem with 6.x. I'm using XanMod 6.x LTS kernel and it seems to be ok with Kubuntu (Mint is fine too.)

Matombo commented 1 year ago

A rework fixing https://gitlab.gnome.org/GNOME/gnome-settings-daemon/-/issues/744 is coming, no ETA though.

For the other 2 bugs i'm still waiting for the gnome maintainers reply.

onurbbr commented 1 year ago

The problem seems to be fixed on Ubuntu 22.04.3 and Fedora 38.

Tested operating systems (GNOME Desktop environment): Ubuntu 22.04.3, 6.2 kernel, GNOME 42 Fedora 38, 6.4.9 kernel, GNOME 44

Thanks for your hard work.

Screenshot from 2023-08-13 12-47-19

IzzySoft commented 1 year ago

I've just ran into this issue, upgrading an Aura 15 from Linux Mint 20.3 (Cinnamon) to Mint 21. It didn't start right away, but only after we added back the Tuxedo APT-repo and applied all updates. Using Timeshift to revert (we've luckily created a snapshot right before this step) and applying the changes one-by-one, we've tracked down the culprit being tuxedo-keyboard: as soon as that's updated, on the next reboot, keyboard backlight goes off.

For those still affected, we helped ourselves by apt-pinning tuxedo-keyboard to version 3.1.4 (the version that was in use with Mint 20), to prevent accidental updates in the future. For this we've placed the following content into a file named tuxedo-keyboard.pref located in /etc/apt/preferences.d/:

Package: tuxedo-keyboard
Pin: version 3.1.4*
Pin-Priority: 1000

So until there's an update available solving this regression, the above might help you as a temporary work-around. The "upstream fix" in gnome-settings-daemon mentioned back in June seems not to be coming (the linked issue was closed without taking action, and the resulting issue never responded to for 5 months now), so it might take a while. tuxedo-keyboard version 3.1.4 still seems to do its job.

tuxedo-bot commented 11 months ago

This repository will no longer get any updates as the code here is now part of tuxedo-drivers https://gitlab.com/tuxedocomputers/development/packages/tuxedo-drivers.

Please open a merge request or issue in the new project on GitLab if you contribution or bug report still applies.