Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kselftest run all subsuites #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

roxell
Copy link
Contributor

@roxell roxell commented Jul 8, 2024

Make it possible to run all kselftest subsuites.

Also made it possible to set the KSELFTESTROOT and the LTPROOT via '--env'.

When passing in another LTPROOT path than the default one '/opt/ltp/',
kirk complains that it can't find the LTPROOT since it looks for it in
the default LTPROOT path.  By setting the LTPROOT with '--env
LTPROOT=/my/own/path/to/ltp' it will corretly pick up the new path and
run the tests.

Signed-off-by: Anders Roxell <[email protected]>
When passign another KSELFTESTROOT path than the default one, kirk
complains that it can't find the KSELFTESTROOT. Because it still assumes
the root is the default one '/opt/linux/tools/testing/selftests'.

Set the new KSELFTESTROOT with '--env KSELFTESTROOT=/path/to/the/new/kselftest_root'
and update sef.env also update self._root with this new path.

Signed-off-by: Anders Roxell <[email protected]>
@acerv
Copy link
Collaborator

acerv commented Jul 8, 2024

Our first implementation was exactly this one: we took a kselftests folder and we run all tests inside it.
The problem came when ebpf kselftests were supported: in that case we needed to ask test_progs --list the available tests in the suite.

At the same time, each kselftests suite doesn't follow any rule. Most of the times they use their own framework or plain C to test some features of the kernel. This (of course) requires a separate function for each one of them (and there it comes the new implementation).

@acerv acerv self-assigned this Jul 8, 2024
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.

2 participants