** Description changed: - It looks like bbswitch is using acpi_bus_get_device() on any - architectures, but this function is not available on armhf or ppc64el: + [Impact] + + bbswitch is failing to build with the latest jammy kernel 5.15 on armhf + and ppc64el. + + The build error that we get looks like the following: CC [M] /var/lib/dkms/bbswitch/0.8/build/bbswitch.o /var/lib/dkms/bbswitch/0.8/build/bbswitch.c: In function ‘bbswitch_off’: /var/lib/dkms/bbswitch/0.8/build/bbswitch.c:271:13: error: implicit declaration of function ‘acpi_bus_get_device’; did you mean ‘acpi_get_gpe_device’? [-Werror=implicit-function-declaration] 271 | r = acpi_bus_get_device(dis_handle, &ad); | ^~~~~~~~~~~~~~~~~~~ | acpi_get_gpe_device /var/lib/dkms/bbswitch/0.8/build/bbswitch.c:276:15: error: invalid use of undefined type ‘struct acpi_device’ 276 | if (ad->power.state == ACPI_STATE_UNKNOWN) { | ^~ /var/lib/dkms/bbswitch/0.8/build/bbswitch.c:278:15: error: invalid use of undefined type ‘struct acpi_device’ 278 | ad->power.state = ACPI_STATE_D0; | ^~ + + [Test case] + + Install bbswitch-dkms. + + [Fix] + + It looks like bbswitch is using acpi_bus_get_device() on any + architectures to check for suspend events, but this function is not + available on armhf or ppc64el. The fix would be to avoid doing this + check (and using acpi_bus_get_device()) on architectures that don't have + this capability. + + [Regression potential] + + We should see potential bbswitch regressions on armhf or ppc64el if they + have suspend events (similar problem that is addressed on amd64 / + arm64).
-- You received this bug notification because you are a member of Desktop Packages, which is subscribed to bbswitch in Ubuntu. https://bugs.launchpad.net/bugs/1951583 Title: bbswitch dkms build failure with 5.15 on armhf and ppc64el Status in bbswitch package in Ubuntu: New Status in bbswitch source package in Jammy: New Bug description: [Impact] bbswitch is failing to build with the latest jammy kernel 5.15 on armhf and ppc64el. The build error that we get looks like the following: CC [M] /var/lib/dkms/bbswitch/0.8/build/bbswitch.o /var/lib/dkms/bbswitch/0.8/build/bbswitch.c: In function ‘bbswitch_off’: /var/lib/dkms/bbswitch/0.8/build/bbswitch.c:271:13: error: implicit declaration of function ‘acpi_bus_get_device’; did you mean ‘acpi_get_gpe_device’? [-Werror=implicit-function-declaration] 271 | r = acpi_bus_get_device(dis_handle, &ad); | ^~~~~~~~~~~~~~~~~~~ | acpi_get_gpe_device /var/lib/dkms/bbswitch/0.8/build/bbswitch.c:276:15: error: invalid use of undefined type ‘struct acpi_device’ 276 | if (ad->power.state == ACPI_STATE_UNKNOWN) { | ^~ /var/lib/dkms/bbswitch/0.8/build/bbswitch.c:278:15: error: invalid use of undefined type ‘struct acpi_device’ 278 | ad->power.state = ACPI_STATE_D0; | ^~ [Test case] Install bbswitch-dkms. [Fix] It looks like bbswitch is using acpi_bus_get_device() on any architectures to check for suspend events, but this function is not available on armhf or ppc64el. The fix would be to avoid doing this check (and using acpi_bus_get_device()) on architectures that don't have this capability. [Regression potential] We should see potential bbswitch regressions on armhf or ppc64el if they have suspend events (similar problem that is addressed on amd64 / arm64). To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/bbswitch/+bug/1951583/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp