-
Notifications
You must be signed in to change notification settings - Fork 349
ptl: library-manager: enable library authentication #9922
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
Conversation
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.
This make perfect sense for LL kernel modules but not for userspace DP modules protected by MMU.
|
Required zephyr patch: |
|
zephyrproject-rtos/zephyr#88019 has been merged, this can be merged now too |
|
@wszypelt good to merge ? |
|
@lyakh @lgirdwood Internal Intel CI System, issue with loadable downmixer |
@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 ? |
|
@lgirdwood the error appears after trying to load the downmixer, I sent the test logs directly to @lyakh |
|
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 |
|
@lyakh Can you rebase? The v2.13 stable branch is on Monday. We might need to backport this to stable if this takes more time. |
|
@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>
|
@lyakh Internal Intel CI tests, all green :) |
Library authentication is enabled on MTL and LNL, enable it on PTL too.