ClangBuiltLinux / dockerimage

https://hub.docker.com/r/clangbuiltlinux/ubuntu/
Apache License 2.0
11 stars 6 forks source link

Dockerfile: Add qemu for RISCV #30

Closed martell closed 4 years ago

martell commented 5 years ago

I presume someone with access to dockerhub has to build this and merge it? :)

nathanchance commented 5 years ago

I should probably make a check script that checks for both the binutils and QEMU binaries...

Done in #31. That should probably be merged first then RISCV can be added to the scripts/check-binaries.sh script.

nathanchance commented 5 years ago

Alright, this should be rebased on master so that the RISCV checks can be added to scripts/check-binaries.sh, then it should be good to go.

martell commented 4 years ago

Wow, I am very late to the party here. I updated everything on top of the script improvements.

nickdesaulniers commented 4 years ago

Hmmm...that's not quite right; the debian package opensbi doesn't contain opensbi-riscv64-virt-fw_jump.bin. https://packages.debian.org/sid/all/opensbi/filelist

nathanchance commented 4 years ago

Do remember that we are on Ubuntu Focal now so make sure you are checking those packages. I am at work right now, I can look tonight when I am home.

nickdesaulniers commented 4 years ago

https://packages.ubuntu.com/focal/all/opensbi/filelist

looks like /usr/lib/riscv64-linux-gnu/opensbi/qemu/virt/fw_jump.elf gets installed, but qemu is searching for a file named opensbi-riscv64-virt-fw_jump.bin

nathanchance commented 4 years ago

We can probably have some logic in boot-utils to find the right file. Does that file work if you tell QEMU to use it?

tpimh commented 4 years ago

Adding -bios /usr/lib/riscv64-linux-gnu/opensbi/qemu/virt/fw_jump.elf to qemu arguments list should be enough.

nickdesaulniers commented 4 years ago

Last night's RISCV boot tests failed

qemu-system-riscv64 could not be found, please install it!

So we need this. @martell , can you please rebase?

nickdesaulniers commented 4 years ago

and add opensbi to the list of installed packages?

nickdesaulniers commented 4 years ago

closing in favor of https://github.com/ClangBuiltLinux/dockerimage/pull/50