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

TLSF usage issue on rv-virt target #12209

Closed
yf13 opened this issue Apr 23, 2024 · 3 comments · Fixed by #12263
Closed

TLSF usage issue on rv-virt target #12209

yf13 opened this issue Apr 23, 2024 · 3 comments · Fixed by #12263
Assignees

Comments

@yf13
Copy link
Contributor

yf13 commented Apr 23, 2024

It seems that TLSF heap manager doesn't work well with rv-virt on Ubuntu 22.04.

Reproduce:

  • Used configurations: rv-virt/knsh64, rv-virt/knsh32, or rv-virt/nsh64
  • Before building, do kconfig-tweak -e MM_TLSF_MANAGER -d MM_DEFAULT_MANAGER to enable TLSF manager.
  • Build nuttx,
  • Run nuttx with qemu 6,2 but can't see NSH> prompt.

Please teach if the usage is incorrect.

@yf13 yf13 changed the title rv-virt TLSF_MANGER usage issue rv-virt TLSF usage issue Apr 23, 2024
@yf13 yf13 changed the title rv-virt TLSF usage issue rv-virt TLSF usage issue Apr 23, 2024
@yf13 yf13 changed the title rv-virt TLSF usage issue TLSF usage issue on rv-virt target Apr 23, 2024
@acassis
Copy link
Contributor

acassis commented Apr 29, 2024

@masayuki2009 @lupyuen did you see this issue?

@xiaoxiang781216
Copy link
Contributor

this is due to compiler generate infinite recursion for ffs on RISCV, @Gary-Hobson will provide a patch fix it.

@Gary-Hobson
Copy link
Contributor

@yf13 The fixed patch has been submitted #12263

You can add the -s -S parameter after the qemu startup command to connect to gdb for debugging

qemu-system-riscv64 -semihosting -M virt,aclint=on -cpu rv64 -smp 8 -bios none -kernel nuttx -nographic -S -s
gdb-multiarch nuttx  -ex "target remote 127.0.0.1:1234"

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 a pull request may close this issue.

5 participants