Skip to content

Commit a7d8ff4

Browse files
Enjia Mainashif
authored andcommitted
tests: common: fix newly added test_nop failing the CI
The newly added testcase test_nop failed the CI. Give RISCV more arch_nop() instructions to archieve one cycle. Signed-off-by: Enjia Mai <enjiax.mai@intel.com>
1 parent 5f6e257 commit a7d8ff4

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

tests/kernel/common/src/irq_offload.c

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,11 @@ __no_optimization void test_nop(void)
117117

118118
arch_nop();
119119

120-
#if defined(CONFIG_ARMV6_M_ARMV8_M_BASELINE)
120+
#if defined(CONFIG_RISCV)
121+
/* do 2 nop instructions to cost cycles */
122+
arch_nop();
123+
arch_nop();
124+
#elif defined(CONFIG_ARMV6_M_ARMV8_M_BASELINE)
121125
/* do 4 nop instructions to cost cycles */
122126
arch_nop();
123127
arch_nop();

tests/kernel/common/testcase.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ tests:
1717
- CONFIG_MISRA_SANE=y
1818
kernel.common.nano32:
1919
tags: kernel userspace
20-
skip: true
2120
filter: not CONFIG_KERNEL_COHERENCE
2221
extra_configs:
2322
- CONFIG_CBPRINTF_NANO=y

0 commit comments

Comments
 (0)