Skip to content
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

up_squared_adsp:running tests/kernel/smp/ failed #28590

Closed
KangJianX opened this issue Sep 22, 2020 · 4 comments · Fixed by #29227
Closed

up_squared_adsp:running tests/kernel/smp/ failed #28590

KangJianX opened this issue Sep 22, 2020 · 4 comments · Fixed by #29227
Assignees
Labels
bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@KangJianX
Copy link
Collaborator

Describe the bug
running tests/kernel/smp/ error, it showed assertion failed.

To Reproduce
Steps to reproduce the behavior:

sanitycheck -W -p up_squared_adsp --device-testing -T tests/kernel/smp/ --west-flash="/home/ztest/work/zephyrproject/zephyr/boards/xtensa/up_squared_adsp/tools/up_squared_adsp_flash.sh, /home/ztest/work/sof/rimage/keys/otc_private_key.pem" --device-serial-pty="/home/ztest/work/ambient/set_ambient /home/ztest/work/zephyrproject/zephyr/boards/xtensa/up_squared_adsp/tools/mbterm.py" -x=CONFIG_IPM=y -x=CONFIG_CONSOLE=y -x=CONFIG_LOG_PRINTK=n

see error

*** Booting Zephyr OS build v2.4.0-rc2-23-g2cd556a689e3 ***
ASSERTION FAIL [!z_smp_cpu_mobile()] @ WEST_TOPDIR/zephyr/kernel/include/kswap.h:198
@ WEST_TOPDIR/zephyr/lib/os/assert.c:45

Environment (please complete the following information):
OS: Fedora28
Toolchain: Zephyr-sdk-0.11.3
Commit ID: 2cd556a

@KangJianX KangJianX added the bug The issue is a bug, or the PR is fixing a bug label Sep 22, 2020
@MaureenHelm MaureenHelm added the priority: low Low impact/importance bug label Sep 22, 2020
dcpleung added a commit to dcpleung/zephyr that referenced this issue Sep 22, 2020
In smp_init_top(), it initializes a dummy thread and set it
to be the current thread. The _current_cpu macro asserts
on intel_apl_adsp since interrupts are unmasked at smp entry.
So don't unmask.

Fixes zephyrproject-rtos#28590

Signed-off-by: Daniel Leung <[email protected]>
@dcpleung
Copy link
Member

This will be fixed by #26483

@dcpleung
Copy link
Member

@KangJianX Could you re-test? The SOF PR has been merged which should fix this.

@dcpleung dcpleung linked a pull request Oct 21, 2020 that will close this issue
@KangJianX
Copy link
Collaborator Author

KangJianX commented Oct 22, 2020

@dcpleung I can't re-test, all test cases are skiped when I run (sanitycheck -W -p up_squared_adsp --device-testing -T tests/kernel/smp/ --west-flash="/home/ztest/work/zephyrproject/zephyr/boards/xtensa/up_squared_adsp/tools/up_squared_adsp_flash.sh, /home/ztest/work/sof/rimage/keys/otc_private_key.pem" --device-serial-pty="/home/ztest/work/ambient/set_ambient /home/ztest/work/zephyrproject/zephyr/boards/xtensa/up_squared_adsp/tools/mbterm.py" -x=CONFIG_IPM=y -x=CONFIG_CONSOLE=y -x=CONFIG_LOG_PRINTK=n) this cmd. I block by https://github.com/zephyrproject-rtos/zephyr/issues/29430

@dcpleung dcpleung added the Blocked Blocked by another PR or issue label Dec 11, 2020
@KangJianX
Copy link
Collaborator Author

Verify this issue, it is fixed.

@dcpleung dcpleung removed the Blocked Blocked by another PR or issue label Dec 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
3 participants