Change in coreboot[master]: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0
Philipp Hug has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35247 )
Change subject: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0 ......................................................................
docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0
Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Signed-off-by: Philipp Hug philipp@hug.cx --- M Documentation/mainboard/emulation/qemu-riscv.md 1 file changed, 5 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/35247/1
diff --git a/Documentation/mainboard/emulation/qemu-riscv.md b/Documentation/mainboard/emulation/qemu-riscv.md index 3aad816..97a4532 100644 --- a/Documentation/mainboard/emulation/qemu-riscv.md +++ b/Documentation/mainboard/emulation/qemu-riscv.md @@ -1,8 +1,11 @@ # Qemu RISC-V emulator
-## Building coreboot and running it in Qemu +## Building coreboot and running it in Qemu >= v4.1.0
-- Configure coreboot and run `make` as usual +- Configure coreboot and run`make`as usual +- Run `qemu-system-riscv64 -M virt -m 1024M -nographic -bios build/coreboot.rom` + +## Building coreboot and running it in Qemu < v4.1.0 - Run `util/riscv/make-spike-elf.sh build/coreboot.rom build/coreboot.elf` to convert coreboot to an ELF that Qemu can load - Run `qemu-system-riscv64 -M virt -m 1024M -nographic -kernel build/coreboot.elf`
Philipp Hug has uploaded a new patch set (#2). ( https://review.coreboot.org/c/coreboot/+/35247 )
Change subject: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0 ......................................................................
docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0
qemu v4.1.0 supports "-bios coreboot.rom"
Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Signed-off-by: Philipp Hug philipp@hug.cx --- M Documentation/mainboard/emulation/qemu-riscv.md 1 file changed, 5 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/35247/2
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35247 )
Change subject: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0 ......................................................................
Patch Set 2: Code-Review+2
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35247 )
Change subject: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0 ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35247/2/Documentation/mainboard/emu... File Documentation/mainboard/emulation/qemu-riscv.md:
https://review.coreboot.org/c/coreboot/+/35247/2/Documentation/mainboard/emu... PS2, Line 5: run`make`as there are some spaces missing here, no?
Philipp Hug has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35247 )
Change subject: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0 ......................................................................
Patch Set 2: Code-Review-1
Hold this until it is clear we will load coreboot to begin of RAM on qemu. This might be changed and will affect how qemu is invoked.
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/35247?usp=email )
Change subject: docs/mb/qemu-riscv: elf conversion no longer needed with qemu v4.1.0 ......................................................................
Abandoned
This patch has not been touched in over 12 months. Anyone who wants to take over work on this patch, please feel free to restore it and do any work needed to get it merged. If you create a new patch based on this work, please credit the original author.
-
Martin L Roth (Code Review) -
Patrick Georgi (Code Review)
-
Patrick Rudolph (Code Review)
-
Philipp Hug (Code Review)