[edk2] [PATCH] MdeModulePkg: Delete useless case code

- Delete useless case code to fix /WHOLEARCHIVE build error with VS2015 tool chain Cc: Star Zeng Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bell Song ---

[edk2] [PATCH] MdeModulePkg: Delete never touched code

- Delete never touched code Cc: Star Zeng Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bell Song --- MdeModulePkg/Universal/RegularExpressionDxe/Oniguruma/reggnu.c | 3 +--

[edk2] [PATCH] IntelFsp2Pkg: Delete useless external TempRamInitApi API

- Delete useless external TempRamInitApi API to fix /WHOLEARCHIVE build error with VS2015 tool chain Cc: Jiewen Yao Cc: Giri P Mudusuru Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bell Song ---

Re: [edk2] [PATCH v2][Patch 2/4] MdePkg/UefiDevicePathLib: Add DevPathFromTextDns and DevPathToTextDns libraries

You are right. Thanks Eric, will send out V3 later. > -Original Message- > From: Jin, Eric > Sent: Thursday, August 3, 2017 1:10 PM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Wu, > Jiaxin

Re: [edk2] [PATCH v2][Patch 2/4] MdePkg/UefiDevicePathLib: Add DevPathFromTextDns and DevPathToTextDns libraries

Jiaxin, In V2, the input parameter, TextDeviceNode, in the DevPathFromTextDns() is parsed by GetNextParamStr() twice. After the first parse, the content of TextDeviceNode is modified and can convert to correct device path node. Suggest to duplicate the TextDeviceNode or restore the return

Re: [edk2] [PATCH] ShellPkg: Update header file including style

Reviewed-by: Ruiyu Ni Thanks/Ray > -Original Message- > From: Song, BinX > Sent: Thursday, August 3, 2017 10:45 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu > Subject: [PATCH] ShellPkg: Update header

Re: [edk2] [Patch] NetworkPkg: Display HTTP redirection info to the screen if need.

Ok, that's make sense. Reviewed-by: Wu Jiaxin > -Original Message- > From: Fu, Siyuan > Sent: Thursday, August 3, 2017 11:28 AM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Ye, Ting > Subject: RE: [Patch] NetworkPkg:

Re: [edk2] [Patch] NetworkPkg: Display HTTP redirection info to the screen if need.

Hi, Jiaxin We actually don't support HTTP redirect here, which means we won't auto redirect and download the boot image. The HTTP boot process will be aborted, so I think using ERROR makes more sense. Thanks, Siyuan -Original Message- From: Wu, Jiaxin Sent: Thursday, August 3, 2017

Re: [edk2] [patch] BaseTools/VfrCompile: Remove the MAX_PATH limitation

Reviewed-by: Eric Dong -Original Message- From: Bi, Dandan Sent: Tuesday, August 1, 2017 3:51 PM To: edk2-devel@lists.01.org Cc: Dong, Eric ; Gao, Liming ; Daniel D�az Subject: [patch]

Re: [edk2] [patch] BaseTools/VfrCompile: Fix segmentation fault issues

Reviewed-by: Eric Dong -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Dandan Bi Sent: Thursday, July 27, 2017 10:53 AM To: edk2-devel@lists.01.org Cc: Bo Chen ; Gao, Liming Subject:

[edk2] [PATCH] ShellPkg: Update header file including style

- Update header file including style to avoid MSDN C4464 warning Cc: Jaben Carsey Cc: Ruiyu Ni Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bell Song ---

Re: [edk2] [PATCH v1 1/1] ShellPkg/Ifconfig6: Update error message and add a new line

Reviewed-by: Wu Jiaxin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > hegdenag > Sent: Tuesday, August 1, 2017 1:48 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu

Re: [edk2] [Patch] NetworkPkg: Display HTTP redirection info to the screen if need.

Hi siyuan, > + if (HttpHeader != NULL) { > +Print (L"\n HTTP ERROR: Resource Redirected.\n New > Location: %a\n", HttpHeader->FieldValue); > + } How about we use HTTP WARNING instead of HTTP ERROR? Thanks, Jiaxin > -Original Message- > From: Fu, Siyuan >

