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

start building kselftests under CI #618

Open
nickdesaulniers opened this issue Aug 25, 2023 · 3 comments
Open

start building kselftests under CI #618

nickdesaulniers opened this issue Aug 25, 2023 · 3 comments

Comments

@nickdesaulniers
Copy link
Member

Once ClangBuiltLinux/linux#1698 is resolved, we should wire up at least 1 build of the kselftests so that they don't regress //building//.

Running them will be more complicated and should be tracked in a second distinct issue, since that will involve fetching the build artifacts, repacking them in our userspace images, then running them upon boot, and then trying to understand the output.

Perhaps one build for x86_64 is sufficient for now.

cc @JustinStitt

@gctucker
Copy link

gctucker commented Sep 4, 2023

Out of interest, are you planning to use make kselftest-merge to create the kernel config?

@nickdesaulniers
Copy link
Member Author

I'm unfamiliar with that make target. Do you recommend that we should be using that instead? Or in addition?

@gctucker
Copy link

gctucker commented Sep 6, 2023

It's the standard upstream way of creating a kernel config with all the fragments for all the kselftests. I think it ought to work, and if it doesn't then the CI should help identify issues. It's something to discuss with Shuah I think in terms of actual recommendation.

FIY So far, KernelCI has been relying on its own way to collate all the kselftest fragments. I think kselftest-merge wasn't really ready when we started. Now with the new API & Pipeline, the build jobs are being reworked and it would seem like a logical move to use it. We'll see how it goes. Also, some kselftests can be run with a plain defconfig build.

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

No branches or pull requests

2 participants