Re: [edk2] Can I do this in an INF file?

2017-05-16 Thread Andrew Fish
Liming, Why does INF syntax support [LibraryClasses.common.DXE_RUNTIME_DRIVER] if it does nothing? https://edk2-docs.gitbooks.io/edk-ii-inf-specification/content/3_edk_ii_inf_file_format/36_[libraryclasses]_sections.html Thanks, Andrew Fish > On May 16, 2017, at 6:46 PM, Andrew Fish wrote:

Re: [edk2] Can I do this in an INF file?

2017-05-16 Thread Andrew Fish
> On May 16, 2017, at 6:41 PM, Gao, Liming wrote: > > Andrew: > There is no such usage. INF can specify source files for the different > ARCHs, but not specify source files for the different module type. In fact, > INF module type is fixed. It can't be changed to other type in build time. If

Re: [edk2] Can I do this in an INF file?

2017-05-16 Thread Gao, Liming
Andrew: There is no such usage. INF can specify source files for the different ARCHs, but not specify source files for the different module type. In fact, INF module type is fixed. It can't be changed to other type in build time. If you expect the library to be linked to the different type dri

Re: [edk2] [RFC] PCD: Database size optimization for multi-SKU

2017-05-16 Thread Zeng, Star
Tim, Yes, right. The PCD_DATABASE_INIT.BuildVersion will be increased for the PCD database format change to support multi-SKU. That is the rule we maintain the PCD database. :) Thanks, Star From: Tim Lewis [mailto:tim.le...@insyde.com] Sent: Wednesday, May 17, 2017 2:04 AM To: Zeng, Star ; edk

[edk2] Can I do this in an INF file?

2017-05-16 Thread Andrew Fish
I'm trying to cross compile a runtime library to work in an Application (for testing). I can't seem to restrict files and libs to specific module types? [Sources.common.UEFI_APPLICATION] FakeRuntime.c [LibraryClasses.common.DXE_RUNTIME_DRIVER] UefiRuntimeLib Am I using the wrong syntax?

Re: [edk2] [RFC v4 06/13] OvmfPkg:AmdSevDxe: add AmdSevDxe driver

2017-05-16 Thread Brijesh Singh
On 05/16/2017 12:56 PM, Jordan Justen wrote: On 2017-05-16 05:04:58, Brijesh Singh wrote: Hi Jordan, On 5/15/17 12:47 PM, Jordan Justen wrote: On 2017-05-11 11:01:57, Brijesh Singh wrote: We basically need some kind of guarantee that this driver is run before any other drivers or libs acc

Re: [edk2] [RFC] PCD: Database size optimization for multi-SKU

2017-05-16 Thread Tim Lewis
Star- One more requirement that I propose is the the format update be detectable easily by inspection of the binary image. We have tools that dump and manipulate the PCD database and the revision or version number allows us to support multiple core versions. I think this is already your inten

Re: [edk2] [RFC v4 06/13] OvmfPkg:AmdSevDxe: add AmdSevDxe driver

2017-05-16 Thread Jordan Justen
On 2017-05-16 05:04:58, Brijesh Singh wrote: > Hi Jordan, > > On 5/15/17 12:47 PM, Jordan Justen wrote: > > On 2017-05-11 11:01:57, Brijesh Singh wrote: > >> > >> We basically need some kind of guarantee that this driver is run before > >> any other > >> drivers or libs access MMIO register/buffe

Re: [edk2] [PATCH] MdePkg/BaseLib: Fix PathRemoveLastItem to ignore consecutive '\'

2017-05-16 Thread Jeff Westfahl
Hi Ray, The problem with 'ls ..\..\' that I'm trying to solve here is the call to PathRemoveLastItem from within PathCleanUpDirectories. This intermediate path is not clean. Here is a log of the operations performed on a path as it goes through PathCleanUpDirectories: FS2:\ABC\DEF\> ls .

[edk2] [RFC] PCD: Database size optimization for multi-SKU

