Skip to content

Conversation

@lyakh
Copy link
Collaborator

@lyakh lyakh commented Mar 24, 2025

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

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.

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

@abonislawski
Copy link
Member

abonislawski commented Apr 2, 2025

@lyakh
Copy link
Collaborator Author

lyakh commented Apr 4, 2025

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
Member

@wszypelt good to merge ?

@wszypelt
Copy link

wszypelt commented Apr 7, 2025

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

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

@wszypelt
Copy link

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

kv2019i commented May 9, 2025

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

lyakh commented May 12, 2025

@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
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 <guennadi.liakhovetski@linux.intel.com>
@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

@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