Kernel Module bauen nicht mit aktuellem LTS-Kernel 6.18

[df8oe]

Es haben sich ein paar Funktionsaufrufe geändert, offensichtlich. Mit dem aktuellen LTS-Kernel bauen die Module nicht. Ist dort eine Anpassung zeitnah geplant? Debian will den Dino 6.12 wohl noch ein paar Jahre reiten, ich möchte aber aktuell bleiben ;) Ich würde auch bei der Anpassung mitwirken, jederzeit mit Logs, ggf. mit Pull-Requests. Die Thematik ist zwar nicht mein Hauptgebiet, aber ich würde ungern noch ein paar Jahre auf dem 6.12er verharren... Würden Logs weiterhelfen?

[df8oe]

Die Probleme treten auf in generic_raw_uart.c in Zeile 946.
Vermutete Lösung: void *addrp in const void *addrp ändern

Weiter tauchen Probleme in hb_rf_usb.c, hb_rf_usb_2.c und hb_rf_eth.c auf bei den Deklarationen der Funktionen
hb_xyz_gpio_set und hb_xyz_gpio_set_multiple auf. Diese sind als void deklariert, was zu Fehlern führt. Ändere ich diese zu int (und gebe ihnen ein return 0;), dann läuft der Bau durch.

Ich mache die Testbuilds aber nicht auf dem Raspi, wo auch tatsächlich eine OpenCCU läuft (der ist auf den Kernel 6.12 festgepinnt), sondern auf einem zweiten Raspi, an dem KEINE OpenCCU läuft.
Preisfrage:
Können die Ansätze richtig sein? Bei der für generic_raw_uart denke ich schon, beim Rest bin ich mir unsicher...

[jens-maus]

Ich würde sagen: Zeig bzw. droppe hier einfach mal in einem code-block die compiler-fehler die mit kernel 6.18+ auftreten und dann kann man das ruckzuck beheben. Da braucht es keine umfangreiche beschreibung/diskussion der Probleme ;-)

[jens-maus]

Weiterer hinweis: I habe noch ein kernel 6.19.x compatibility change zu #575 hinzugefügt und konnte damit auch das korrekte kompilieren der kernel module mit kernel 6.18 und 6.19 kontrollieren. Bitte also (falls es definitiv noch probleme für das kompilieren gegen kernel 6.18.x gibt) die konkreten compilerausgaben hier zeigen.

[df8oe]

Danke für die schnelle Reaktion. Sowie der PR gemerged ist werde ich testen & berichten! Bis auf eine zusätzliche Änderung in generic_raw_uart.c, die ich gemacht habe, entspricht das auch meinen Änderungen. We will see.

[df8oe]

Ich habe es mal von deinem GitHub gezogen und getestet. Es baut bei mir nicht mit Kernel 6.18.1. Hier das Log:

generic_raw_uart.c:936:5: Warnung: kein vorheriger Prototyp für »generic_raw_uart_get_gpio_index« [-Wmissing-prototypes]
  936 | int generic_raw_uart_get_gpio_index(struct device *dev, char *con_id, unsigned int idx)
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
generic_raw_uart.c:951:5: Warnung: kein vorheriger Prototyp für »generic_raw_uart_get_led_gpio_index« [-Wmissing-prototypes]
  951 | int generic_raw_uart_get_led_gpio_index(struct generic_raw_uart_instance *instance, struct device *dev, enum generic_raw_uart_led led)
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
generic_raw_uart.c: In Funktion »i2c_find_client«:
generic_raw_uart.c:1064:51: Fehler: Übergabe des Arguments 3 von »device_find_child« von inkompatiblem Zeigertyp [-Wincompatible-pointer-types]
 1064 |   child = device_find_child(&adapter->dev, &addr, __match_i2c_client_by_address);
      |                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                                   |
      |                                                   int (*)(struct device *, void *)
In Datei, eingebunden von generic_raw_uart.c:24:
/usr/lib/modules/6.18.0-1-rpi/build/include/linux/device.h:997:49: Anmerkung: »device_match_t« {alias »int (*)(struct device *, const void *)«} erwartet, aber Argument hat Typ »int (*)(struct device *, void *)«
  997 |                                  device_match_t match);
      |                                  ~~~~~~~~~~~~~~~^~~~~
