Nice to see a possible fix. Thank you very much, going to test it tomorrow.

On Thu, May 14, 2020 at 11:29 PM Aaron Durbin via coreboot
<coreboot@coreboot.org> wrote:
>
>
>
> On Thu, May 14, 2020 at 2:25 PM Keith Hui <buu...@gmail.com> wrote:
>>
>> Hi Aaron,
>>
>> I think I have success after applying 41363 as well. Please see
>> attached and tell us if this is what you set out to do.
>
>
> That looks correct, and what I was expecting. Thanks for the help gathering 
> logs.
>
>>
>> Thanks
>> Keith
>>
>> On Thu, May 14, 2020 at 4:01 PM Aaron Durbin <adur...@google.com> wrote:
>> >
>> > Hi Keith,
>> >
>> > Did the newresalloc.log file contain this patch? 
>> > https://review.coreboot.org/c/coreboot/+/41363
>> >
>> > I ask because I see the fixed resources on the domain that are dram:
>> >   DOMAIN: 0000 resource base 0 size a0000 align 0 gran 0 limit 0 flags 
>> > e0004200 index a
>> >   DOMAIN: 0000 resource base c0000 size 2ff40000 align 0 gran 0 limit 0 
>> > flags e0004200 index b
>> >
>> > However, I don't see them being removed the address space selection:
>> > DOMAIN: 0000 mem: base: 0 size: 0 align: 0 gran: 0 limit: fffffffff
>> > update_constraints: PCI: 00:04.0 02 base ff800000 limit ffffffff mem 
>> > (fixed)
>> > Resource ranges:
>> > Base: 0, Size: ff800000, Tag: 200
>> > Base: 100000000, Size: f00000000, Tag: 100200
>> >
>> > update_constraints isn't called for the resources above.
>> >
>> > On Thu, May 14, 2020 at 1:32 PM Keith Hui <buu...@gmail.com> wrote:
>> >>
>> >> Hi Aaron,
>> >>
>> >> Here are two more logs at SPEW. Old log is a separate branch where I
>> >> reverted all the problem commits; new log has everything.
>> >>
>> >> Thanks
>> >> Keith
>
> _______________________________________________
> coreboot mailing list -- coreboot@coreboot.org
> To unsubscribe send an email to coreboot-le...@coreboot.org
_______________________________________________
coreboot mailing list -- coreboot@coreboot.org
To unsubscribe send an email to coreboot-le...@coreboot.org

Reply via email to