> -----Original Message-----
> From: [email protected] <[email protected]> On Behalf Of Ni, Ray via
> groups.io
> Sent: Thursday, April 23, 2020 7:58 PM
> To: [email protected]; Wasim Khan <[email protected]>
> Cc: [email protected]; Varun Sethi <[email protected]>; Wu, Hao A
> <[email protected]>
> Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/PciHostBridge: Update Mem
> and PMem Limit Checks
>
> > -----Original Message-----
> > From: [email protected] <[email protected]> On Behalf Of Wasim
> > Khan
> > Sent: Thursday, April 23, 2020 9:52 PM
> > To: Ni, Ray <[email protected]>; [email protected]
> > Cc: [email protected]; Varun Sethi <[email protected]>; Wu, Hao
> > A <[email protected]>
> > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/PciHostBridge: Update
> > Mem and PMem Limit Checks
> >
> >
> >
> > > -----Original Message-----
> > > From: Ni, Ray <[email protected]>
> > > Sent: Thursday, April 23, 2020 5:07 PM
> > > To: Wasim Khan <[email protected]>; [email protected]
> > > Cc: [email protected]; Varun Sethi <[email protected]>; Wu,
> > > Hao A <[email protected]>
> > > Subject: RE: [PATCH] MdeModulePkg/PciHostBridge: Update Mem and
> PMem
> > > Limit Checks
> > >
> > > Thanks for fixing the check.
> > >
> > > PCI_ROOT_BRIDGE_APERTURE.Base/Limit are device address while the
> > > memory space in GCD belongs to host domain.
> > > So, host address for Mem/Pmem should be below 4GB while device
> > > address can across 4GB.
> > >
> >
> >
> > Hi Ray,
> > Thank you for the review.
> > There are cases when we don't have PCIe host address below 4GB, and
> > the PCIe HOST Address space is only available above 4GB.
> > For Example, Mem: 40000000 - FFFFFFFF Translation=FFFFFF6000000000
> > will result in HOST Address = 0xA0FFFFFFFF . This is a valid use case, but
> > below
> check will report ASSERT for this HOST ADDRESS.
>
> OK. Now I remember that "Mem" reports the 32bit memory space (device
> address) and "MemAbove4GB" reports the 64bit memory space (device address).
>
> Then if "Mem" reports memory range that across 4GB, it means the range above
> 4GB should be reported through "MemAbove4GB".
>
Yes this is true, but some devices needs MMIO 32bit space only as per their BAR
property, including E1000 EP.
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#57978): https://edk2.groups.io/g/devel/message/57978
Mute This Topic: https://groups.io/mt/73215737/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
