Skip to content

ptl: library-manager: enable library authentication#9922

Merged
kv2019i merged 1 commit into
thesofproject:mainfrom
lyakh:auth
May 15, 2025
Merged

ptl: library-manager: enable library authentication#9922
kv2019i merged 1 commit into
thesofproject:mainfrom
lyakh:auth

Conversation

@lyakh

@lyakh lyakh commented Mar 24, 2025

Copy link
Copy Markdown
Collaborator

Library authentication is enabled on MTL and LNL, enable it on PTL too.

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This make perfect sense for LL kernel modules but not for userspace DP modules protected by MMU.

@abonislawski

abonislawski commented Apr 2, 2025

Copy link
Copy Markdown
Member

@lyakh

lyakh commented Apr 4, 2025

Copy link
Copy Markdown
Collaborator Author

zephyrproject-rtos/zephyr#88019 has been merged, this can be merged now too

@kv2019i kv2019i changed the title ptl: library-manager: enable library authentication ptl: library-manager: enable library authentication (includes west.yml update) Apr 4, 2025
@lgirdwood

Copy link
Copy Markdown
Member

@wszypelt good to merge ?

@wszypelt

wszypelt commented Apr 7, 2025

Copy link
Copy Markdown

@lyakh @lgirdwood Internal Intel CI System, issue with loadable downmixer
09:46:31,358 INFO - E FW status: 0x50000005 09:46:31,358 INFO - E IPC error: 6 (ADSP_IPC_FAILURE) 09:46:31,358 INFO - E ADSP mailbox error: 108 (ADSP_FREE_SHARED_VAR_FAILED_BAD_ALIGNMENT)

@lgirdwood

Copy link
Copy Markdown
Member

@lyakh @lgirdwood Internal Intel CI System, issue with loadable downmixer 09:46:31,358 INFO - E FW status: 0x50000005 09:46:31,358 INFO - E IPC error: 6 (ADSP_IPC_FAILURE) 09:46:31,358 INFO - E ADSP mailbox error: 108 (ADSP_FREE_SHARED_VAR_FAILED_BAD_ALIGNMENT)

@wszypelt are you able to share any more info on this failure. Does it main mailbox data is corrupted/not aligned or does it mean a module free() IPC failed ?

@wszypelt

Copy link
Copy Markdown

@lgirdwood the error appears after trying to load the downmixer, I sent the test logs directly to @lyakh

@wszypelt

Copy link
Copy Markdown

The problem is the key that is currently being used to sign the libs, a ticket has been created to the internal service to change it, I will let you know as soon as it is done

@kv2019i kv2019i added this to the v2.13 milestone Apr 28, 2025
@kv2019i

kv2019i commented May 9, 2025

Copy link
Copy Markdown
Collaborator

@lyakh Can you rebase?
@abonislawski @wszypelt any update on updating the signing key issue in quickbuild?

The v2.13 stable branch is on Monday. We might need to backport this to stable if this takes more time.

@lyakh

lyakh commented May 12, 2025

Copy link
Copy Markdown
Collaborator Author

@lyakh Can you rebase?

@kv2019i sure, but I think it's best to wait for a QB fix first to have the test pass?

@abonislawski

Copy link
Copy Markdown
Member

@lyakh it should be done now, please rebase to test it

Library authentication is enabled on MTL and LNL, enable it on PTL
too.

Signed-off-by: Guennadi Liakhovetski <[email protected]>
@kv2019i kv2019i changed the title ptl: library-manager: enable library authentication (includes west.yml update) ptl: library-manager: enable library authentication May 15, 2025
@wszypelt

Copy link
Copy Markdown

@lyakh Internal Intel CI tests, all green :)

@kv2019i kv2019i merged commit b193c48 into thesofproject:main May 15, 2025
42 of 48 checks passed
@lyakh lyakh deleted the auth branch May 15, 2025 10:51
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.

7 participants