west.yml: update zephyr to bfa1218a9e63 (Aug 14th) - #11098
Conversation
CONFIG_SMP_BOOT_DELAY was removed in Zephyr (commit 7a72151634e8 "kernel: smp: replace SMP_BOOT_DELAY with per-CPU devicetree deferral") in favor of the per-CPU 'zephyr,deferred-start' devicetree property. SOF relied on CONFIG_SMP_BOOT_DELAY=y (default y if SMP) so that no secondary DSP core was started at kernel boot; the cores are brought up on demand by SOF/host power management. Add per-board overlays marking every secondary core (cpu1..cpuN-1) with 'zephyr,deferred-start' to keep that behavior. The boot core (cpu0) is unaffected by the property. Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Total of 1438 commits.
Changes include:
7a72151634e8 kernel: smp: replace SMP_BOOT_DELAY with per-CPU
devicetree deferral
8ab6cd9190ad arch: xtensa: mmu: defer TLB flush on ASID free using
dirty bitmap
3590f4b45b6a arch: xtensa: mmu: add PTEVADDR 0x60000000 option, cap
ASID range
cf244b983d8a arch: xtensa: mmu: add ASID recycling for memory domains
78ca43befcd4 xtensa: skip mmu/mpu swapping if same memory domain
54a45000c0d0 xtensa: move CONFIG_ARCH_HAS_DIRECTED_IPIS to SoC
d9db3de0d46b arch: xtensa: fix arch_user_string_nlen() reading past
maxsize
d73ef2de7b83 drivers: dai: intel: ssp: Fix aux data size check for DMA
transmission
48fb236912d9 counter: ace_v1x: fix header include issue
85c1c4c21945 kernel: userspace: reject size overflow in dynamic object
allocation
2cb53feca11e llext-edk: Properly get topdir when --sysbuild is used
d46238fd276a logging: do not pass an uninitialized spinlock key
da95dbff265e logging: log_mgmt: Add missing cache releasing
Additional changes:
- app/Kconfig: drop the SMP_BOOT_DELAY default, because Zephyr commit
7a72151634e8 replaced CONFIG_SMP_BOOT_DELAY with per-CPU devicetree
deferral and the symbol no longer exists.
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
There was a problem hiding this comment.
Pull request overview
Updates the Zephyr dependency revision and adapts Intel ADSP SMP bring-up behavior by removing an app-level Kconfig default and replacing it with board-specific devicetree overlays that defer secondary CPU start.
Changes:
- Bump
west.ymlZephyrrevisiontobfa1218a9e63a9621316178ca2c3125d16844a96. - Remove the
SMP_BOOT_DELAYKconfig symbol default fromapp/Kconfig. - Add per-board
.overlayfiles to applyzephyr,deferred-startto secondary CPU nodes on affected Intel ADSP boards.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| west.yml | Updates the pinned Zephyr revision in the west manifest. |
| app/Kconfig | Drops the former SMP boot-delay default Kconfig symbol. |
| app/boards/intel_adsp_cavs25.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_cavs25_tgph.overlay | Defers start of secondary CPU via zephyr,deferred-start. |
| app/boards/intel_adsp_ace40_nvls.overlay | Defers start of secondary CPU via zephyr,deferred-start. |
| app/boards/intel_adsp_ace40_nvl.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_ace30_wcl.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_ace30_wcl_sim.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_ace30_ptl.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_ace30_ptl_sim.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_ace20_lnl.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
| app/boards/intel_adsp_ace15_mtpm.overlay | Defers start of secondary CPUs via zephyr,deferred-start. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
kv2019i
changed the title
|
We are having some CI fails due to cmake minimum-req bump in Zephyr commit: |
@kv2019i Yes, bumping newer cmake in a pipeline failed at first try, I will let you know, when the pipeline is ready. |
|
I solved this directly in DTS: zephyrproject-rtos/zephyr#116943 without creating additional files. IMO those overlays would come in handy if we had plans for changing this behavior (starting FW with primary core only) in the future. So please decide. |
|
@wjablon1 wrote:
This is a bit tricky. We have two complicating factors:
Unless anyone objects (probably would be someone outside Intel), I think we can proceed with your Zephyr PR. This PR is anyways stuck due to cmake dependencies. I'll keep this open until your PR is merged (the overlays I add here won't cause problem, they will just become unnecessary). If in the future, somebody wants to support CONFIG_SMP with a new target in SOF, either one needs to add deferred attribute on Zephyr side, add an overlay like here (= allows to use traditional Zephyr SMP, maybe for some other app than SOF), or implement new code in SOF. |
No description provided.