fix: build on kernel v6.11 #48

tskaar · 2024-07-30T20:44:27Z

Fixes the build issue:

/var/lib/dkms/xone/0.3.r57.g29ec357/build/bus/bus.c:126:18: error: initialization of ‘int (*)(struct device *, const struct device_driver *)’ from incompatible pointer type ‘int (*)(struct device *, struct device_driver *)’ [-Wincompatible-pointer-types]
  126 |         .match = gip_bus_match,
      |                  ^~~~~~~~~~~~~
/var/lib/dkms/xone/0.3.r57.g29ec357/build/bus/bus.c:126:18: note: (near initialization for ‘gip_bus_type.match’)

Tested and built on torvalds/linux/v6.11-rc1

BrycensRanch · 2024-08-01T01:49:34Z

builds on Fedora Rawhide too. Thanks!

Greenphlem · 2024-08-14T01:00:18Z

Works on KDE Neon, thank you!

mben42 · 2024-08-26T21:54:44Z

Hi, unfortunately it doesn't fix the problem on Manjaro (kernel 6.11.0-rc3-2-MANJARO).

tskaar · 2024-08-26T22:17:57Z

Hi, unfortunately it doesn't fix the problem on Manjaro (kernel 6.11.0-rc3-2-MANJARO).

What exactly is failing, you getting the same issue as in my opening post?

mben42 · 2024-08-26T22:20:39Z

Hi, unfortunately it doesn't fix the problem on Manjaro (kernel 6.11.0-rc3-2-MANJARO).

What exactly is failing, you getting the same issue as in my opening post?

Exact same issue yes, I was able to compile by adding -Wno-error=incompatible-pointer-types as a CFLAFS in Makefile.

tskaar · 2024-08-26T22:51:09Z

Hi, unfortunately it doesn't fix the problem on Manjaro (kernel 6.11.0-rc3-2-MANJARO).

What exactly is failing, you getting the same issue as in my opening post?

Exact same issue yes, I was able to compile by adding -Wno-error=incompatible-pointer-types as a CFLAFS in Makefile.

Compiled fine here using Manjaro's PKGBUILD for their kernel and using xone-dkms-git as a reference PKGBUILD (then changing the source to my fixed branch);

(4/5) Install DKMS modules
==> dkms install --no-depmod xone/0.3.r57.g29ec357 -k 6.11.0-rc5-1-MANJARO
==> dkms install --no-depmod hid-xpadneo/v0.9.6 -k 6.11.0-rc5-1-MANJARO
==> depmod 6.11.0-rc5-1-MANJARO

dlundqvist · 2024-09-06T09:18:42Z

I've added this to my fork.

Grief · 2024-09-15T23:20:17Z

@medusalix do you consider merging it?

BeiNacht · 2024-09-18T11:39:40Z

Tested this PR under NixOS 24.05 with 6.11 kernel. It's building and running

trevorjamesmartin · 2024-09-19T16:58:05Z

✅ nixos-rebuild switch

tested and confirmed PR works

Mavarice · 2024-09-22T22:02:21Z

This PR solved the install issue on Nobara Linux 40 (a fork of Fedora)
No additional steps needed from the main readme except to purge to lingering install files from /usr/src and /var/lib/dkms

chizzyc-rust · 2024-09-24T13:05:09Z

Working on 6.11.0-061100-generic, Mint. Can I ask what exactly your code does? I'm not a dev. I dont understand what your if statement accomplishes. Thanks

tskaar · 2024-09-24T21:59:59Z

Working on 6.11.0-061100-generic, Mint. Can I ask what exactly your code does? I'm not a dev. I dont understand what your if statement accomplishes. Thanks

It's just re-defining the signature of the function declaration based on whether it's newer or older than kernel v6.11.
The reason for the re-definition is that the kernel changed device_driver pointer to be of const struct (v6.11) as opposed to struct, you can find the (start of) upstream commit(s) here: torvalds/linux@c28dd08

