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

kernel 6.12: error install #498

Closed
RaulKong898 opened this issue Oct 5, 2024 · 10 comments
Closed

kernel 6.12: error install #498

RaulKong898 opened this issue Oct 5, 2024 · 10 comments
Labels
0 | type: bug Something isn't working
Milestone

Comments

@RaulKong898
Copy link

raulkong898@fedora:~/xpadneo$ sudo ./install.sh

  • creating dkms.conf
  • adding hid-xpadneo-v0.9-175-g70ef8ee folder to /usr/src
  • installing module (using DKMS)

Sign command: /lib/modules/6.12.0-0.rc1.17.fc42.x86_64/build/scripts/sign-file
Signing key: /var/lib/dkms/mok.key
Public certificate (MOK): /var/lib/dkms/mok.pub
Creating symlink /var/lib/dkms/hid-xpadneo/v0.9-175-g70ef8ee/source -> /usr/src/hid-xpadneo-v0.9-175-g70ef8ee

Cleaning build area... done.
Building module(s)...(bad exit status: 2)
Failed command:
make -j12 KERNELRELEASE=6.12.0-0.rc1.17.fc42.x86_64 -C /lib/modules/6.12.0-0.rc1.17.fc42.x86_64/build M=/var/lib/dkms/hid-xpadneo/v0.9-175-g70ef8ee/build/src VERSION=v0.9-175-g70ef8ee modules
Error! Bad return status for module build on kernel: 6.12.0-0.rc1.17.fc42.x86_64 (x86_64)
Consult /var/lib/dkms/hid-xpadneo/v0.9-175-g70ef8ee/build/make.log for more information.

@kakra
Copy link
Collaborator

kakra commented Oct 5, 2024

Please attach this file: /var/lib/dkms/hid-xpadneo/v0.9-175-g70ef8ee/build/make.log.

@RaulKong898
Copy link
Author

Please attach this file: /var/lib/dkms/hid-xpadneo/v0.9-175-g70ef8ee/build/make.log.

make.log

@kode54
Copy link

kode54 commented Oct 8, 2024

I can attach a similar make.log. Also from a 6.12-rc kernel, rc2 to be precise.

make.log

@kakra
Copy link
Collaborator

kakra commented Oct 8, 2024

Okay, this looks like an incompatibility with kernel 6.12. I'll fix this. Thanks for reporting.

@kakra kakra added the 0 | type: bug Something isn't working label Oct 8, 2024
@kakra kakra changed the title error install kernel 6.12: error install Oct 8, 2024
@kakra kakra added this to the v0.10 milestone Oct 8, 2024
@TarsiSurdi
Copy link

Can confirm that there is an issue with the kernel module with this version of the kernel (6.12). Just tried it out on Fedora 41 Beta and it fails to build.

@hithere-at
Copy link

hithere-at commented Oct 27, 2024

I think i have found a workaround. You can change the code from

static u8 *xpadneo_report_fixup(struct hid_device *hdev, u8 *rdesc, unsigned int *rsize)

to

static const u8 *xpadneo_report_fixup(struct hid_device *hdev, u8 *rdesc, unsigned int *rsize)

Its on line 664 (on nano), the file is at xpadneo/hid-xpadneo/src/hid-xpadneo.c

@kakra kakra closed this as completed in 4bfe0a1 Oct 31, 2024
kakra added a commit to kakra/xpadneo that referenced this issue Oct 31, 2024
Kernel 6.12 changed the signature of a HID function. We can simply
conditionally use the one or the other based on the kernel version.

Fixes: atar-axis#498
Signed-off-by: Kai Krakow <[email protected]>
@ionenwks
Copy link

ionenwks commented Nov 18, 2024

It'd be nice if got a 0.9.x release with that fix now that kernel 6.12 is released (Edit: or a 0.10.x release would be fine too, but imagine do not want to rush that).

(I received https://bugs.gentoo.org/943776 not long ago, backported the fix so a release is not urgent for us, but it'll be affecting others too)

@kakra
Copy link
Collaborator

kakra commented Nov 18, 2024

I've queued some patches here: https://github.com/kakra/xpadneo/commits/queue/for-0.9/

Feel free to test those. I yet want to finalize the remaining hardware support commits before releasing a final v0.9. From there on, v0.10 will restructure a lot of code so it will become impossible to backport patches from v0.10 to v0.9. So I want to let that settle first.

@ionenwks
Copy link

ionenwks commented Nov 18, 2024

I've queued some patches here: https://github.com/kakra/xpadneo/commits/queue/for-0.9/

Seems to be missing commit 242e9b4's addition of #include <linux/version.h> to xpadneo.h which is needed for the 6.12 commit (I've added that to my own backport in Gentoo).

Beside that seems fine at a glance, albeit I don't really use/test 0.9.x much myself (I've been using the master branch without issues for a while for my own gamepad -- which is updated to latest every kernel bumps I do -- but most normal Gentoo users use the releases).

@kakra
Copy link
Collaborator

kakra commented Nov 18, 2024

Seems to be missing commit 242e9b4's addition of #include <linux/version.h> to xpadneo.h which is needed for the 6.12 commit (I've added that to my own backport in Gentoo).

Good find. For some reason, the same patch builds just fine in master. I will fix that.

@kakra kakra reopened this Nov 18, 2024
@github-project-automation github-project-automation bot moved this from Done to In progress in Compatibility and Compliance Nov 18, 2024
kakra added a commit to kakra/xpadneo that referenced this issue Nov 18, 2024
Includes the missing header file.

Fixes: d611d70 ("hid-xpadneo: Allow building with kernel 6.12")
Closes: atar-axis#498
Signed-off-by: Kai Krakow <[email protected]>
@kakra kakra closed this as completed Nov 18, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Compatibility and Compliance Nov 18, 2024
@kakra kakra pinned this issue Dec 19, 2024
kakra added a commit that referenced this issue Dec 23, 2024
Kernel 6.12 changed the signature of a HID function. We can simply
conditionally use the one or the other based on the kernel version.

Fixes: #498
Signed-off-by: Kai Krakow <[email protected]>
kakra added a commit that referenced this issue Dec 23, 2024
Includes the missing header file.

Fixes: d611d70 ("hid-xpadneo: Allow building with kernel 6.12")
Closes: #498
Signed-off-by: Kai Krakow <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 | type: bug Something isn't working
Projects
Development

No branches or pull requests

6 participants