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

Os spinlock test #2375

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Os spinlock test #2375

wants to merge 2 commits into from

Conversation

TaiJuWu
Copy link
Contributor

@TaiJuWu TaiJuWu commented May 5, 2024

Summary

Os spinlock test

Impact

Test for spinlock

Testing

local test

Two threads use share variable and the variable should be same as
excepted.

Signed-off-by: TaiJu Wu <[email protected]>
@TaiJuWu
Copy link
Contributor Author

TaiJuWu commented May 6, 2024

@acassis @xiaoxiang781216 Do you have any idea about the fail cases?
I have no idea what make these testcases fail.

@xiaoxiang781216
Copy link
Contributor

@acassis @xiaoxiang781216 Do you have any idea about the fail cases? I have no idea what make these testcases fail.

here is the error:

====================================================================================
Configuration/Tool: rv-virt/knetnsh64_smp
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0

100  496k  100  496k    0     0  1736k      0 --:--:-- --:--:-- --:--:-- 1736k
/github/workspace/sources/apps /github/workspace/sources/nuttx
Warning: fpu.c:55:8: warning: #warning "FPU test not built; Only available in the flat build (CONFIG_BUILD_FLAT)" [-Wcpp]
   55 | #      warning "FPU test not built; Only available in the flat build (CONFIG_BUILD_FLAT)"
      |        ^~~~~~~
riscv-none-elf-ld: /github/workspace/sources/apps/libapps.a(spinlock.c.github.workspace.sources.apps.testing.ostest_1.o): in function `thread_native_spinlock':
/github/workspace/sources/apps/testing/ostest/spinlock.c:60:(.text.thread_native_spinlock+0x42): undefined reference to `spin_lock'
riscv-none-elf-ld: /github/workspace/sources/apps/testing/ostest/spinlock.c:62:(.text.thread_native_spinlock+0x54): undefined reference to `spin_unlock'
riscv-none-elf-ld: /github/workspace/sources/apps/libapps.a(spinlock.c.github.workspace.sources.apps.testing.ostest_1.o): in function `thread_wrt_spinlock':
/github/workspace/sources/apps/testing/ostest/spinlock.c:128:(.text.thread_wrt_spinlock+0x42): undefined reference to `write_lock'
riscv-none-elf-ld: /github/workspace/sources/apps/testing/ostest/spinlock.c:131:(.text.thread_wrt_spinlock+0x54): undefined reference to `write_unlock'
riscv-none-elf-ld: /github/workspace/sources/apps/libapps.a(spinlock.c.github.workspace.sources.apps.testing.ostest_1.o): in function `thread_read_spinlock':
/github/workspace/sources/apps/testing/ostest/spinlock.c:109:(.text.thread_read_spinlock+0x56): undefined reference to `read_lock'
riscv-none-elf-ld: /github/workspace/sources/apps/testing/ostest/spinlock.c:113:(.text.thread_read_spinlock+0x78): undefined reference to `read_unlock'
make[3]: *** [/github/workspace/sources/apps/Application.mk:251: /github/workspace/sources/apps/bin/ostest] Error 1
make[2]: *** [Makefile:52: /github/workspace/sources/apps/testing/ostest_install] Error 2
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [Makefile:97: .import] Error 2
make: *** [Makefile:103: import] Error 2
/github/workspace/sources/nuttx

@acassis
Copy link
Contributor

acassis commented Jun 26, 2024

ping @TaiJuWu
Did you test it again?

@TaiJuWu
Copy link
Contributor Author

TaiJuWu commented Jun 27, 2024

ping @TaiJuWu Did you test it again?

Hi @acassis , I have no idea how to resolve these fail tests.
If anyone know how to reslove this issue, welcome to take it over.

@acassis
Copy link
Contributor

acassis commented Jun 27, 2024

@xiaoxiang781216 any suggestion?

@cederom
Copy link

cederom commented Nov 2, 2024

ping :-)

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.

4 participants