Warmup #10
Merged
Warmup #10
Conversation
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
It was incorrectly set (copy/paste) from kirkstone. Signed-off-by: Nicolas Dechesne <[email protected]>
This is a cosmetic change, it seems better to have the runs-on section earlier in the job definition. Signed-off-by: Nicolas Dechesne <[email protected]>
We want to split the compile job matrix into two steps. In order to avoid too much duplication of code, let's first isolate the compile steps into a custom , local composite action. In the next commit, we will split the matrix into two, and reuse the action. Signed-off-by: Nicolas Dechesne <[email protected]>
The matrix jobs results in a large amount of jobs being started at the same time. Running all jobs in parallel is not ideal to leverage the OE sstate cache. For any common tasks (native, armv8, .. ) when the object is missing from the sstate cache, it's likely going to be a miss for several jobs. As a result, each of them will build the missed object, and each of them will attempt to create the object in the sstate cache. Instead, with this change, we first build for the qcs8300 platform, for both BSP variants. If these builds succeed we carry on with the other machines/configs, with the hope that the sstate cache hit will be better. Signed-off-by: Nicolas Dechesne <[email protected]>
ricardosalveti
approved these changes
Sep 3, 2025
lumag
approved these changes
Sep 3, 2025
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.
No description provided.