Re: [edk2-devel] [PATCH v4 23/35] OvmfPkg/XenPlatformPei: Rework memory detection

2019-08-08 Thread Anthony PERARD
On Wed, Aug 07, 2019 at 05:34:32PM +0200, Roger Pau Monné wrote: > On Mon, Jul 29, 2019 at 04:39:32PM +0100, Anthony PERARD wrote: > > When running as a Xen PVH guest, there is no CMOS to read the memory > > size from. Rework GetSystemMemorySize(Below|Above)4gb() so they can > > work without CMOS

Re: [edk2-devel] [PATCH v4 23/35] OvmfPkg/XenPlatformPei: Rework memory detection

2019-08-07 Thread Roger Pau Monné
On Mon, Jul 29, 2019 at 04:39:32PM +0100, Anthony PERARD wrote: > When running as a Xen PVH guest, there is no CMOS to read the memory > size from. Rework GetSystemMemorySize(Below|Above)4gb() so they can > work without CMOS by reading the e820 table. > > Rework XenPublishRamRegions to also care

Re: [edk2-devel] [PATCH v4 23/35] OvmfPkg/XenPlatformPei: Rework memory detection

2019-07-30 Thread Laszlo Ersek
On 07/30/19 13:45, Laszlo Ersek wrote: > On 07/29/19 17:39, Anthony PERARD wrote: >> When running as a Xen PVH guest, there is no CMOS to read the memory >> size from. Rework GetSystemMemorySize(Below|Above)4gb() so they can >> work without CMOS by reading the e820 table. >> >> Rework XenPublishRa

Re: [edk2-devel] [PATCH v4 23/35] OvmfPkg/XenPlatformPei: Rework memory detection

2019-07-30 Thread Laszlo Ersek
On 07/29/19 17:39, Anthony PERARD wrote: > When running as a Xen PVH guest, there is no CMOS to read the memory > size from. Rework GetSystemMemorySize(Below|Above)4gb() so they can > work without CMOS by reading the e820 table. > > Rework XenPublishRamRegions to also care for the reserved and AC

[edk2-devel] [PATCH v4 23/35] OvmfPkg/XenPlatformPei: Rework memory detection

2019-07-29 Thread Anthony PERARD
When running as a Xen PVH guest, there is no CMOS to read the memory size from. Rework GetSystemMemorySize(Below|Above)4gb() so they can work without CMOS by reading the e820 table. Rework XenPublishRamRegions to also care for the reserved and ACPI entry in the e820 table. The region that was add