search

zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.87k stars 6.62k forks source link

cc3220sf_launchxl fails tests/kernel/interrupt/arch.interrupt #18654

Closed galak closed 5 years ago

galak commented 5 years ago

When run via:

./scripts/sanitycheck -p cc3220sf_launchxl --device-testing --device-serial /dev/serial/by-id/usb-Texas_Instruments_XDS110__03.00.00.02__Embed_with_CMSIS-DAP_CC322001-if00 -s tests/kernel/interrupt/arch.interrupt

***** Booting Zephyr OS build v2.0.0-rc1-208-g4f5186b256df *****
Running test suite interrupt_feature
===================================================================
starting test - test_isr_dynamic
PASS - test_isr_dynamic
===================================================================
starting test - test_nested_isr
Triggering irq : 176
isr0 running !!
isr0 execution completed !!

    Assertion failed at ZEPTriggering irq : 175
isr1 ran !!
HYR_BASE/tests/kernel/interrupt/src/nested_irq.c:72: isr0: (new_val not equal to old_val)
Nested interrupt is not working

FAIL - test_nested_isr
===================================================================
starting test - test_prevent_interruption
PASS - test_prevent_interruption
===================================================================
Test suite interrupt_feature failed.
===================================================================
PROJECT EXECUTION FAILED
ioannisg commented 5 years ago

I could take a quick look

ioannisg commented 5 years ago

The test runs properly on several platforms - I think the observed failure is specific to the platform - maybe one of the IRQ lines used in the test is not wired to the NVIC.

galak commented 5 years ago

The test runs properly on several platforms - I think the observed failure is specific to the platform - maybe one of the IRQ lines used in the test is not wired to the NVIC.

Dropping it to low for now.