-
Notifications
You must be signed in to change notification settings - Fork 349
ptl: Enable userspace support for PTL (includes Zephyr Dec02 update) #10408
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.
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
ptl: Enable userspace support for PTL (includes Zephyr Dec02 update) #10408
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR enables userspace support for the PTL platform by updating the Zephyr revision to include userspace fixes and implementing a workaround to disable DP deadline recalculation for userspace DP threads.
Key changes:
- Updates Zephyr to revision
6cd7cfa104a5f4de68408c9bf6516cd1109b9782which includes userspace fixes - Conditionally disables DP deadline recalculation for userspace threads as a temporary measure
- Enables userspace by default on the PTL platform configuration
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| west.yml | Updates Zephyr revision to include userspace fixes |
| src/schedule/zephyr_dp_schedule.c | Adds conditional logic to skip DP scheduler access in userspace threads |
| app/boards/intel_adsp_ace30_ptl.conf | Enables userspace configuration for PTL platform |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
abonislawski
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we need #10406
Merged. Will restart CI |
|
SOFCI TEST |
dd0d3cf to
f57ed63
Compare
lgirdwood
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@softwarecki can you check internal CI thanks !
f57ed63 to
aeadccf
Compare
|
Rebased to include #10409. Now CI should pass :) |
|
CI failed because there was no disk space.
|
|
SOFCI TEST |
1 similar comment
|
SOFCI TEST |
Total of 91 commits. Changes include:
6cd7cfa104a xtensa: mmu: ptables: Set PPN in region_map_update to fix
memory mapping
942b1e85715 xtensa: mmu: ptables: Introduce PTE_PPN_SET macro
f5f56113fe2 xtensa: mmu: ptables: Introduce PTE_PPN_GET macro
53eaf0f71ee llext: llext_priv: INSTR_FETCHABLE missing parentheses in
definition
Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
Disable DP tasks deadline recalculation from DP threads running in userspace. Recalculation requires information about other DP threads, which is not available from a DP thread operating in userspace. This is a temporary change until a better solution is found. Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
|
@softwarecki The ptl CI build (with debug on) fails to: |
PTL supports user-space, enable it by default. Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
aeadccf to
2003d2e
Compare
|
@lgirdwood @kv2019i There seem to be only minor on-device test failures left. If you think it's appropriate, please restart CI, as it seems I can't trigger this with a "SOFCI TEST" comment. |
|
I think we can proceed to merge:
|
Update Zephyr revision to
6cd7cfa104a5f4de68408c9bf6516cd1109b9782to include the latest userspace fixes (zephyrproject-rtos/zephyr#99254).Temporarily disable DP deadline recalculation in userspace DP threads.
Enable support for the PTL platform by default.