[coreboot-gerrit] [XS] Change in coreboot[master]: allocator_v4: Fix top-level allocations w/o IORESOURCE_ABOVE_4G

[Nico Huber (Code Review)]

Attention is currently required from: Arthur Heymans.

Hello Arthur Heymans,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/76090?usp=email

to look at the new patch set (#3).

Change subject: allocator_v4: Fix top-level allocations w/o IORESOURCE_ABOVE_4G
......................................................................

allocator_v4: Fix top-level allocations w/o IORESOURCE_ABOVE_4G

When moving the code to allocate at the top level in commit 9260ea60bfa4
(allocator_v4: Use memranges only for toplevel), a call to restrict the
limit of the resource was dropped. Probably by accident in one of the
earliest rebases. Without this call to effective_limit(), 64-bit resour-
ces at the top level, i.e. PCI bus 0, were always placed above 4G. Even
when this was not requested with the IORESOURCE_ABOVE_4G flag.

Tested on kontron/ktqm77 where the issue could be reproduced with
x86_64. Without the fix, boot hangs when trying to access the GMA
aperture of PCI 00:02.0, which was placed above 4G.

Change-Id: Ied3a0695ef5e91f092bf2d442c1c482057643483
Signed-off-by: Nico Huber 
Found-by: 9elements QA
---
M src/device/resource_allocator_v4.c
1 file changed, 2 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/76090/3