Re: [edk2] [Patch 2/4] DuetPkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

I am surprised DuetPkg is still being used. Reviewed-by: Ruiyu Ni Regards, Ray >-Original Message- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >Yonghong Zhu >Sent: Wednesday, August 2, 2017 5:28 PM >To: edk2-devel@lists.01.org >Cc: Ni,

Re: [edk2] [PATCH 10/12] OvmfPkg/IoMmuDxe: implement in-place decryption/encryption for Map/Unmap

On 8/2/17 7:13 PM, Laszlo Ersek wrote: > (CC Andrew) > > On 08/03/17 01:01, Brijesh Singh wrote: >> >> On 8/2/17 4:24 PM, Laszlo Ersek wrote: >> >> [Snip] >>> At the moment, we have the foll+// The buffer at >>> MapInfo->CryptedAddress comes from AllocateBuffer(). >>> // >>>

Re: [edk2] issue about booting centos fail with edk2

On 2017/8/2 16:29, Laszlo Ersek wrote: > On 08/02/17 00:57, Ard Biesheuvel wrote: >> On 1 August 2017 at 23:29, Laszlo Ersek wrote: >>> On 08/01/17 19:23, Ard Biesheuvel wrote: On 1 August 2017 at 16:42, Laszlo Ersek wrote: > On 08/01/17 10:34, Zhu

Re: [edk2] [PATCH 10/12] OvmfPkg/IoMmuDxe: implement in-place decryption/encryption for Map/Unmap

(CC Andrew) On 08/03/17 01:01, Brijesh Singh wrote: > > > On 8/2/17 4:24 PM, Laszlo Ersek wrote: > > [Snip] >> At the moment, we have the foll+// The buffer at MapInfo->CryptedAddress >> comes from AllocateBuffer(). >> // >> MapInfo->PlainTextAddress = MapInfo->CryptedAddress; >> -

[edk2] [PATCH 12/12] OvmfPkg/IoMmuDxe: Unmap(): recycle MAP_INFO after BusMasterCommonBuffer[64]

In order for Unmap() to be callable from ExitBootServices() event handler context (for cleaning up a BusMasterCommonBuffer[64] operation), we have to completely liberate the affected path in Unmap() from dynamic memory management. The last remaining piece is the release of the MAP_INFO structure.

[edk2] [PATCH 10/12] OvmfPkg/IoMmuDxe: implement in-place decryption/encryption for Map/Unmap

At the moment, we have the following distribution of actions between the IOMMU protocol member functions: - AllocateBuffer() allocates pages and clears the memory encryption mask. - FreeBuffer() re-sets the memory encryption mask, and deallocates pages. - Map() does nothing at all when

[edk2] [PATCH 09/12] OvmfPkg/IoMmuDxe: rework setup of "MapInfo->PlainTextAddress" in Map()

There are three issues with the current calculations: - The initial logic that sets up "DmaMemoryTop" and "AllocateType" checks for the BusMasterCommonBuffer64 operation in two places. The inner check for BusMasterCommonBuffer64 will never evaluate to TRUE however, because the outer check

[edk2] [PATCH 08/12] OvmfPkg/IoMmuDxe: zero out pages before releasing them

Whenever we release the plaintext bounce buffer pages that were allocated implicitly in Map() for BusMasterRead[64] and BusMasterWrite[64], we restore the encryption mask on them. However, we should also rewrite the area (fill it with zeros) so that the hypervisor is not left with a plaintext view

[edk2] [PATCH 06/12] OvmfPkg/IoMmuDxe: propagate errors from AmdSevInstallIoMmuProtocol()

If we cannot install the IOMMU protocol for whatever reason, exit the driver with an error. The same is already done for the IOMMU Absent protocol. Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Jordan Justen Cc: Tom

[edk2] [PATCH 01/12] OvmfPkg/IoMmuDxe: rewrap source code to 79 characters

No functional changes. Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Jordan Justen Cc: Tom Lendacky Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek

[edk2] [PATCH 05/12] OvmfPkg/IoMmuDxe: don't initialize local variables

