debug: temporarily disable dynamic slots for "thread" userspace#10519
debug: temporarily disable dynamic slots for "thread" userspace#10519lyakh wants to merge 1 commit intothesofproject:mainfrom
Conversation
The "thread" userspace variant currently breaks when dynamic debug slots are used. Temporarily disable that configuration until fixed. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
lyakh
requested review from
dbaluta,
kv2019i,
lbetlej,
mmaka1 and
plbossart
as code owners
There was a problem hiding this comment.
Pull request overview
This PR temporarily disables dynamic debug slots for configurations using the SOF userspace proxy feature to work around a known compatibility issue. This is explicitly marked as a temporary fix until the underlying problem is resolved.
Changes:
- Modified the default value of
INTEL_ADSP_DEBUG_SLOT_MANAGERto be conditional, disabling it whenSOF_USERSPACE_PROXYis enabled
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
fixes QB failure in #10469 |
@lgirdwood hard to tell. I know the following - a single internal Intel CI userspace test fails with my PR #10469. That test is passing with the current SOF "main" though. And it's also passing with that PR's head which is 12 commits behind the current "main." So when bisecting those 12 commits it converged on the "enable dynamic debug slot" commit. That's why this PR. OTOH the current SOF "main" with the newer Zephyr version, to which my #10469 is updating is also failing. So that failure can be triggered by one of two updates - either dynamic debug slots or one of the almost 1000 Zephyr commits between those two versions. Currently bisecting them. |
|
Zephyr bisect: on the Zephyr side it's zephyrproject-rtos/zephyr@57b553b that breaks that QB test |
|
seems not needed any more |
2 participants
The "thread" userspace variant currently breaks when dynamic debug slots are used. Temporarily disable that configuration until fixed.