Lioli7k · 2024-09-29T12:52:15Z

Works on OpenSUSE Tumbleweed 20240927. Thank you for the fix!

AccaliaDeElementia · 2024-10-01T21:58:45Z

Just took kernel 6.11.1 on Archlinux.

Can confirm this patch allows the build on that kernel version. and functionality works as expected with the patch as well.

Hope it gets merged soon for others to use!

dusanpol · 2024-10-03T04:13:58Z

Builds and works fine on liquorix kernel 6.11.1 on Linux Mint 22 👍

Dratnet · 2024-10-06T21:49:50Z

Confirmed, works well with Linux 6.11.2-arch1-1 (x86-64) and resolves build error.

andrewbaker-uk · 2024-10-15T08:16:49Z

Install fails on Ubuntu 24.10

Installing xone unknown...
Sign command: /usr/bin/kmodsign
Signing key: /var/lib/shim-signed/mok/MOK.priv
Public certificate (MOK): /var/lib/shim-signed/mok/MOK.der
Creating symlink /var/lib/dkms/xone/unknown/source -> /usr/src/xone-unknown

Building module:
Cleaning build area...
make -j16 KERNELRELEASE=6.11.0-8-generic -C /lib/modules/6.11.0-8-generic/build M=/var/lib/dkms/xone/unknown/build...(bad exit status: 2)
ERROR (dkms apport): binary package for xone: unknown not found
Error! Bad return status for module build on kernel: 6.11.0-8-generic (x86_64)
Consult /var/lib/dkms/xone/unknown/build/make.log for more information.
DKMS make.log for xone-unknown for kernel 6.11.0-8-generic (x86_64)
Tue Oct 15 09:15:42 AM BST 2024
make: Entering directory '/usr/src/linux-headers-6.11.0-8-generic'
warning: the compiler differs from the one used to build the kernel
  The kernel was built by: x86_64-linux-gnu-gcc-14 (Ubuntu 14.2.0-4ubuntu2) 14.2.0
  You are using:           gcc-14 (Ubuntu 14.2.0-4ubuntu2) 14.2.0
  CC [M]  /var/lib/dkms/xone/unknown/build/transport/wired.o
  CC [M]  /var/lib/dkms/xone/unknown/build/transport/dongle.o
  CC [M]  /var/lib/dkms/xone/unknown/build/transport/mt76.o
  CC [M]  /var/lib/dkms/xone/unknown/build/bus/bus.o
  CC [M]  /var/lib/dkms/xone/unknown/build/bus/protocol.o
  CC [M]  /var/lib/dkms/xone/unknown/build/auth/auth.o
  CC [M]  /var/lib/dkms/xone/unknown/build/auth/crypto.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/common.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/gamepad.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/headset.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/chatpad.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/madcatz_strat.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/madcatz_glam.o
  CC [M]  /var/lib/dkms/xone/unknown/build/driver/pdp_jaguar.o
  LD [M]  /var/lib/dkms/xone/unknown/build/xone-gip-pdp-jaguar.o
/var/lib/dkms/xone/unknown/build/bus/bus.c:126:18: error: initialization of ‘int (*)(struct device *, const struct device_driver *)’ from incompatible pointer type ‘int (*)(struct device *, struct device_driver *)’ [-Wincompatible-pointer-types]
  126 |         .match = gip_bus_match,
      |                  ^~~~~~~~~~~~~
