Re: Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Keith Hui]

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

Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Aaron Durbin]

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  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

Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Keith Hui]

It did not. Let me grab that and take another log.

On Thu, May 14, 2020 at 4:01 PM Aaron Durbin  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  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

Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Keith Hui]

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.

Thanks
Keith

On Thu, May 14, 2020 at 4:01 PM Aaron Durbin  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  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

Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Aaron Durbin]

On Thu, May 14, 2020 at 2:25 PM Keith Hui  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  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  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

Change in coreboot[master]: nb/intel/i440bx: add resources during read_resources()

[Mike Banon]

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
 wrote:
> On Thu, May 14, 2020 at 2:25 PM Keith Hui  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  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  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 -- [e-mail]
> To unsubscribe send an email to [e-mail]