Skip to content

west.yml: update to current Zephyr #10127

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.

Sign up for GitHub

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

Jump to bottom
Merged
kv2019i merged 2 commits into from
Jul 29, 2025
Merged

west.yml: update to current Zephyr #10127

kv2019i merged 2 commits into from
Jul 29, 2025

Conversation

@lyakh
Copy link
Collaborator

@lyakh lyakh commented Jul 21, 2025

Update to Zephyr commit 87917a10874d ("tests: bt: host: mock k_heap_aligned_alloc").

@lyakh lyakh force-pushed the west branch 3 times, most recently from eab7d81 to b3159ba Compare July 21, 2025 10:59
lgirdwood
lgirdwood reviewed Jul 21, 2025
View reviewed changes
Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look fine - but do have we pulled in a regression ?

@lyakh
ci: switch to ubuntu 24.04 for build tests
ebf0224 
clang from Ubuntu 22.04 is too old for the current Zephyr, switch to
24.04 for the "stub-build" CI test.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lyakh
Copy link
Collaborator Author

lyakh commented Jul 21, 2025

Changes look fine - but do have we pulled in a regression ?

@lgirdwood yes zephyrproject-rtos/zephyr#93444 . The new version is trying to update SOF to a commit before that regression - should be enough for us for now

@lyakh
Copy link
Collaborator Author

lyakh commented Jul 21, 2025 •
edited
Loading

Changes look fine - but do have we pulled in a regression ?

@lgirdwood yes zephyrproject-rtos/zephyr#93444 . The new version is trying to update SOF to a commit before that regression - should be enough for us for now

...or did you mean this https://sof-ci.01.org/sofpr/PR10127/build14094/devicetest/index.html ? I think that's zephyrproject-rtos/zephyr#93339

@lgirdwood
Copy link
Member

lgirdwood commented Jul 21, 2025

Changes look fine - but do have we pulled in a regression ?

Changes look fine - but do have we pulled in a regression ?

@lgirdwood yes zephyrproject-rtos/zephyr#93444 . The new version is trying to update SOF to a commit before that regression - should be enough for us for now

...or did you mean this https://sof-ci.01.org/sofpr/PR10127/build14094/devicetest/index.html ? I think that's zephyrproject-rtos/zephyr#93339

I mean all teste were red for jenkins PTL.

@lyakh lyakh added the DNM Do Not Merge tag label Jul 21, 2025
@lyakh lyakh marked this pull request as draft July 21, 2025 14:13
@lyakh
Copy link
Collaborator Author

lyakh commented Jul 25, 2025

IPC timeout on MTL https://sof-ci.01.org/sofpr/PR10127/build14228/devicetest/index.html?model=MTLP_RVP_NOCODEC&testcase=multiple-pause-resume-50

@kv2019i
Copy link
Collaborator

kv2019i commented Jul 25, 2025

@thesofproject/nxp we seem to have imx95 fail with latest Zephyr main? https://github.com/thesofproject/sof/actions/runs/16522474902/job/46727312479?pr=10127
How to proceed? Do we proceed to merge or need to wait until some Zephyr side fix?

@LaurentiuM1234
Copy link
Contributor

LaurentiuM1234 commented Jul 25, 2025

@thesofproject/nxp we seem to have imx95 fail with latest Zephyr main? https://github.com/thesofproject/sof/actions/runs/16522474902/job/46727312479?pr=10127 How to proceed? Do we proceed to merge or need to wait until some Zephyr side fix?

Go for the merge, IMO. We’ll fix this later on SOF side.

@lyakh
Copy link
Collaborator Author

lyakh commented Jul 28, 2025

@lrudyX could you please check QB? Should it be restarted?

@lyakh lyakh marked this pull request as ready for review July 28, 2025 08:03
kv2019i
kv2019i approved these changes Jul 28, 2025
View reviewed changes
Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lot of known fails in the CI, which is less than ideal, but does not seem to made worse by this PR.

@lyakh Did we have an upstream bug for the sparse issue?

@LaurentiuM1234
Copy link
Contributor

LaurentiuM1234 commented Jul 28, 2025

Fix for the MX95 issues submitted via #10139.

@kv2019i @lyakh how should we proceed with this? Merge this one and have the fix merged later on or add the fix in this PR alongside the manifest update?

@kv2019i
Copy link
Collaborator

kv2019i commented Jul 28, 2025

@LaurentiuM1234 Thanks for the quick fix! Are you ok to merge this (will break bisect for MX95 for this commit)? Otherwise @lyakh can take in your fix to this PR (same commit that updates the west.yaml).

@LaurentiuM1234
Copy link
Contributor

LaurentiuM1234 commented Jul 28, 2025

@LaurentiuM1234 Thanks for the quick fix! Are you ok to merge this (will break bisect for MX95 for this commit)? Otherwise @lyakh can take in your fix to this PR (same commit that updates the west.yaml).

hm, let’s try to avoid breaking git bisect and go with the second option if @lyakh doesnt mind

@lyakh
Copy link
Collaborator Author

lyakh commented Jul 28, 2025

@LaurentiuM1234 Thanks for the quick fix! Are you ok to merge this (will break bisect for MX95 for this commit)? Otherwise @lyakh can take in your fix to this PR (same commit that updates the west.yaml).

hm, let’s try to avoid breaking git bisect and go with the second option if @lyakh doesnt mind

@LaurentiuM1234 @kv2019i how about the third option - we merge #10139 first and then this one? Any reason #10139 is still a draft? We just need to wait for QB for it to finish I assume. Although I don't think QB tests imx95 so we don't even have to wait. Just make it ready and merge?

@kv2019i
Copy link
Collaborator

kv2019i commented Jul 28, 2025

#10139 merged, @lyakh please rebase

@lyakh
Copy link
Collaborator Author

lyakh commented Jul 29, 2025

Lot of known fails in the CI, which is less than ideal, but does not seem to made worse by this PR.

@lyakh Did we have an upstream bug for the sparse issue?

@kv2019i yes zephyrproject-rtos/zephyr#93444

@lyakh
west.yml: update to current Zephyr
67b5224 
Update to Zephyr commit c99605126cd9 ("llext: only invalidate
instruction cache where needed").

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@lyakh lyakh removed the DNM Do Not Merge tag label Jul 29, 2025
@lyakh lyakh mentioned this pull request Jul 29, 2025
Closed
@kv2019i kv2019i merged commit 446c576 into thesofproject:main Jul 29, 2025
71 of 83 checks passed
@lyakh lyakh deleted the west branch July 29, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lgirdwood lgirdwood lgirdwood left review comments

@jsarha jsarha jsarha approved these changes

@kv2019i kv2019i kv2019i approved these changes

@plbossart plbossart Awaiting requested review from plbossart plbossart is a code owner

@mmaka1 mmaka1 Awaiting requested review from mmaka1 mmaka1 is a code owner

@lbetlej lbetlej Awaiting requested review from lbetlej lbetlej is a code owner

@dbaluta dbaluta Awaiting requested review from dbaluta dbaluta is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@lyakh @lgirdwood @kv2019i @LaurentiuM1234 @jsarha