Skip to content

Conversation

JenySadadia
Copy link
Collaborator

Replaced #1262 with this one since it needed rebase.

For now, we only run a (very) limited set of kselftests. However, enabling more of those would allow for greater code coverage. This patch creates kselftest jobs suited to running on the ChromiumOS kernel (in particular, ensuring we don't enable the ChromiumOS downstream LSM, which doesn't work with a Debian userspace).

Those tests are triggered by coverage-enabled kbuilds only, so we can eventually lower the frequency they run.

For now, we only run a (very) limited set of kselftests. However,
enabling more of those would allow for greater code coverage. This
patch creates kselftest jobs suited to running on the ChromiumOS kernel
(in particular, ensuring we don't enable the ChromiumOS downstream LSM,
which doesn't work with a Debian userspace).

Those tests are triggered by coverage-enabled kbuilds only, so we can
eventually lower the frequency they run.

Signed-off-by: Arnaud Ferraris <[email protected]>
@JenySadadia
Copy link
Collaborator Author

@a-wai
Copy link
Contributor

a-wai commented Oct 7, 2025

@JenySadadia IMHO the errors are a consequence of kernelci/kernelci-core@df4d46d : Lava doesn't recognize hatch as a device type, but it does recognize asus-C436FA-Flip-hatch, for instance.

@nuclearcat what motivated this change and has it been thoroughly tested?

@a-wai
Copy link
Contributor

a-wai commented Oct 8, 2025

@JenySadadia Those errors should be fixed with #1329

@nuclearcat
Copy link
Member

@nuclearcat what motivated this change and has it been thoroughly tested?

They are implemented to make qemu work, this was long standing problem and i just reimplemented it as it was in legacy KernelCI.
Obviously it was not well tested, as seems i didnt noticed that some chromebooks was broken. We will have to figure out how to detect broken devices without reviewing every of them manually.

@a-wai
Copy link
Contributor

a-wai commented Oct 17, 2025

Thanks for the rebase! I finally got to curate the list of enabled tests some more and opened #1340 in order to replace this PR, hence closing.

@a-wai a-wai closed this Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants