Re: [edk2-devel] Is there an ACPI solution for ARM (or other) platforms that don't have DRAM under 4 GiB?

2021-03-19 Thread Laszlo Ersek
adding Ard, comment below On 03/19/21 02:47, Andrew Fish via groups.io wrote: > Some one was asking me this question and I did not know the answer? > It looks like the stock ACPI stack needs memory under 4 GiB to fill > in 32-bits as it was the 1990’s when that got written down. Is there >

Re: [edk2-devel] Issue in ShellPkg

2021-03-19 Thread Laszlo Ersek
On 03/19/21 17:21, Ramesh R. wrote: > In ShellPkg, file Cp.C, Function CopySingleFile Could see allocating the > buffer to read/write on the BlockIo device. Don't see code that we are free > the memory. So every time user copy the files in Shell, available memory > going to reduce. > >

Re: [edk2-devel] [PATCH v1] OvmfPkg/X86QemuLoadImageLib: Handle allocation failure for CommandLine

2021-03-19 Thread Laszlo Ersek
On 03/19/21 15:39, Martin Radev wrote: > On Fri, Mar 19, 2021 at 03:27:00PM +0100, Ard Biesheuvel wrote: >> On Thu, 18 Mar 2021 at 22:44, Martin Radev wrote: >>> >>> The CommandLine and InitrdData may be set to NULL if the provided >>> size is too large. Because the zero page is mapped, this

Re: [edk2-devel] [PATCH v1] OvmfPkg/X86QemuLoadImageLib: Handle allocation failure for CommandLine

2021-03-19 Thread Laszlo Ersek
On 03/18/21 22:44, Martin Radev wrote: > The CommandLine and InitrdData may be set to NULL if the provided > size is too large. Because the zero page is mapped, this would not > cause an immediate crash but can lead to memory corruption instead. > This patch just adds validation and returns error

[edk2-devel] [RESEND v2] MdePkg: use CpuPause() in CpuDeadLoop()

2021-03-19 Thread Ankur Arora
CpuPause() might allow the CPU to go into a lower power state state while we spin. On X86, CpuPause() executes a PAUSE instruction which the Intel and AMD specs describe as follows: Intel: "PAUSE: An additional function of the PAUSE instruction is to reduce the power consumed by a processor

Re: [edk2-devel] [PATCH v1] OvmfPkg/X86QemuLoadImageLib: Handle allocation failure for CommandLine

2021-03-19 Thread Martin Radev
On Fri, Mar 19, 2021 at 03:27:00PM +0100, Ard Biesheuvel wrote: > On Thu, 18 Mar 2021 at 22:44, Martin Radev wrote: > > > > The CommandLine and InitrdData may be set to NULL if the provided > > size is too large. Because the zero page is mapped, this would not > > cause an immediate crash but can

Re: [edk2-devel] [PATCH v2 1/2] OvmfPkg/IntelGvtGopDxe: Intel GVT-g GOP Implementation.

2021-03-19 Thread Gerd Hoffmann
Hi, > operations but handle the common logic. In this opregion case, QEMU and > seabios only copies the raw data, but the contents are left to gvt to fill > so that gvt could fill different data in different cases. Yes, gvt could do that. The big question is: does that actually happen? When

[edk2-devel] Build Error

2021-03-19 Thread Chow, Ching
Hi all, Receiving the following error when trying to build I45 ToT. Says to email this group. Thanks, Ching build.py... : error C0DE: Unknown fatal error when processing [e:\git\g11plusa\Synergy\HpPlatforms\I45Pkg\PlatformPkg.dsc] (Please send email to

[edk2-devel] Issue in ShellPkg

2021-03-19 Thread Ramesh R.
In ShellPkg, file Cp.C, Function CopySingleFile Could see allocating the buffer to read/write on the BlockIo device. Don't see code that we are free the memory. So every time user copy the files in Shell, available memory going to reduce. // // copy data between files //

Re: [edk2-devel] [PATCH v1] OvmfPkg/X86QemuLoadImageLib: Handle allocation failure for CommandLine

2021-03-19 Thread Lendacky, Thomas
On 3/18/21 4:44 PM, Martin Radev wrote: > The CommandLine and InitrdData may be set to NULL if the provided > size is too large. Because the zero page is mapped, this would not > cause an immediate crash but can lead to memory corruption instead. > This patch just adds validation and returns error

Re: [edk2-devel] [PATCH v1] OvmfPkg/X86QemuLoadImageLib: Handle allocation failure for CommandLine

2021-03-19 Thread Ard Biesheuvel
On Fri, 19 Mar 2021 at 15:39, Martin Radev wrote: > > On Fri, Mar 19, 2021 at 03:27:00PM +0100, Ard Biesheuvel wrote: > > On Thu, 18 Mar 2021 at 22:44, Martin Radev wrote: > > > > > > The CommandLine and InitrdData may be set to NULL if the provided > > > size is too large. Because the zero page

Re: 回复: [edk2-devel] [PATCH v2] MdePkg: use CpuPause() in CpuDeadLoop()

2021-03-19 Thread Laszlo Ersek
Hi Liming, On 03/19/21 02:04, gaoliming wrote: > Ankur: > Seemly, the patch is missing in this mail. I can see the patch body just fine in my mailbox / list folder. Also, the list archives seem to have it: - https://edk2.groups.io/g/devel/message/73018 -

Re: [edk2-devel] [PATCH v1] OvmfPkg/X86QemuLoadImageLib: Handle allocation failure for CommandLine

2021-03-19 Thread Ard Biesheuvel
On Thu, 18 Mar 2021 at 22:44, Martin Radev wrote: > > The CommandLine and InitrdData may be set to NULL if the provided > size is too large. Because the zero page is mapped, this would not > cause an immediate crash but can lead to memory corruption instead. > This patch just adds validation and

[edk2-devel] [PATCH V4 1/1] EmbeddedPkg: DwMmcHcDxe: Add support for Designware SDMMC driver

2021-03-19 Thread Loh, Tien Hock
From: Loh Tien Hock This adds support for Designware SDMMC driver. The SDMMC driver depends on MdeModulePkg/Bus/Sd/, and produces EFI_SD_MMC_PASS_THRU_PROTOCOL. The driver uses MMIO to read/write, and uses gEdkiiNonDiscoverableDeviceProtocolGuid. Platform needs to register device with