/var/lib/dkms/xone/unknown/build/bus/bus.c:126:18: note: (near initialization for ‘gip_bus_type.match’)
make[2]: *** [scripts/Makefile.build:244: /var/lib/dkms/xone/unknown/build/bus/bus.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [/usr/src/linux-headers-6.11.0-8-generic/Makefile:1931: /var/lib/dkms/xone/unknown/build] Error 2
make: *** [Makefile:224: __sub-make] Error 2
make: Leaving directory '/usr/src/linux-headers-6.11.0-8-generic'

tskaar · 2024-10-15T10:19:38Z

@andrewbaker-uk

Install fails on Ubuntu 24.10

Can you dump the definition of int gip_bus_match(...) from your /var/lib/dkms/xone/unknown/build/bus/bus.c.
Just include everything between the end of struct device_type{...} until static int gip_bus_probe(...)

GeorgeLukasWagner · 2024-10-15T13:48:02Z

@andrewbaker-uk
i fixed the build issue on my machine by typecasting the function pointer like this,
.match = (int (*)(struct device *, const struct device_driver *))gip_bus_match,
i think this would work too if the other solution failed

andrewbaker-uk · 2024-10-15T14:55:41Z

I'll try and get that info over @tskaar. I went scorched earth and reinstalled 24.04 (luckily was a clean install anyway). Will try and reproduce in a VM.

@GeorgeLukasWagner will give it a go

andrewbaker-uk · 2024-10-15T19:02:27Z

@andrewbaker-uk

Install fails on Ubuntu 24.10

Can you dump the definition of int gip_bus_match(...) from your /var/lib/dkms/xone/unknown/build/bus/bus.c. Just include everything between the end of struct device_type{...} until static int gip_bus_probe(...)

Medusalix

static int gip_bus_match(struct device *dev, struct device_driver *driver)
{
	struct gip_client *client;
	struct gip_driver *drv;
	int i;

	if (dev->type != &gip_client_type)
		return false;

	client = to_gip_client(dev);
	drv = to_gip_driver(driver);

	for (i = 0; i < client->classes->count; i++)
		if (!strcmp(client->classes->strings[i], drv->class))
			return true;

	return false;
}

tskaar
Can't check as it has installed successfully in a VM running 24.10.

@tskaar it may have been an Id10T error, just remembered cloning your repo but not checking out the fix-6.11 branch in which case I apologise!

tskaar · 2024-10-15T20:48:14Z

@andrewbaker-uk;
Considering you are missing:

#if LINUX_VERSION_CODE < KERNEL_VERSION(6, 11, 0)
static int gip_bus_match(struct device *dev, struct device_driver *driver)
#else
static int gip_bus_match(struct device *dev, const struct device_driver *driver)
#endif

I suspect that you just forgot to just apply the patch, luckily a relatively easy fix 😄

Freakygnome · 2024-10-19T12:04:15Z

Builds and works fine on 6.11.3-200.fc40.x86_64

thingsiplay · 2024-10-22T16:18:33Z

Is there a reason why this fix is not upstream?

AccaliaDeElementia · 2024-10-30T17:27:59Z

something seems up with this fix for 6.11.5-arch1-1.

my Elite2 gamepad connects and seems functional, but giving inputs on the device causes the device to disconnect and reconnect within a few seconds of input received.

kernel dmesg output starting at the first message produced when the gamepad disconnects until it's fully reconnected again is here: https://pastebin.com/QjQdbWqf

using the dlundqvist fork from the AUR:

3 aur/xone-dlundqvist-dkms-git 0.3.r84.gc7b24b7-1 (+7 5.50) (Installed: 0.3.r90.gf314bce-1)
    Modern Linux driver for Xbox One and Xbox Series X|S controllers - dlundqvist fork

Happy to provide more debug logs or testing if there's anything else that would be handy.

Instinctlol · 2024-10-30T17:48:15Z

@AccaliaDeElementia that's a problem with the fork. I also ran into multiple issues with it and was curious why people in this PR didn't have any issues.

You need to install xone-dkms-git, but based on this PR. I followed this guide which does exactly that and now it seems to be working fine again.

AccaliaDeElementia · 2024-10-30T19:02:28Z

Alright, i can confirm, the issue is to do with the fork, not the patch.

weird.... but at least I'm up and running again. :)

Backported from medusalix#48 https://github.com/tskaar/xone/tree/fix-6.11

fix: build on kernel v6.11

28df566

wantija mentioned this pull request Aug 18, 2024

DKMS cannot install on 6.11 release candidate kernel dlundqvist/xone#12

Closed

mben42 mentioned this pull request Sep 8, 2024

"Error! Installation aborted" on Manjaro dlundqvist/xone#13

Closed

BeiNacht mentioned this pull request Sep 18, 2024

Build failure: xone kernel driver NixOS/nixpkgs#342775

Closed

tskaar mentioned this pull request Sep 30, 2024

fix: build on kernel v6.12 #53

Open

Mast3rL approved these changes Oct 27, 2024

View reviewed changes

dlundqvist mentioned this pull request Oct 29, 2024

Update bus.c #55

Closed

mwick83 added a commit to mwick83/xone that referenced this pull request Nov 1, 2024

Backport kernel 6.11 build fix

e8c8764

Backported from medusalix#48 https://github.com/tskaar/xone/tree/fix-6.11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: build on kernel v6.11 #48

fix: build on kernel v6.11 #48

tskaar commented Jul 30, 2024

BrycensRanch commented Aug 1, 2024

Greenphlem commented Aug 14, 2024

mben42 commented Aug 26, 2024

tskaar commented Aug 26, 2024

mben42 commented Aug 26, 2024

tskaar commented Aug 26, 2024 •

edited

Loading

dlundqvist commented Sep 6, 2024

Grief commented Sep 15, 2024

BeiNacht commented Sep 18, 2024

trevorjamesmartin commented Sep 19, 2024

Mavarice commented Sep 22, 2024

chizzyc-rust commented Sep 24, 2024

tskaar commented Sep 24, 2024

Lioli7k commented Sep 29, 2024

AccaliaDeElementia commented Oct 1, 2024

dusanpol commented Oct 3, 2024

Dratnet commented Oct 6, 2024

andrewbaker-uk commented Oct 15, 2024

tskaar commented Oct 15, 2024

GeorgeLukasWagner commented Oct 15, 2024

andrewbaker-uk commented Oct 15, 2024

andrewbaker-uk commented Oct 15, 2024

tskaar commented Oct 15, 2024

Freakygnome commented Oct 19, 2024

thingsiplay commented Oct 22, 2024

AccaliaDeElementia commented Oct 30, 2024

Instinctlol commented Oct 30, 2024

AccaliaDeElementia commented Oct 30, 2024

fix: build on kernel v6.11 #48

Are you sure you want to change the base?

fix: build on kernel v6.11 #48

Conversation

tskaar commented Jul 30, 2024

BrycensRanch commented Aug 1, 2024

Greenphlem commented Aug 14, 2024

mben42 commented Aug 26, 2024

tskaar commented Aug 26, 2024

mben42 commented Aug 26, 2024

tskaar commented Aug 26, 2024 • edited Loading

dlundqvist commented Sep 6, 2024

Grief commented Sep 15, 2024

BeiNacht commented Sep 18, 2024

trevorjamesmartin commented Sep 19, 2024

Mavarice commented Sep 22, 2024

chizzyc-rust commented Sep 24, 2024

tskaar commented Sep 24, 2024

Lioli7k commented Sep 29, 2024

AccaliaDeElementia commented Oct 1, 2024

dusanpol commented Oct 3, 2024

Dratnet commented Oct 6, 2024

andrewbaker-uk commented Oct 15, 2024

tskaar commented Oct 15, 2024

GeorgeLukasWagner commented Oct 15, 2024

andrewbaker-uk commented Oct 15, 2024

andrewbaker-uk commented Oct 15, 2024

tskaar commented Oct 15, 2024

Freakygnome commented Oct 19, 2024

thingsiplay commented Oct 22, 2024

AccaliaDeElementia commented Oct 30, 2024

Instinctlol commented Oct 30, 2024

AccaliaDeElementia commented Oct 30, 2024

tskaar commented Aug 26, 2024 •

edited

Loading