Repository navigation
[BUG][Needs Check] IPC worker thread can run on secondary cores #8165
Description
Describe the bug
SOF isn't an SMP system, its use of cores is hard-wired. Most management tasks are only supposed to run on the primary core, only pre-assigned audio processing tasks should run on secondary cores. One of such management tasks is the EDF scheduler work handler. To fix it to the primary core a special work queue is created, which is set to only run on the primary core, and the delayed work, implementing the scheduler, is submitted to that queue.
Whereas the IPC delayed work is run on the default work queue, which on Zephyr can run on any core.
Expected behavior
The IPC delayed work should only run on the primary core.
Impact
Running on a secondary core without proper code synchronisation can lead to data corruption.
Activity
lgirdwood commentedon Sep 13, 2023
@marcinszkudlinski are you also aligned with this proposal ?
kv2019i commentedon Nov 20, 2023
Nobody assigned, less than a week to branch, moving to v2.9.
kv2019i commentedon Mar 4, 2024
Stable-v2.9 branched, this didn't make the cut, bumping to 2.10.
kv2019i commentedon Jan 3, 2025
@lyakh Can you check this and if still valid, ensure this is covered for for v2.13 ? Bumping now as v2.12 has been already branched.
ipc: move delayed IPC sending to the primary core
ipc: move delayed IPC sending to the primary core
ipc: move delayed IPC sending to the primary core
ipc: move delayed IPC sending to the primary core
ipc: move delayed IPC sending to the primary core
ipc: move delayed IPC sending to the primary core