Attention is currently required from: Felix Held, Paul Menzel, Tim Wawrzynczak.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75012?usp=email )
Change subject: allocator_v4: Treat above 4G resources more natively
......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/75012/comment/4e5ba082_ed8ddcbc :
PS4, Line 45: Original-reviewed-on: https://review.coreboot.org/c/coreboot/+/65413
Sorry, the Gerrit search failed me. […]
Ack, will do on the next update.
Patchset:
PS4:
Just learned more AMD details. It looks like we should lower […]
So it turned out that it might be alright because the AMD chipsets can
have multiple windows surrounding the MMConf range and covering address
space up to the current default DOMAIN_RESOURCE_32BIT_LIMIT. Still would
feel better to have it tested on something AMD ;)
--
To view, visit
https://review.coreboot.org/c/coreboot/+/75012?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: Ia822f0ce648c7f7afc801d9cb00b6459fe7cebea
Gerrit-Change-Number: 75012
Gerrit-PatchSet: 4
Gerrit-Owner: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Felix Held
felix-coreboot@felixheld.de
Gerrit-Reviewer: Lean Sheng Tan
sheng.tan@9elements.com
Gerrit-Reviewer: Tim Wawrzynczak
inforichland@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Tim Wawrzynczak
inforichland@gmail.com
Gerrit-Attention: Felix Held
felix-coreboot@felixheld.de
Gerrit-Comment-Date: Fri, 02 Jun 2023 21:10:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Nico Huber
nico.h@gmx.de
Comment-In-Reply-To: Paul Menzel
paulepanter@mailbox.org
Gerrit-MessageType: comment