The edk2 coding style requires separate assignments. Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Jordan Justen Cc: Tom Lendacky Contributed-under: TianoCore Contribution Agreement 1.0

[edk2] [PATCH 07/12] OvmfPkg/IoMmuDxe: clean up used library classes

The following library classes are not used by this module, so remove them from the INF file's [LibraryClasses] section: - DxeServicesTableLib - UefiLib The following library classes are used by this module, so add them to the INF file's [LibraryClasses] section: - BaseMemoryLib (e.g. via

[edk2] [PATCH 04/12] OvmfPkg/IoMmuDxe: convert UINTN arguments to UINT64 for the %Lx fmt spec

The portable way to print UINTN values is to use the %Lx format specifier, and to convert the values to UINT64. The second step is currently missing, add it. Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Jordan Justen Cc:

[edk2] [PATCH 02/12] OvmfPkg/IoMmuDxe: rename DeviceAddress to PlainTextAddress in MAP_INFO

In this particular IOMMU driver, "DeviceAddress" is just as accessible to the CPU as "HostAddress", the difference is that the area pointed-to by the former is plain-text and accessible to the hypervisor. Rename "DeviceAddress" to "PlainTextAddress" in MAP_INFO. Cc: Ard Biesheuvel

[edk2] [PATCH 03/12] OvmfPkg/IoMmuDxe: rename HostAddress to CryptedAddress in MAP_INFO

As a continuation of the last patch, clarify that the area pointed-to by "HostAddress" is encrypted and hidden from the hypervisor. Cc: Ard Biesheuvel Cc: Brijesh Singh Cc: Jordan Justen Cc: Tom Lendacky

[edk2] [PATCH 00/12] OvmfPkg/IoMmuDxe: cleanups and fixes

This series is proposed as a replacement (or a replacement "basis") for patches #1 through #3 of Brijesh's series [PATCH v1 0/4] OvmfPkg : IoMmuDxe: BusMasterCommonBuffer support when SEV is active

[edk2] [PATCH] BaseTools/Conf: apply nasmb, asm16 build rule order

Prioritize nasmb rule over asm16 where both source types are specified. Change-Id: I33ec348dab66b313ddb05cb15f2d8407a648c320 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chris Ruffin --- BaseTools/Conf/tools_def.template | 2 +- 1 file changed,

Re: [edk2] [PATCH] staging/edk2-test: TestProfile bug fix

1. Build the project in debug mode so that asserts are visible. 2. Run "sct -u" command and use SCT GUI to create a sequence file. 3. Run "sct -s " command, where is a name of the sequence file created in step 2. The assert is happening during SCT startup when sequence file is parsed.

Re: [edk2] [PATCH] staging/edk2-test: TestProfile bug fix

Hi Felix, Thanks for the contribution. Do you have a test case that reproduces the issue you found? Thanks, Mike > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On > Behalf Of Felix Poludov > Sent: Wednesday, August 2, 2017 6:54 AM > To:

Re: [edk2] [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token

I find that many platform packages have some kind of PCD token to represent size of the PCI express region. That is the reason I think it should be standardized. As far as PCD token name and type (UINT32 vs. UINT64), I'm seeking community feedback. Both PcdPciExpressSize and

Re: [edk2] [Patch 4/4] MdePkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

Hello Yonghong and Andrew, This patch may cause issues when compiling MSVC x IA32 as the optimizer might replace the shift with an intrinsic. I suggest you to use LShiftU64() rather than the << operator. Regards, Marvin. > -Original Message- > From: edk2-devel

[edk2] [PATCH v5 1/3] EmbeddedPkg/AndroidFastboot: split android boot header

Split android boot header definition to share code among different applications and libraries. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jun Nie --- .../Application/AndroidFastboot/AndroidBootImg.c | 35 ++---

[edk2] [PATCH v5 3/3] EmbeddedPkg: add Android boot build entry

The device path specifies where to load android boot image. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jun Nie --- EmbeddedPkg/EmbeddedPkg.dec | 2 ++ EmbeddedPkg/EmbeddedPkg.dsc | 2 ++ 2 files changed, 4 insertions(+) diff --git

[edk2] [PATCH v5 2/3] EmbeddedPkg/AndroidBoot: boot android kernel from storage

Add an android kernel loader that could load kernel from storage device. This android boot image BDS add addtitional cmdline/dtb/ramfs support besides kernel that is introduced by Android boot header. This patch is derived from Haojian's code as below link. https://patches.linaro.org/patch/94683/

Re: [edk2] [PATCH v4 3/4] EmbeddedPkg/AndroidBoot: boot android kernel from storage

2017-08-01 23:50 GMT+08:00 Leif Lindholm : > On Tue, Aug 01, 2017 at 05:29:00PM +0800, Jun Nie wrote: >> Add an android kernel loader that could load kernel from storage >> device. This patch is derived from Haojian's code as below link. >>

Re: [edk2] [PATCH v4 4/4] EmbeddedPkg: add Android boot device path and guid

2017-08-02 0:19 GMT+08:00 Leif Lindholm : > Right, so this one confused me a bit, since it looked like it was > breaking bisect again. It does, which caused me to take a closer > look. > > First of all, you need to add >

Re: [edk2] [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token

A bunch of platforms, such as QuarkSocPkg, already declare such a PCD named PcdPciExpressSize of type UINT64. Maybe PcdPciExpressSize|UINT64 should be 'promoted' to a MdePkg PCD and the platform-defined PCDs replaced? Regards, Marvin. > -Original Message- > From: edk2-devel

[edk2] [PATCH] staging/edk2-test: TestProfile bug fix

The patch is a bug fix for TestFrameworkPkg/Support/TestProfile driver. AsciiStrCpy function was used to copy overlapping strings, which triggered an ASSERT. The function was used on overlapping strings in a couple of places: 1. _alltrim: AsciiStrCpy is replaced with CopyMem, which can handle

[edk2] [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token

Add PcdPciExpressRegionLength PCD Token to MdePkg. The new token can be used in conjunction with PcdPciExpressBaseAddress PCD token to describe PCI Express MMIO region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Felix Polyudov --- Resending with

[edk2] [PATCH v2 2/2] MdePkg/BaseLib: Update internal LinkedList verifications.

1) Replace InternalBaseLibIsNodeInList() with InternalBaseLibIsListValid(). - The verification whether Node is within the doubly-linked List is now done by IsNodeInList(). - Whether the list is valid is returned. 2) The comments within InsertHeadList() and InsertTailList() stated

[edk2] [PATCH v2 1/2] MdePkg/BaseLib: Add IsNodeInList() function.

This patch adds IsNodeInList() to BaseLib, which verifies the given Node is part of the doubly-linked List provided. V2: - Rename "List" to "FirstEntry" and "Node" to "SecondEntry" to clarify that "FirstEntry" does not need to be the doubly-linked list's head node. Contributed-under:

[edk2] Terminal Binding to Serial IO Protocol

Hi edk2 community, it's been a while. We have a situation where we need to use instances of the SERIAL_IO_PROTOCOL to talk to hardware devices that are not part of a console. As written today TerminalDxe's driver binding will consume any handle that implements the SERIAL_IO_PROTOCOL (like in

Re: [edk2] [PATCH v1 3/4] OvmfPkg: IommuDxe: Zero the shared page(s) on Unmap()

On 08/02/17 13:22, Brijesh Singh wrote: > > > On 8/2/17 2:37 AM, Laszlo Ersek wrote: >> // >>> + // Zero the shared memory so that hypervisor no longer able to get >>> intelligentable >>> + // data. >>> + // >>> + SetMem ((VOID *) (UINTN)MapInfo->DeviceAddress, MapInfo->NumberOfBytes, >>>

Re: [edk2] [PATCH 1/2] MdePkg/BaseLib: Add IsNodeInList() function.

Thanks for your correctness. I agree MACRO is better. >-Original Message- >From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] >Sent: Wednesday, August 02, 2017 8:24 PM >To: edk2-devel@lists.01.org >Cc: Gao, Liming ; Kinney, Michael D

Re: [edk2] [PATCH 1/2] MdePkg/BaseLib: Add IsNodeInList() function.

Hey Liming, Thanks for your reply! Sure, that works as well, though usually I prefer to write a static function or a macro instead of repeating. If you prefer this, I'll get a V2 ready that way. Though shouldn't it be ASSERT ((!FeaturePcdGet (PcdVerifyNodeInList) || IsNodeInList (FirstEntry,

Re: [edk2] [PATCH 1/2] MdePkg/BaseLib: Add IsNodeInList() function.

Marvin: How about directly use PCD checker with IsNodeInList()? ASSERT (IsNodeInList (FirstEntry, SecondEntry)); ==> ASSERT (FeaturePcdGet (PcdVerifyNodeInList) && IsNodeInList (FirstEntry, SecondEntry))|| InternalBaseLibIsListValid (FirstEntry)); Thanks Liming >-Original Message-

Re: [edk2] [edk2-UDK2017] issue about AsciiPrint

You can see the usage in MdeModulePkg\Universal\Network\UefiPxeBcDxe\PxeBcDhcp.c line 1618. The detail print format can be found in MdePkg\Include\Library\PrintLib.h >-Original Message- >From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >krishnaLee >Sent: Wednesday,

Re: [edk2] [PATCH v1 3/4] OvmfPkg: IommuDxe: Zero the shared page(s) on Unmap()

On 8/2/17 2:37 AM, Laszlo Ersek wrote: > // >> + // Zero the shared memory so that hypervisor no longer able to get >> intelligentable >> + // data. >> + // >> + SetMem ((VOID *) (UINTN)MapInfo->DeviceAddress, MapInfo->NumberOfBytes, >> 0); > Please use ZeroMem(). > > Furthermore,

[edk2] [PATCH 0/2] Add SecHobData PPI support

This PPI is introduced in PI Version 1.5. Cc: Liming Gao Star Zeng (2): MdePkg: Add definition for SecHobData PPI MdeModulePkg PeiCore: Install SEC HOB data MdeModulePkg/Core/Pei/Hob/Hob.c | 74 ++-

[edk2] [PATCH 2/2] MdeModulePkg PeiCore: Install SEC HOB data

If the EFI_SEC_HOB_DATA_PPI is in the list of PPIs passed to the PEI entry point, the PEI Foundation will call the GetHobs() member function and install all HOBs returned into the HOB list. It does this after installing all PPIs passed from SEC into the PPI database and before dispatching any

[edk2] [PATCH 1/2] MdePkg: Add definition for SecHobData PPI

This PPI is introduced in PI Version 1.5. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- MdePkg/Include/Ppi/SecHobData.h | 65 + MdePkg/MdePkg.dec

[edk2] [Patch 4/4] MdePkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

Bug: https://bugzilla.tianocore.org/show_bug.cgi?id=635 Cc: Liming Gao Cc: Michael D Kinney Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Andrew Fish ---

[edk2] [Patch 2/4] DuetPkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

Bug: https://bugzilla.tianocore.org/show_bug.cgi?id=635 Cc: Ruiyu Ni Cc: Hao Wu Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Andrew Fish --- DuetPkg/EfiLdr/TianoDecompress.c | 4 ++-- 1 file changed, 2

[edk2] [Patch 3/4] IntelFrameworkModulePkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined

Bug: https://bugzilla.tianocore.org/show_bug.cgi?id=635 Cc: Liming Gao Cc: Michael D Kinney Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Andrew Fish ---

[edk2] [Patch 0/4] Fix Xcode 9 Beta treating 32-bit left shift as undefined behavior

Bug: https://bugzilla.tianocore.org/show_bug.cgi?id=635 Yonghong Zhu (4): BaseTools: Fix Xcode 9 Beta treating 32-bit left shift as undefined DuetPkg: Fix Xcode 9 Beta treating 32-bit left shift as undefined IntelFrameworkModulePkg: Fix Xcode 9 Beta treating 32-bit left shift as

[edk2] [Patch 1/4] BaseTools: Fix Xcode 9 Beta treating 32-bit left shift as undefined

Bug: https://bugzilla.tianocore.org/show_bug.cgi?id=635 Cc: Liming Gao Cc: Michael D Kinney Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Andrew Fish --- BaseTools/Source/C/Common/Decompress.c

[edk2] [edk2-UDK2017] issue about AsciiPrint

Hi All, This function will output nothing or strange-output in my real Lenovo_Q270 machine, follow is the key point: //for(UINTN j=0;jSignature);j++) //{ //output byte by byte,or custom range...,or just some specified bytes in the string...

Re: [edk2] [PATCH v1 2/4] OvmfPkg: IommuDxe: Provide support for mapping BusMasterCommonBuffer operation

On 08/02/17 01:51, Brijesh Singh wrote: > On 8/1/17 4:59 PM, Laszlo Ersek wrote: >> Please implement the free-list idea that I outlined earlier. Unmap() >> must not call FreePool() on the MAP_INFO structures. > > Sorry it was my bad. I missed implementing that feedback. I will fix it > in next

[edk2] [PATCH] UefiCpuPkg/Msr: Add a missing IvyBridge processor signature

This commit modifies the CPUID signature check MACRO for IvyBridge processor by adding a missing DisplayModel 0x3E. The missing one appears at Section 35.10.1 to Section 35.10.3 of the Intel(R) 64 and IA-32 Architectures Software Developer's Manual, Volume 3, September 2016. Cc: Eric Dong

Re: [edk2] issue about booting centos fail with edk2

On 08/02/17 00:57, Ard Biesheuvel wrote: > On 1 August 2017 at 23:29, Laszlo Ersek wrote: >> On 08/01/17 19:23, Ard Biesheuvel wrote: >>> On 1 August 2017 at 16:42, Laszlo Ersek wrote: On 08/01/17 10:34, Zhu Yijun wrote: > Thanks for your reply!

Re: [edk2] [PATCH] MdePkg/Ftp4: Fix wrong function pointer declaration

Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Alcantara, Paulo Sent: Tuesday, June 6, 2017 8:34 AM To: edk2-devel@lists.01.org Cc: Kinney, Michael D ; Gao, Liming

Re: [edk2] [PATCH] UefiCpuPkg SecCore: Fix operands of different size in bitwise operation

Reviewed-by: Liming Gao >-Original Message- >From: Zeng, Star >Sent: Wednesday, August 02, 2017 10:10 AM >To: edk2-devel@lists.01.org >Cc: Zeng, Star ; Wu, Hao A ; Gao, >Liming ; Fan, Jeff

Re: [edk2] [PATCH v1 4/4] OvmfPkg : QemuFwCfgLib: Map DMA buffer with CommonBuffer when SEV is enable

On 07/31/17 21:31, Brijesh Singh wrote: > diff --git a/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c > b/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c > index d4e09c434e6a..7f42f38d1c05 100644 > --- a/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c > +++ b/OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c >

Re: [edk2] [PATCH 0/2] Implementation of UFS Device Config Protocol

Reviewed-by: Star Zeng -Original Message- From: Wu, Hao A Sent: Thursday, July 20, 2017 10:19 AM To: edk2-devel@lists.01.org Cc: Wu, Hao A ; Zeng, Star Subject: [PATCH 0/2] Implementation of UFS Device Config Protocol The

[edk2] [edk2-DSCSpecification Patch] DSC Spec: Remove Restriction on Using NULL in [LibraryClasses] Section

https://bugzilla.tianocore.org/show_bug.cgi?id=645 Cc: Liming Gao Cc: Michael Kinney Cc: Kevin W Shaw Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu ---

Re: [edk2] [PATCH v1 3/4] OvmfPkg: IommuDxe: Zero the shared page(s) on Unmap()

On 07/31/17 21:31, Brijesh Singh wrote: > To support the Map(), we allocate bounce buffer with C-bit cleared, > the buffer is referred as a DeviceAddress. Typically, DeviceAddress > is used as communication block between guest and hypervisor. When > guest is done with communication block, it calls