Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29667 )
Change subject: mb/emulation/qemu-q35,qemu-i440fx: Add x86_64 support
......................................................................
Patch Set 41:
(1 comment)
https://review.coreboot.org/c/coreboot/+/29667/41/Documentation/arch/x86/in…
File Documentation/arch/x86/index.md:
https://review.coreboot.org/c/coreboot/+/29667/41/Documentation/arch/x86/in…
PS41, Line 52: Identity map memory above 4GiB in ramstag
> isn't this done https://review.coreboot.org/c/coreboot/+/39488
> https://review.coreboot.org/c/coreboot/+/39487/18
That code is about resource allocation above 4G, while this is about page tables covering anything at all above 4G.
--
To view, visit https://review.coreboot.org/c/coreboot/+/29667
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If2f02a95b2f91ab51043d4e81054354f4a6eb5d5
Gerrit-Change-Number: 29667
Gerrit-PatchSet: 41
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich(a)gmail.com>
Gerrit-CC: Nico Huber <nico.h(a)gmx.de>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Comment-Date: Tue, 18 Aug 2020 12:08:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-MessageType: comment
Johnny Lin has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/44539 )
Change subject: [hard-coded offset]mb/ocp/deltalake: Add SMBIOS OEM string for SPD register vendor ID
......................................................................
Abandoned
--
To view, visit https://review.coreboot.org/c/coreboot/+/44539
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I504be36f034562358321c3fbc17ca47664772c60
Gerrit-Change-Number: 44539
Gerrit-PatchSet: 3
Gerrit-Owner: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-MessageType: abandon
Hello Felix Singer,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/44180
to review the following change.
Change subject: payloads/nvramcui: Fix `make clean`
......................................................................
payloads/nvramcui: Fix `make clean`
After `make clean` a new build should not be based on stale artifacts.
Hence we have to remove them.
Change-Id: I540a83a6c87b843b1c4c9c55990bf3e91fe90d79
Signed-off-by: Nico Huber <nico.huber(a)secunet.com>
---
M payloads/nvramcui/Makefile
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/44180/1
diff --git a/payloads/nvramcui/Makefile b/payloads/nvramcui/Makefile
index bf7053b..269d558 100644
--- a/payloads/nvramcui/Makefile
+++ b/payloads/nvramcui/Makefile
@@ -26,9 +26,9 @@
endif
clean:
- rm -f nvramcui.elf
+ rm -rf build libpayload nvramcui.elf
distclean: clean
- rm -rf build libpayload .config .config.old
+ rm -rf .config .config.old
.PHONY: all clean distclean
--
To view, visit https://review.coreboot.org/c/coreboot/+/44180
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I540a83a6c87b843b1c4c9c55990bf3e91fe90d79
Gerrit-Change-Number: 44180
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-MessageType: newchange
Hello Felix Singer,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/44179
to review the following change.
Change subject: payloads/coreinfo: Fix `make clean`
......................................................................
payloads/coreinfo: Fix `make clean`
After `make clean` a new build should not be based on stale artifacts.
Hence we have to remove them.
Change-Id: I18292c674986078d991668124193b6aa31234d47
Signed-off-by: Nico Huber <nico.huber(a)secunet.com>
---
M payloads/coreinfo/Makefile
1 file changed, 1 insertion(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/44179/1
diff --git a/payloads/coreinfo/Makefile b/payloads/coreinfo/Makefile
index 4171796..898b12b 100644
--- a/payloads/coreinfo/Makefile
+++ b/payloads/coreinfo/Makefile
@@ -128,10 +128,9 @@
else
clean:
- rm -rf build/*.elf build/*.o .xcompile
+ rm -rf build lpbuild .xcompile
distclean: clean
- rm -rf build lpbuild
rm -f .config* lp.config*
.PHONY: clean distclean
--
To view, visit https://review.coreboot.org/c/coreboot/+/44179
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I18292c674986078d991668124193b6aa31234d47
Gerrit-Change-Number: 44179
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net>
Gerrit-MessageType: newchange