-
Notifications
You must be signed in to change notification settings - Fork 265
feat: support risc-v #709
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
base: master
Are you sure you want to change the base?
feat: support risc-v #709
Conversation
@dendibakh I added some support for risc-v. I can't really test this, but the result looks sane. |
Requires KDAB/perfparser#39 to be merged. Fixes: #705
Thanks @lievenhey! Is there an easy way for me to test this? IIRC, building hotspots from sources is not trivial. But if I can update a few libs and test, then I can test the fix. |
@dendibakh there is a ci job running which builds an appimage. You can download the current appimage here https://github.com/KDAB/hotspot/actions/runs/14440980603?pr=709 |
Thanks @lievenhey. The appimage is x86, so I can't install it on my P550 board. However, I installed it on a Linux x86 system and opened the profile there. I can see the correct function names, which is good. |
Yeah the disassembly is a little bit funky but I am working on a better solution. |
I am still not sure if hotspot works with bigger recordings of riscv programs. But I don't have a risc-v board so I can't test it. |
I see. How hard do you think it would be to add the riscv disasm support to hotspot (assuming you have access to a riscv board)? Anyway, from my perspective, this patch is already a step forward. |
We are using |
I installed riscv binutils with
|
Can you get us a backtrace for the crash by any chance? Ideally self-compiled? |
@milianw , I tried getting a stack trace but it seems the image provided by @lievenhey doesn't have symbols. So, when I run it under gdb it shows no useful data.
I don't know how to build hotspots and I'm afraid it will take me a lot of time to do it. The crash that I see can be reproduced on an x86 system as I described above. |
@dendibakh Been there before - but don't be afraid, the CI run also has debug info and there's a documentation how to use it (please suggest changes if you stumble over something as this was written after doing it more than once, sometimes you leave important pieces out then...) https://github.com/KDAB/hotspot/blob/master/HACKING.md#debugging-the-appimage |
Thanks @GitMensch , that helps!
@milianw , @lievenhey , let me know if anything else is needed. |
As there are two possible null references - can you execute |
Requires KDAB/perfparser#39 to be merged. Fixes: #705