Skip to content

Conversation

@softwarecki
Copy link
Collaborator

Update Zephyr revision to 6cd7cfa104a5f4de68408c9bf6516cd1109b9782 to 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.

Copy link

Copilot AI left a 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 6cd7cfa104a5f4de68408c9bf6516cd1109b9782 which 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.

@kv2019i kv2019i changed the title ptl: Enable userspace support for PTL ptl: Enable userspace support for PTL (includes Zephyr Dec02 update) Dec 2, 2025
Copy link
Member

@abonislawski abonislawski left a 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

@lgirdwood
Copy link
Member

Looks like we need #10406

Merged. Will restart CI

@lgirdwood
Copy link
Member

SOFCI TEST

@softwarecki softwarecki force-pushed the userspace-enable-tiny-12 branch from dd0d3cf to f57ed63 Compare December 2, 2025 11:11
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.

@softwarecki can you check internal CI thanks !

@softwarecki softwarecki force-pushed the userspace-enable-tiny-12 branch from f57ed63 to aeadccf Compare December 2, 2025 19:05
@softwarecki
Copy link
Collaborator Author

Rebased to include #10409. Now CI should pass :)

@softwarecki
Copy link
Collaborator Author

CI failed because there was no disk space.

/github/home/zephyr-sdk-0.17.4/xtensa-nxp_imx8ulp_adsp_zephyr-elf/bin/xtensa-nxp_imx8ulp_adsp_zephyr-elf-ar: modules/sof/libmodules_sof.a: No space left on device

@softwarecki
Copy link
Collaborator Author

SOFCI TEST

1 similar comment
@abonislawski
Copy link
Member

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>
@kv2019i
Copy link
Collaborator

kv2019i commented Dec 3, 2025

@softwarecki The ptl CI build (with debug on) fails to:

[570/592] Generating kobject_prebuilt_hash.gperf
FAILED: zephyr/kobject_prebuilt_hash.gperf /srv/home/jenkins/workspace/sof_config_build@2/build-ptl/zephyr/kobject_prebuilt_hash.gperf 
cd /srv/home/jenkins/workspace/sof_config_build@2/build-ptl/zephyr && /usr/bin/python3 /srv/home/jenkins/workspace/sof_config_build@2/zephyr/scripts/build/gen_kobject_list.py --kernel /srv/home/jenkins/workspace/sof_config_build@2/build-ptl/zephyr/zephyr_pre0.elf --gperf-output kobject_prebuilt_hash.gperf --include-subsystem-list /srv/home/jenkins/workspace/sof_config_build@2/build-ptl/zephyr/misc/generated/struct_tags.json
Too many thread objects (21)
Increase CONFIG_MAX_THREAD_BYTES to 3
ninja: build stopped: subcommand failed.

PTL supports user-space, enable it by default.

Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
@softwarecki softwarecki force-pushed the userspace-enable-tiny-12 branch from aeadccf to 2003d2e Compare December 3, 2025 17:39
@softwarecki
Copy link
Collaborator Author

@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.

@kv2019i
Copy link
Collaborator

kv2019i commented Dec 4, 2025

I think we can proceed to merge:

@kv2019i kv2019i merged commit 9cc7ad9 into thesofproject:main Dec 4, 2025
54 of 60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants