Hi Ard,
On 11/23/18 13:14, Ard Biesheuvel wrote:
> The ArmVirtQemu targets currently limit the size of the IPA space to
> 40 bits because that is all what KVM supports. However, this is about
> to change, and so we need to update the code if we want to ensure that
> our UEFI firmware builds can
On 11/23/18 13:14, Ard Biesheuvel wrote:
> The ArmVirtQemu targets currently limit the size of the IPA space to
> 40 bits because that is all what KVM supports. However, this is about
> to change, and so we need to update the code if we want to ensure that
> our UEFI firmware builds can keep
On 11/26/18 10:42, Laszlo Ersek wrote:
> On 11/23/18 13:14, Ard Biesheuvel wrote:
>> + MaxAddress = (1UL << MaxAddressBits) - 1;
> not 1 U;, which
Sorry, typo: s/;/L/, clearly. (They are adjacent in my keyboard layout.)
Laszlo
___
edk2-devel mailing
On 11/26/18 10:42, Laszlo Ersek wrote:
> On 11/23/18 13:14, Ard Biesheuvel wrote:
>> In preparation of permitting the virt code to define a larger PA space
>> size via gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize than what the
>> CPU actually supports, take the CPU's capabilities into account
On 11/23/18 13:14, Ard Biesheuvel wrote:
> Move the call to BuildCpuHob () into ArmVirtMemoryInitPeiLib (which is
> shared between all the ArmVirtPkg targets), and drop the inclusion of
> CpuPei.inf [which calls it on ArmVirtQemu] and the BuildCpuHob () call
> from ArmVirtPrePiUniCoreRelocatable
On Mon, 26 Nov 2018 at 11:22, Laszlo Ersek wrote:
>
> On 11/23/18 13:14, Ard Biesheuvel wrote:
> > The ArmVirtQemu targets currently limit the size of the IPA space to
> > 40 bits because that is all what KVM supports. However, this is about
> > to change, and so we need to update the code if we
Hi Liming,
On 11/26/18 04:31, Gao, Liming wrote:
> [...]
Thanks, I think I understand your points better.
That doesn't mean that I agree with all of your points. :)
My suggestion/request is that this series please drop the OvmfPkg &
ArmVirtPkg patches for now, as it's obvious that my
On 11/23/18 13:14, Ard Biesheuvel wrote:
> In preparation of permitting the virt code to define a larger PA space
> size via gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize than what the
> CPU actually supports, take the CPU's capabilities into account when
> setting up the page tables. This is
On 11/23/18 13:14, Ard Biesheuvel wrote:
> In preparation of dropping the hardcoded 40-bit limit on the physical
> address space when running under virtualization, let's refactor the
> code a bit so we read the hardware limit in a single place.
>
> Contributed-under: TianoCore Contribution
(1) s/is/its/ in $SUBJECT, please.
On 11/23/18 13:14, Ard Biesheuvel wrote:
> Drop the PcdPrePiCpuMemorySize definitions that limit it to 40
> bits on AArch64 targets.
Indeed, after this series is applied, we still have
[PcdsFixedAtBuild.ARM]
gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize|40
On Mon, 26 Nov 2018 at 10:42, Laszlo Ersek wrote:
>
> On 11/23/18 13:14, Ard Biesheuvel wrote:
> > In preparation of permitting the virt code to define a larger PA space
> > size via gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize than what the
> > CPU actually supports, take the CPU's capabilities
Reviewed-by: Star Zeng
Hao and Jian, thanks to spend effort as the maintainer.
Star
-Original Message-
From: Wu, Hao A
Sent: Tuesday, November 27, 2018 8:37 AM
To: edk2-devel@lists.01.org
Cc: Wu, Hao A ; Zeng, Star ; Wang,
Jian J
Subject: [PATCH] Maintainers.txt: Update maintainer &
GenVtf C tool is IPF specific. IPF support has been removed
from edk2 trunk. This tool can be removed.
https://bugzilla.tianocore.org/show_bug.cgi?id=1349
v2:Remove GenVtf in Makefile and GNUmakefile.
Cc: Yonghong Zhu
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.1
On Tue, Nov 20, 2018 at 02:37:44PM +, Gao, Liming wrote:
> Reviewed-by: Liming Gao
>
Hi Liming,
Would you mind to help to push to fix into the git repo?
Thanks,
Gary Lin
> > -Original Message-
> > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> > Laszlo
On 11/23/2018 5:19 PM, Laszlo Ersek wrote:
On 11/23/18 07:27, prabin ca wrote:
Hi Team,
I’m new to UEFI and display interface in UEFI. I would like to have deep dive
into how display is working in UEFI (display architecture) and how display is
have been bring up (porting of display panel in
Thanks a lot to Star for his great works as MdeModulePkg maintainer.
Reviewed-by: Jian J Wang
> -Original Message-
> From: Wu, Hao A
> Sent: Tuesday, November 27, 2018 8:37 AM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Zeng, Star ;
> Wang, Jian J
> Subject: [PATCH]
DuetPkg depends on Legacy BIOS to provide a UEFI environment.
It was invented in the era when UEFI environment is hard to find.
Since now UEFI is very popular in PC area, we could stop the
official support of this package and remove it from the master.
And moreover, the tools only used by DuetPkg
Update Star as the reviewer of MdeModulePkg;
Add Hao Wu as the 2nd maintainer of MdeModulePkg.
Cc: Star Zeng
Cc: Jian J Wang
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Hao Wu
---
Maintainers.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
Done at 385c0bf55a3e26dd1d3b5671f4ee69cd3f829baf.
>-Original Message-
>From: Gary Lin [mailto:g...@suse.com]
>Sent: Tuesday, November 27, 2018 9:48 AM
>To: Gao, Liming
>Cc: Laszlo Ersek ; edk2-devel@lists.01.org; Kinney,
>Michael D
>Subject: Re: [edk2] [PATCH 1/1] MdePkg ACPI: fix the
The MM_VERSION SMC call uses SMC32 calling convention. Fix the macro
to reflect the correct value.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Sughosh Ganu
---
ArmPkg/Include/IndustryStandard/ArmMmSvc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
From: Achin Gupta
This patch adds the Management Mode(MM) - Secure Partition
Manager(SPM) SVC return codes.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Sughosh Ganu
---
ArmPkg/Include/IndustryStandard/ArmMmSvc.h | 7 +++
1 file changed, 7 insertions(+)
diff
From: Achin Gupta
The Standalone MM environment runs in S-EL0 in AArch64 on ARM Standard
Platforms. Privileged firmware e.g. ARM Trusted Firmware sets up its
architectural context including the initial translation tables for the
S-EL1/EL0 translation regime. The MM environment will still request
Changes since v1:
A new patch has been added to reflect the library class added for
changing the MMU attributes in StandaloneMM image, based on review
comments from Ard Biesheuvel.
These patches needs to be applied on top of the following patch series
- "ArmPkg related changes for StandaloneMM
Fix all GUID values defined in invalid format.
Cc: Supreeth Venkatesh
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Eric Jin
---
.../IHV/Protocol/SimpleNetwork/BlackBoxTest/Guid.h | 140 ++---
1 file changed, 70 insertions(+), 70 deletions(-)
diff
Changes since v3:
Based on review comments from Ard, moved the MMU attribute changing
functions for StandaloneMM image into a new library class.
Moved the addition of memory space used as a MM_COMMUNICATE buffer to
memory type 'EfiGcdMemoryTypeReserved' and removed the call to
AllocatgePages.
From: Achin Gupta
This patch defines PCDs to describe the base address and size of
communication buffer between normal world (uefi) and standalone MM
environment in the secure world.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Sughosh Ganu
---
ArmPkg/ArmPkg.dec | 3
From: Achin Gupta
PI v1.5 Specification Volume 4 defines Management Mode Core Interface
and defines EFI_MM_COMMUNICATION_PROTOCOL. This protocol provides a
means of communicating between drivers outside of MM and MMI
handlers inside of MM.
This patch implements the EFI_MM_COMMUNICATION_PROTOCOL
The patch is applied to the EFI part.
Fix bug - set EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST bit
in Enable parameter to make sure Multicast is enabled.
Add one checkpoint when MCastFilterCount is zero
Cc: Supreeth Venkatesh
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Eric
The MMU functions needed for StandaloneMM image are now exported
through a separate library class. Make the corresponding change in the
core's entry point inf file so that it references the correct library
class for modifying the MMU attributes.
Contributed-under: TianoCore Contribution Agreement
The patch is applied to the IHV part.
Fix bug - set EFI_SIMPLE_NETWORK_RECEIVE_MULTICAST bit
in Enable parameter to make sure Multicast is enabled.
Add one checkpoint when MCastFilterCount is zero
Cc: Supreeth Venkatesh
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Eric
The StandaloneMM image executes in S-EL0 on reference Arm platforms
and is deployed by the trusted firmware as BL32 image. Memory for the
Standalone MM drivers is marked as RW+XN initially, allowing the
drivers to be loaded into the memory. Once loaded, the memory
attributes need to be changed to
On Fri, 23 Nov 2018 at 13:14, Ard Biesheuvel wrote:
>
> Add a helper function that returns the maximum physical address space
> size as supported by the current CPU.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel
> ---
>
On Mon, 26 Nov 2018 at 11:47, Laszlo Ersek wrote:
>
> On 11/23/18 13:14, Ard Biesheuvel wrote:
> > Move the call to BuildCpuHob () into ArmVirtMemoryInitPeiLib (which is
> > shared between all the ArmVirtPkg targets), and drop the inclusion of
> > CpuPei.inf [which calls it on ArmVirtQemu] and
Thanks Mike for the PACKAGE_PATH comment and the information of non-osi repo.
I am also considering merging below three binary packages into one
Vlv2BinaryPkg, post the ECP free Vlv2BinaryPkg on
https://github.com/tianocore/edk2-non-osi/tree/master/Silicon/Intel/ , and
protect it with the
On Wed, Nov 21, 2018 at 02:26:23AM +0100, Marcin Wojtas wrote:
> Hi Leif,
>
> śr., 14 lis 2018 o 18:33 Leif Lindholm napisał(a):
> >
> > On Wed, Nov 14, 2018 at 07:05:01AM +0100, Marcin Wojtas wrote:
> > > > I think all of my comments on this patch can be summarised as "what is
> > > > an AP in
On Mon, 26 Nov 2018 at 11:00, Laszlo Ersek wrote:
>
> On 11/23/18 13:14, Ard Biesheuvel wrote:
> > In preparation of dropping the hardcoded 40-bit limit on the physical
> > address space when running under virtualization, let's refactor the
> > code a bit so we read the hardware limit in a single
Do you type ". edksetup.bat BaseTools" in Edk2 directory? What output of this
command? This script will add edk2 BaseTools path into system PATH, then build
can be used.
And, you can use GCC5 as the default tool chain. It supports GCC 5.0 or the
above version.
> -Original Message-
>
This covers the bulk of the information originally present in
https://github.com/tianocore/tianocore.github.io/wiki/ArmPlatformPkg-AArch64
regarding building and running the Foundation/Base FVP Platforms.
The sections on fetching source have been delegated to the root Readme.md
Signed-off-by:
Openssl is fetched as a git submodule, so make sure git clone --recursive
is specified in the instructions to fetch a full source tree.
Signed-off-by: Nariman Poushin
---
Readme.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Readme.md b/Readme.md
index 6ad5953..bb53c6f
This covers the bulk of the information originally present in
https://github.com/tianocore/tianocore.github.io/wiki/ArmPlatformPkg-AArch64
regarding building and running the Foundation/Base FVP Platforms.
The sections on fetching source have been delegated to the root Readme.md
Signed-off-by:
On Fri, Nov 23, 2018 at 09:44:03AM +0100, Ard Biesheuvel wrote:
> The deprecated BdsLib library class in ArmPkg is still depended upon, but
> only a single implementation exists, which now resides in edk2-platforms.
>
> This implementation has some issues in how it deals with Device Paths,
> so
On Fri, Nov 23, 2018 at 07:54:59PM +0100, Ard Biesheuvel wrote:
> The last remaining users of the BdsLib.h header reside in the
> edk2-platforms tree, and so it has been copied there. This
> allows us to remove the original from ArmPkg.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
On Fri, Nov 23, 2018 at 07:53:21PM +0100, Ard Biesheuvel wrote:
> RdkBootManagerLib/RdkBootManagerLib.h includes BdsLib.h from ArmPkg
> but does not actually depend on it. So drop the #include.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel
On Wed, Nov 21, 2018 at 01:01:42PM +0100, Ard Biesheuvel wrote:
> Align edk2-platform with upcoming changes to EDK2 to get rid of per-bank
> NOR flash GUIDs.
For the series:
Reviewed-by: Leif Lindholm
> Ard Biesheuvel (3):
> Platform/ARM: replace hardcoded VenHW() device paths referring to
On Mon, 26 Nov 2018 at 16:12, Leif Lindholm wrote:
>
> On Fri, Nov 23, 2018 at 07:54:59PM +0100, Ard Biesheuvel wrote:
> > The last remaining users of the BdsLib.h header reside in the
> > edk2-platforms tree, and so it has been copied there. This
> > allows us to remove the original from ArmPkg.
On Mon, 26 Nov 2018 at 16:06, Leif Lindholm wrote:
>
> On Fri, Nov 23, 2018 at 09:44:03AM +0100, Ard Biesheuvel wrote:
> > The deprecated BdsLib library class in ArmPkg is still depended upon, but
> > only a single implementation exists, which now resides in edk2-platforms.
> >
> > This
On Mon, 26 Nov 2018 at 17:50, Ard Biesheuvel wrote:
>
> On Mon, 26 Nov 2018 at 16:12, Leif Lindholm wrote:
> >
> > On Fri, Nov 23, 2018 at 07:54:59PM +0100, Ard Biesheuvel wrote:
> > > The last remaining users of the BdsLib.h header reside in the
> > > edk2-platforms tree, and so it has been
Reviewed-by: Supreeth Venkatesh
-Original Message-
From: Lokesh B V
Sent: Tuesday, November 20, 2018 12:50 AM
To: edk2-devel@lists.01.org; Supreeth Venkatesh ;
eric@intel.com
Cc: Lokesh Belathur Veerappa
Subject: [edk2-test][PATCH] SctPkg/UEFI: Fix invalid GUID value format error
Reviewed-by: Supreeth Venkatesh
-Original Message-
From: Lokesh B V
Sent: Tuesday, November 20, 2018 12:51 AM
To: edk2-devel@lists.01.org; Supreeth Venkatesh ;
eric@intel.com
Cc: Lokesh Belathur Veerappa
Subject: [edk2-test][PATCH] SctPkg/build: Add support for GenBin tool build
With the latest edk2, I am getting this compilation error (even when this patch
is applied).
I will be looking at it tomorrow. Please check this out if you get a chance.
GenBin
Hi,
Of course, I will resend both patches with the appropriate Contribution
agreement.
Thanks
Nariman
On Mon, 26 Nov 2018 at 14:11, Gao, Liming wrote:
> Could you add [edk2-platform] in the title? So, I know this change is for
> edk2-platform repo.
>
> And, please add Contributed-under:
This covers the bulk of the information originally present in
https://github.com/tianocore/tianocore.github.io/wiki/ArmPlatformPkg-AArch64
regarding building and running the Foundation/Base FVP Platforms.
The sections on fetching source have been delegated to the root Readme.md
On Mon, Nov 26, 2018 at 01:17:48PM +, Nariman Poushin wrote:
> This covers the bulk of the information originally present in
> https://github.com/tianocore/tianocore.github.io/wiki/ArmPlatformPkg-AArch64
> regarding building and running the Foundation/Base FVP Platforms.
>
> The sections on
On Mon, Nov 26, 2018 at 01:10:53PM +, Nariman Poushin wrote:
> Openssl is fetched as a git submodule, so make sure git clone --recursive
> is specified in the instructions to fetch a full source tree.
Can you add a
Contributed-under: TianoCore Contribution Agreement 1.1
tag for all patches
Could you add [edk2-platform] in the title? So, I know this change is for
edk2-platform repo.
And, please add Contributed-under: TianoCore Contribution Agreement 1.1 before
Signed-off-by.
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
>
Openssl is fetched as a git submodule, so make sure git clone --recursive
is specified in the instructions to fetch a full source tree.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Nariman Poushin
---
Changes since v1:
- Added Contribution Agreement tag
On Wed, 21 Nov 2018 at 12:58, Ard Biesheuvel wrote:
>
> This series fixes an issue reported by Hongbo and Philippe, where
> ArmVirtQemuKernel will crash on an attempt to access flash bank #0,
> which is secure-only when running QEMU with support for EL3.
>
> So let's switch to discovering the NOR
On Sun, 25 Nov 2018 at 11:02, Laszlo Ersek wrote:
>
> Repo: https://github.com/lersek/edk2.git
> Branch: acpi_test_support
>
> The feature is described in the first patch. Build OvmfPkg and
> ArmVirtPkg platforms with "--pcd PcdAcpiTestSupport=TRUE" to enable it.
>
> Cc: Anthony Perard
> Cc:
On Fri, Nov 23, 2018 at 01:14:28PM +0100, Ard Biesheuvel wrote:
> In preparation of permitting the virt code to define a larger PA space
> size via gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize than what the
> CPU actually supports, take the CPU's capabilities into account when
> setting up the
On Mon, 26 Nov 2018 at 18:00, Ard Biesheuvel wrote:
>
> On Wed, 21 Nov 2018 at 12:58, Ard Biesheuvel
> wrote:
> >
> > This series fixes an issue reported by Hongbo and Philippe, where
> > ArmVirtQemuKernel will crash on an attempt to access flash bank #0,
> > which is secure-only when running
On Mon, Nov 26, 2018 at 06:50:09PM +0100, Ard Biesheuvel wrote:
> On Mon, 26 Nov 2018 at 18:46, Leif Lindholm wrote:
> >
> > On Fri, Nov 23, 2018 at 01:14:28PM +0100, Ard Biesheuvel wrote:
> > > In preparation of permitting the virt code to define a larger PA space
> > > size via
On Mon, 26 Nov 2018 at 16:15, Leif Lindholm wrote:
>
> On Wed, Nov 21, 2018 at 01:01:42PM +0100, Ard Biesheuvel wrote:
> > Align edk2-platform with upcoming changes to EDK2 to get rid of per-bank
> > NOR flash GUIDs.
>
> For the series:
> Reviewed-by: Leif Lindholm
>
Thanks
Pushed as
On Mon, 26 Nov 2018 at 18:46, Leif Lindholm wrote:
>
> On Fri, Nov 23, 2018 at 01:14:28PM +0100, Ard Biesheuvel wrote:
> > In preparation of permitting the virt code to define a larger PA space
> > size via gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize than what the
> > CPU actually supports,
On 26/11/18 12:46, Ard Biesheuvel wrote:
> On Fri, 23 Nov 2018 at 13:14, Ard Biesheuvel
> wrote:
>>
>> Add a helper function that returns the maximum physical address space
>> size as supported by the current CPU.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by:
Excellent. Thank you for all your work here. I'll be compiling
information regarding all our options on the wiki and we can further
discuss them at our Community Meeting in December.
I'll share the link to the wiki post once it is live on this mailing list.
Cheers,
Stephano
On 11/26/2018
Derive the size of the GCD memory space map directly from the CPU's
information registers rather than from the PcdPrePiCpuMemorySize PCD,
which will be removed.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
BeagleBoardPkg/PrePi/PeiUniCore.inf | 1 -
In preparation of dropping PcdPrePiCpuMemorySize entirely, base the
maximum size of the identity map on the capabilities of the CPU.
Since that may exceed what is architecturally permitted when using
4 KB pages, take ARM_MMU_IDMAP_RANGE into account as well.
Contributed-under: TianoCore
Derive the size of the GCD memory space map directly from the CPU's
information registers rather than from the PcdPrePiCpuMemorySize PCD,
which will be removed.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
In preparation of dropping the hardcoded 40-bit limit on the physical
address space when running under virtualization, let's refactor the
code a bit so we read the hardware limit in a single place.
Note that the hardware capabilities may exceed what the architecture
permits when using 4 KB pages,
Drop some PCD references that are not actually referenced from the
PlatformPei code.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPlatformPkg/PlatformPei/PlatformPeiLib.inf | 3 ---
ArmPlatformPkg/PlatformPei/PlatformPeim.inf | 3 ---
2 files
Derive the size of the GCD memory space map directly from the CPU's
information registers rather than from the PcdPrePiCpuMemorySize PCD,
which will be removed.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPkg/Drivers/CpuPei/CpuPei.inf | 1 -
Derive the size of the GCD memory space map directly from the CPU's
information registers rather than from the PcdPrePiCpuMemorySize PCD,
which will be removed.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
Add a helper function that returns the maximum physical address space
size as supported by the current CPU.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPkg/Include/Library/ArmLib.h | 17 +
The ArmVirtQemu targets currently limit the size of the IPA space to
40 bits because that is all what KVM supports. However, this is about
to change, and so we need to update the code if we want to ensure that
our UEFI firmware builds can keep running on systems that set values
other than 40
Derive the size of the GCD memory space map directly from the CPU's
information registers rather than from the PcdPrePiCpuMemorySize PCD,
which will be removed.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmPlatformPkg/PrePi/PeiMPCore.inf | 1 -
PcdPrePiCpuMemorySize is no longer used so drop the declarations from
the package DEC file.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
EmbeddedPkg/EmbeddedPkg.dec | 4
1 file changed, 4 deletions(-)
diff --git a/EmbeddedPkg/EmbeddedPkg.dec
Now that the size of the GCD memory map is based directly on the
CPU's capabilities, we no longer have a need for PcdPrePiCpuMemorySize
and so it will be removed. So drop it from QemuVirtMemInfoLib as well.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
PcdPrePiCpuMemorySize is no longer used so drop the PCD overrides
from all platform descriptions in ArmVirtPkg.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
ArmVirtPkg/ArmVirt.dsc.inc | 3 ---
ArmVirtPkg/ArmVirtQemu.dsc | 4
Drop the reference to gEmbeddedTokenSpaceGuid.PcdPrePiCpuMemorySize
which is never used.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
EmbeddedPkg/Library/PrePiLib/PrePiLib.inf | 1 -
1 file changed, 1 deletion(-)
diff --git
On 26/11/18 23:37, Ard Biesheuvel wrote:
> Add a helper function that returns the maximum physical address space
> size as supported by the current CPU.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel
Reviewed-by: Philippe Mathieu-Daudé
> ---
>
On Mon, Nov 26, 2018 at 11:43 PM Philippe Mathieu-Daudé
wrote:
>
> On 26/11/18 23:37, Ard Biesheuvel wrote:
> > Add a helper function that returns the maximum physical address space
> > size as supported by the current CPU.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.1
> >
On Tue, 27 Nov 2018 at 00:02, Philippe Mathieu-Daudé wrote:
>
> On Mon, Nov 26, 2018 at 11:43 PM Philippe Mathieu-Daudé
> wrote:
> >
> > On 26/11/18 23:37, Ard Biesheuvel wrote:
> > > Add a helper function that returns the maximum physical address space
> > > size as supported by the current
Feedback on GitHub as follows…
> 1. No Lock-In - What automated data export is available?
> We want to be able to leave and take all our data with us. "Data" here
> includes: review comments, pull requests / patches (including metadata),
> old (rejected) pull requests and metadata, issue
Hi Laszlo,
On 25/11/18 11:01, Laszlo Ersek wrote:
> QEMU's test suite includes a set of cases called "BIOS tables test". Among
> other things, it locates the RSD PTR ACPI table in guest RAM, and then
> (chasing pointers to other ACPI tables) performs various sanity checks on
> the QEMU-generated
84 matches
Mail list logo