oss4/4.2-build2010-5ubuntu6~20.04.2 ADT test failure with linux-hwe-5.11/5.11.0-20.21~20.04.1
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux-hwe-5.11 (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Focal |
Invalid
|
Undecided
|
Unassigned | ||
oss4 (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Focal |
Fix Released
|
Medium
|
Thadeu Lima de Souza Cascardo |
Bug Description
[Impact]
oss4 dkms will fail to build on 5.11 kernels on focal, preventing users from using those modules on more recent kernels.
[Test case]
Install the package and make sure it builds.
[Fix]
Conditionally define a function that is not used on the Linux port when LICENSED_VERSION is undefined.
[Fix risk]
Licensed versions will fail to build, but we don't do those. Other ports should not fail as we only patch the Linux port.
[Potential regression]
The dkms package may still fail to build, install or load. Or sound might not work on systems that depend on oss4.
-------
This is a scripted bug report about ADT failures while running oss4 tests for linux-hwe-
Testing failed on:
amd64: https:/
[Other]
NB! dkms ftbfs fixes must be built in security, such that after SRU process in -proposed & -updates it can be copied into -security pocket too.
Changed in oss4 (Ubuntu Focal): | |
assignee: | nobody → Thadeu Lima de Souza Cascardo (cascardo) |
importance: | Undecided → Medium |
status: | Confirmed → In Progress |
tags: | added: patch |
description: | updated |
Changed in oss4 (Ubuntu): | |
status: | New → Invalid |
Changed in linux-hwe-5.11 (Ubuntu): | |
status: | New → Invalid |
Changed in linux-hwe-5.11 (Ubuntu Focal): | |
status: | New → Invalid |
get_seconds() is gone, as it returns a 32-bit, which will wrap either around 2038 or 2106. It is only used on Linux port of oss4 to determine whether a "license" is good. Just conditioning the definition of that to the macro LICENSED_VERSION should be good enough to let it build again. If anyone cares enough, one could use ktime_get_ real_seconds( ), or fix oss_license_ handle_ time(), which is not even present in the code we have.
Cascardo.