Attention is currently required from: Jonathan Zhang, Arthur Heymans, Felix Held.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67018 )
Change subject: device/allocator: Allow for multiple domain resources of a type ......................................................................
Patch Set 6:
(1 comment)
File src/device/resource_allocator_v4.c:
https://review.coreboot.org/c/coreboot/+/67018/comment/26e89480_966dd6fe PS6, Line 317: if (res->flags & IORESOURCE_FIXED) : continue;
haven't properly root-caused the behavior i've seen, but when having the IORESOURCE_FIXED flag set i […]
All fixed resources are avoided during allocation. It is our way to tell the allocator that a resource range is already occupied (cf. constrain_ domain_resources()).
The "windows" you open for allocation shoudn't be fixed, and I assume it's also not necessary to tag them subtractive. In ACPI terms, AIUI, one could say all our *non-fixed domain* resources are "ResourceProducer" while everything else is a "ResourceConsumer". And *non-fixed downstream* resources need allocation.