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` -- To view, visit https://review.coreboot.org/c/coreboot/+/35247 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Gerrit-Change-Number: 35247 Gerrit-PatchSet: 1 Gerrit-Owner: Philipp Hug <philipp@hug.cx> Gerrit-MessageType: newchange
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 -- To view, visit https://review.coreboot.org/c/coreboot/+/35247 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Gerrit-Change-Number: 35247 Gerrit-PatchSet: 2 Gerrit-Owner: Philipp Hug <philipp@hug.cx> Gerrit-MessageType: newpatchset
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 -- To view, visit https://review.coreboot.org/c/coreboot/+/35247 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Gerrit-Change-Number: 35247 Gerrit-PatchSet: 2 Gerrit-Owner: Philipp Hug <philipp@hug.cx> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-Comment-Date: Tue, 10 Sep 2019 17:27:38 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
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? -- To view, visit https://review.coreboot.org/c/coreboot/+/35247 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Gerrit-Change-Number: 35247 Gerrit-PatchSet: 2 Gerrit-Owner: Philipp Hug <philipp@hug.cx> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Patrick Georgi <pgeorgi@google.com> Gerrit-Comment-Date: Wed, 11 Sep 2019 14:44:18 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
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. -- To view, visit https://review.coreboot.org/c/coreboot/+/35247 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Gerrit-Change-Number: 35247 Gerrit-PatchSet: 2 Gerrit-Owner: Philipp Hug <philipp@hug.cx> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: Philipp Hug <philipp@hug.cx> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Patrick Georgi <pgeorgi@google.com> Gerrit-Comment-Date: Thu, 12 Sep 2019 09:06:17 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
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. -- To view, visit https://review.coreboot.org/c/coreboot/+/35247?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1257f474e23da0a00e149c8ba8b5ec33253a7b98 Gerrit-Change-Number: 35247 Gerrit-PatchSet: 2 Gerrit-Owner: Philipp Hug <philipp@hug.cx> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: Philipp Hug <philipp@hug.cx> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Martin L Roth <gaumless@gmail.com> Gerrit-MessageType: abandon
participants (4)
-
Martin L Roth (Code Review) -
Patrick Georgi (Code Review)
-
Patrick Rudolph (Code Review)
-
Philipp Hug (Code Review)