generic_raw_uart.c:1054:12: Anmerkung: »__match_i2c_client_by_address« wird hier deklariert
 1054 | static int __match_i2c_client_by_address(struct device *dev, void *addrp)
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In Datei, eingebunden von /usr/lib/modules/6.18.0-1-rpi/build/include/linux/device.h:29:
/usr/lib/modules/6.18.0-1-rpi/build/include/linux/device/bus.h:133:15: Anmerkung: »device_match_t« wird hier deklariert
  133 | typedef int (*device_match_t)(struct device *dev, const void *data);
      |               ^~~~~~~~~~~~~~
generic_raw_uart.c: Auf höchster Ebene:
generic_raw_uart.c:1083:5: Warnung: kein vorheriger Prototyp für »generic_raw_uart_probe_rtc_device« [-Wmissing-prototypes]
 1083 | int generic_raw_uart_probe_rtc_device(struct device *dev, bool *rtc_detected)
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  CC [M]  rpi_rf_mod_led.o
make[3]: *** [/usr/lib/modules/6.18.0-1-rpi/build/scripts/Makefile.build:287: generic_raw_uart.o] Fehler 1
make[3]: *** Es wird auf noch nicht beendete Prozesse gewartet …
make[2]: *** [/usr/lib/modules/6.18.0-1-rpi/build/Makefile:2010: .] Fehler 2
make[1]: *** [/usr/lib/modules/6.18.0-1-rpi/build/Makefile:248: __sub-make] Fehler 2
make[1]: Verzeichnis „/var/lib/dkms/pivccu-modules/1.0.85/build“ wird verlassen
make: *** [Makefile:248: __sub-make] Fehler 2
make: Verzeichnis „/usr/lib/modules/6.18.0-1-rpi/build“ wird verlassen

Ich habe dann mal in Zeile 1054 von generic_raw_uart.c void *addrp in const void *addrp geändert, dann baut es.

Die Funktion habe ich noch nicht getestet, da erstaunlicherweise der Docker-Daemon mit Kernel 6.18 nicht mehr startet und auf den ersten Blick die Meldungen dazu nicht weiterführend sind. Ich teste weiter - warum baut es bei Dir ohne die Änderung in Zeile 1054?

[df8oe]

Das Problem lag bei mir (Kernel 6.18.1 war offenbar buggy - nehme jetzt den aktuellen 6.18.7). Docker startet jetzt, aber OpenCCU läuft fehlerhaft. Es funktioniert kein Gerät mehr, das ich über Funk (hb-rf-eth) anspreche. Im dmesg steht aber - wie beim 6.12er-Kernel, dass die Hardware gefunden wurde:

[    6.911859] hb-rf-eth hb-rf-eth: failed creating connect sysfs file: 0
[  112.308903] hb-rf-eth hb-rf-eth: Trying to connect to 192.168.0.250
[  112.411851] hb-rf-eth hb-rf-eth: Successfully connected to 192.168.0.250

Im OpenCCU-log finde ich das hier:

Starting multimacd: modprobe: FATAL: Module eq3_char_loop not found in directory /lib/modules/6.18.7-2-rpi
..........ERROR: couldn't setup /dev/eq3loop
Starting rfd: ERROR: /dev/mmd_bidcos missing, no BidCos-RF hardware found
Starting HMIPServer: ERROR: /dev/mmd_hmip missing

[jens-maus]

Starting multimacd: modprobe: FATAL: Module eq3_char_loop not found in directory /lib/modules/6.18.7-2-rpi
..........ERROR: couldn't setup /dev/eq3loop
Starting rfd: ERROR: /dev/mmd_bidcos missing, no BidCos-RF hardware found
Starting HMIPServer: ERROR: /dev/mmd_hmip missing

Na dann fehlt dir noch das eq3_char_loop kernel modul in deinem /lib/modules verzeichnisbaum. Das müsstest du natürlich auch kompilieren und dort installieren.

[df8oe]

Das kommt davon wenn man sich auf Dinge verlässt... In der Tat fehlt das Modul. Quellcode ist da, die dkms.conf des Arch-Paketes ist fehlerhaft! Bislang lief alles auch mit dem fehlenden eq3_char_loop, jetzt nicht mehr. Modul inkludiert und alles läuft wieder auch unter dem 6.18.7-Kernel. Einzig die besagte Änderung in Zeile 1054 von generic_raw_uart.c ist bei mir noch notwendig damit dieses Modul baut. Ich habe den Maintainer des Arch-Paketes schon informiert... Asche auf mein Haupt - sorry für das Gerausche.

[dallmair]

Fixed today with the merge of #575.