2017-05-16 Thread Star Zeng
- Requirement Reduce PCD database size for multi-SKU case to save image size at build time and memory resource at boot time. - Current limitation When multiple SKUs are enabled, the full set of PCD values for all SKUs will be generated into PCD database, and HOB will be created for the PCD data

[edk2] How to look up ACPI device node from DXE driver

2017-05-16 Thread Evgeny Yakovlev
I am writing a DXE driver for a paravirtualized HyperV storage device for OvmfPkg. Host hypervisor exposes the presence of this device through ACPI device node in DSDT. Specific AML path itself may be different from host to host but device UID is always a string: "VMBus". I was hoping to be able t

Re: [edk2] [PATCH] MdePkg: Fix undefined behavior on variadic parameters

2017-05-16 Thread Sergei Temerkhanov
On Tue, May 16, 2017 at 8:10 AM, Gao, Liming wrote: > Sergey: > Could you give more detail on the undefined behavior on variadic parameters? > > I see https://bugzilla.tianocore.org/show_bug.cgi?id=410 describe this > issues found in the latest CLANG tool chain. Do you find other tool chain

Re: [edk2] [RFC v4 06/13] OvmfPkg:AmdSevDxe: add AmdSevDxe driver

2017-05-16 Thread Brijesh Singh
Hi Jordan, On 5/15/17 12:47 PM, Jordan Justen wrote: > On 2017-05-11 11:01:57, Brijesh Singh wrote: >> >> We basically need some kind of guarantee that this driver is run before any >> other >> drivers or libs access MMIO register/buffers. In additional to clearing >> encryption >> bit from MMIO

[edk2] [PATCH 2/2] ArmPlatformPkg: Timer access for non-secure EL1/0

2017-05-16 Thread evan . lloyd
From: Sami Mujawar According to Section 2.3.6 of the "UEFI Specification 2.6 Errata A"; the primary CPU must be configured such that 'Timer access must be provided to non-secure EL1 and EL0 by setting bits EL1PCTEN and EL1PCEN in register CNTHCTL_EL2.' This commit adds this missing set-up to the

[edk2] [PATCH 0/2] Initialize CNTHCTL

2017-05-16 Thread evan . lloyd
From: Evan A pair of minor patches to correct an omission. This patch set enables timer access as required by the UEFI specification. The first patch adds accessors for the register to ArmPkg, the second fixes the error in ArmPlatofrmPkg. They are in different packages, but dependent. As the resp

[edk2] [PATCH 1/2] ArmPkg: Add CNTHCTL_EL2 support functions

2017-05-16 Thread evan . lloyd
From: Sami Mujawar Added helper functions for reading and writing the CNTHCTL_EL2 register. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Sami Mujawar Signed-off-by: Evan Lloyd --- ArmPkg/Include/Chipset/AArch64.h | 12 +++- ArmPkg/Library/ArmLib

Re: [edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Update GenBiosId

2017-05-16 Thread Lu, ShifeiX A
Reviewed-by: lushifex Shifei -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of zwei4 Sent: Tuesday, May 16, 2017 4:41 PM To: edk2-devel@lists.01.org Subject: [edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Update GenBiosId Contributed-

[edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Update GenBiosId

2017-05-16 Thread zwei4
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: zwei4 --- Platform/BroxtonPlatformPkg/BuildBios.sh| 12 .../Common/Tools/GenBiosId/GenBiosId| Bin 12236 -> 36128 bytes 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/P

Re: [edk2] [PATCH] MdePkg/BaseLib: Fix PathRemoveLastItem to ignore consecutive '\'

2017-05-16 Thread Ni, Ruiyu
Jeff, PathRemoveLastItem() is expected to be called after PathCleanUpDirectories(). E.g.: what should we expect PathRemoveLastItem() do for "fs0:\a\b\..\"? So PathRemoveLastItem() expects the incoming path is cleaned. Thanks/Ray > -Original Message- > From: edk2-devel [mailto:edk2-devel-b