run-zephyr-tests: let make build the test boards, in parallel - #11347
Open
lynt-smitka wants to merge 1 commit into
Open
run-zephyr-tests: let make build the test boards, in parallel#11347lynt-smitka wants to merge 1 commit into
lynt-smitka wants to merge 1 commit into
Conversation
Collaborator
|
@lynt-smitka GitHub now reporting merge conflicts on this PR. |
The native_sim shard builds two boards, one after the other. With -j the test target builds them at the same time; the bsim shards build one board each and are unaffected. Locally on four cores, four clean board builds took 479 s one at a time and 194 s together.
lynt-smitka
force-pushed
the
ci-zephyr-tests-builds
branch
from
September 11, 2026 18:48
97aeed3 to
5b2028e
Compare
Author
|
The newer #11350 makes the split better. The only thing left of this PR is nproc, which has only a minor impact. 🙂 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Since #11317 the test target builds every board the suite needs, so the three board steps here build them a second time:
west build -b native_native_simruns once in the step and again insidemake test.Dropping the steps and giving the test target -j$(nproc) also lets the four builds overlap. Locally on four cores the same four clean builds took 479 s one at a time and 194 s together (on the runner the gain is smaller). What overlaps is mostly the serial part of each build, the cmake configure, the generators and the link.
The bsim step stays. Before each bsim board is built,
cptools/pre_zephyr_build_prep.pyruns make in tools/bsim, and the two bsim boards now build at the same time, so both would run it in the same directory at once. Keeping the step means the tools are already built and both calls have nothing to do.