> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray via
> groups.io
> Sent: Thursday, April 23, 2020 7:58 PM
> To: devel@edk2.groups.io; Wasim Khan <wasim.k...@nxp.com>
> Cc: ard.biesheu...@linaro.org; Varun Sethi <v.se...@nxp.com>; Wu, Hao A
> <hao.a...@intel.com>
> Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/PciHostBridge: Update Mem
> and PMem Limit Checks
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Wasim
> > Khan
> > Sent: Thursday, April 23, 2020 9:52 PM
> > To: Ni, Ray <ray...@intel.com>; devel@edk2.groups.io
> > Cc: ard.biesheu...@linaro.org; Varun Sethi <v.se...@nxp.com>; Wu, Hao
> > A <hao.a...@intel.com>
> > Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/PciHostBridge: Update
> > Mem and PMem Limit Checks
> >
> >
> >
> > > -----Original Message-----
> > > From: Ni, Ray <ray...@intel.com>
> > > Sent: Thursday, April 23, 2020 5:07 PM
> > > To: Wasim Khan <wasim.k...@nxp.com>; devel@edk2.groups.io
> > > Cc: ard.biesheu...@linaro.org; Varun Sethi <v.se...@nxp.com>; Wu,
> > > Hao A <hao.a...@intel.com>
> > > 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: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-