Reviewed-by: Giri P Mudusuru
> -Original Message-
> From: Yao, Jiewen
> Sent: Monday, August 22, 2016 7:05 PM
> To: edk2-devel@lists.01.org
> Cc: Mudusuru, Giri P ; Yarlagadda, Satya P
> ; Chan, Amy
> Subject: [PATCH] Maintainers.txt: Add IntelFsp2Pkg, IntelFsp2WrapperPkg and
> IntelSil
Many thanks, Jordan
Leif,
I am ok with that. Reviewed-by: Feng Tian
Thanks
Feng
-Original Message-
From: Justen, Jordan L
Sent: Tuesday, August 23, 2016 12:33 PM
To: Tian, Feng ; Leif Lindholm ;
edk2-devel@lists.01.org
Cc: Ard Biesheuvel ; Zeng, Star
; Laszlo Ersek ; Tian, Feng
Su
Feng,
The spec says " If the platform does not recognize the EFI_GUID in ResetData
the platform must pick a supported reset type to perform".
For the NULL instance of ResetSystemLib, I treat it as an implementation for a
Certain platform, then it's ok to let it pick a supported reset type to per
On 2016-08-18 20:12:16, Tian, Feng wrote:
> Jordan,
>
> Any comments on this? Is it ok for you?
>
Since there is a Contributed-under and Signed-off-by, it appears to
follow our current contribution process. It looks like Leif is
confirming that we can use the code under the "TianoCore Contributi
Star,
PciLib uses smaller address and fewer params, so those functions
can be simpler and smaller. We only want to use PciSegmentLib
when we have to.
Mike
> -Original Message-
> From: Zeng, Star
> Sent: Monday, August 22, 2016 7:09 PM
> To: Kinney, Michael D ; edk2-devel@lists.01.org
>
Do you need directly assert in ResetPlatformSpecific() for this NULL instance?
Other series look good to me
Reviewed-by: Feng Tian
Thanks
Feng
-Original Message-
From: Ni, Ruiyu
Sent: Wednesday, August 10, 2016 1:56 PM
To: edk2-devel@lists.01.org
Cc: Tian, Feng ; Chan, Amy
Subject: [
Reviewed-by: Sriram Subramanian
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Zhang
Lubo
Sent: Friday, August 19, 2016 1:23 PM
To: edk2-devel@lists.01.org
Cc: Ye Ting ; Fu Siyuan ; Wu Jiaxin
Subject: [edk2] [patch] MdeModulePkg:Fix bug in fun
When system power on, one processor (usually, it is thread 0) will be selected
to be BSP, it is descripted in IA32 Software Developer User Manual.
On POST phase, PI MP PPI and Protocol service provide SwitchBSP() service to
switch BSP to other enabled thread, it is descripted in PI specification.
This reverts commit c0bcd3433f33876c519bf5567e0ab69261b57fe9.
The above commit causes several regression of "echo" command:
1. Double quotes are not being stripped from the final text. UEFI Shell 2.2
section 3.4.5 chops out the quotes.
2. Output redirection is not working as expected. Text is bein
This reverts commit 95fc5a877502a6e6324300eed0136243d359fa96.
The above commit causes several regression of "echo" command:
1. Double quotes are not being stripped from the final text. UEFI Shell 2.2
section 3.4.5 chops out the quotes.
2. Output redirection is not working as expected. Text is bein
This reverts commit 0fcf8d4df85d861b6e721bd1d8abb449f05e15ed.
The above commit causes several regression of "echo" command:
1. Double quotes are not being stripped from the final text. UEFI Shell 2.2
section 3.4.5 chops out the quotes.
2. Output redirection is not working as expected. Text is bein
This reverts commit 95fc5a877502a6e6324300eed0136243d359fa96.
The above commit causes several regression of "echo" command:
1. Double quotes are not being stripped from the final text. UEFI Shell 2.2
section 3.4.5 chops out the quotes.
2. Output redirection is not working as expected. Text is bein
Mike,
That means PiDxeS3BootScriptLib will depend on both PciLib and PciSegmentLib.
Is there any negative impact for PiDxeS3BootScriptLib to only depend on
PciSegmentLib?
Thanks,
Star
-Original Message-
From: Kinney, Michael D
Sent: Tuesday, August 23, 2016 9:59 AM
To: Zeng, Star ; edk
Cc: Giri P Mudusuru
Cc: Satya Yarlagadda
Cc: Amy Chan
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jiewen Yao
---
Maintainers.txt | 12
1 file changed, 12 insertions(+)
diff --git a/Maintainers.txt b/Maintainers.txt
index d0d5f5c..d294c78 100644
--- a/Ma
Star,
I do not think all PCI opcodes should be updated to use the PciSegmentLib.
Only the "2" versions of the PCI opcodes should use the PciSegmentLib.
This means that functions like ScriptPciCfgRead() should not be modified at all
and should still use the PciLib. You will need to add a new MAC
Reviewed-by: Ye Ting
-Original Message-
From: Zhang, Lubo
Sent: Friday, August 19, 2016 3:53 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin
; Hegde Nagaraj P
Subject: [patch] MdeModulePkg:Fix bug in function AsciiStrToIp4.
If a FQDN contains 3 dots '.' like "a.
Reviewed-by: Fu Siyuan
> -Original Message-
> From: Zhang, Lubo
> Sent: Friday, August 19, 2016 3:53 PM
> To: edk2-devel@lists.01.org
> Cc: Fu, Siyuan ; Ye, Ting ; Wu,
> Jiaxin ; Hegde Nagaraj P
> Subject: [patch] MdeModulePkg:Fix bug in function AsciiStrToIp4.
>
> If a FQDN contains 3
Reviewed-by: Giri P Mudusuru
> -Original Message-
> From: Gao, Liming
> Sent: Sunday, August 21, 2016 11:29 PM
> To: edk2-devel@lists.01.org
> Cc: Mudusuru, Giri P ; Zhu, Yonghong
>
> Subject: [Patch] BaseTools GNU Makefile: Add the missing rules for cpp source
> file
>
> Cc: Giri P Mu
Reviewed-by: Giri P Mudusuru
> -Original Message-
> From: Gao, Liming
> Sent: Sunday, August 21, 2016 11:32 PM
> To: edk2-devel@lists.01.org
> Cc: Mudusuru, Giri P ; Zhu, Yonghong
>
> Subject: [Patch] BaseTools GnuMakefile: Update GCC Flags to the specific one
> with BUILD_ prefix
>
>
Reviewed-by: Yonghong Zhu
Best Regards,
Zhu Yonghong
-Original Message-
From: Gao, Liming
Sent: Monday, August 22, 2016 2:32 PM
To: edk2-devel@lists.01.org
Cc: Mudusuru, Giri P ; Zhu, Yonghong
Subject: [Patch] BaseTools GnuMakefile: Update GCC Flags to the specific one
with BUILD_
Reviewed-by: Yonghong Zhu
Best Regards,
Zhu Yonghong
-Original Message-
From: Gao, Liming
Sent: Monday, August 22, 2016 2:29 PM
To: edk2-devel@lists.01.org
Cc: Mudusuru, Giri P ; Zhu, Yonghong
Subject: [Patch] BaseTools GNU Makefile: Add the missing rules for cpp source
file
Cc: G
Reviewed-by: Erik Bjorge
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Yonghong Zhu
> Sent: Monday, August 22, 2016 4:46 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming ; Bjorge, Erik C
>
> Subject: [edk2] [Patch] BaseTools: update Bin
add Pkcs7Sign.exe and related pem file into BinaryFiles.txt for build
server to automatically build the binary win32 files.
Cc: Liming Gao
Cc: Erik Bjorge
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Yonghong Zhu
---
BaseTools/Source/BinaryFiles.txt | 9 -
1 f
Ard:
I have no comment. Reviewed-by: Liming Gao
mailto:liming@intel.com>>
Thanks
Liming
From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
Sent: Monday, August 22, 2016 7:33 PM
To: Shi, Steven
Cc: edk2-devel@lists.01.org; Gao, Liming ; Zhu, Yonghong
Subject: Re: [PATCH] BaseTools/G
On 08/22/16 02:35, Ard Biesheuvel wrote:
> Now that Laszlo's virtio-gpu-pci have removed the last remaining obstacle,
> we can get rid of the special PciHostBridgeDxe implementation in ArmVirtPkg,
> and move to the generic one. This will allow us to perform DMA above 4GB
> without bounce buffering,
On 22 August 2016 at 13:28, Shi, Steven wrote:
> Hi Ard,
> I like you path, and it makes the code look more clear. But in fact, old code
> should already ignore the dynamic RELA sections implicitly.
>
> Like below code, when the GenFw use .rela section info value, which is 0, to
> get .rela targ
Hi Ard,
I like you path, and it makes the code look more clear. But in fact, old code
should already ignore the dynamic RELA sections implicitly.
Like below code, when the GenFw use .rela section info value, which is 0, to
get .rela target applying section address in SecShdr, it will point to N
On 22 August 2016 at 12:04, Vikas C Sajjan wrote:
> Added missing dependency of FileExplorerLib, which was causing
> build error for ArmVirtXen, due to inclusion of ramdisk support.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Vikas C Sajjan
Thanks
Pushed as 5217
Hi Ard,
-Original Message-
From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
Sent: Monday, August 22, 2016 2:19 PM
To: Sajjan, Vikas C
Cc: Laszlo Ersek ; edk2-de...@ml01.01.org;
leif.lindh...@linaro.org; Wei, Dong (Office of the CTO)
Subject: Re: [PATCH v3 0/2] Add Ramdisk suppor
When building PIE (ET_DYN) executables, an additional RELA section is
emitted (in addition to the per-section .rela.text and .rela.data sections)
that is intended to be resolved at runtime by a ET_DYN compatible loader.
At the moment, due to the fact that we don't support GOT based relocations,
th
Added missing dependency of FileExplorerLib, which was causing
build error for ArmVirtXen, due to inclusion of ramdisk support.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Vikas C Sajjan
---
ArmVirtPkg/ArmVirt.dsc.inc | 4
1 file changed, 4 insertions(+)
diff --g
Adds the RAMDisk support to ArmVirtPkg platforms.
This patch actually ports OvmfPkg commit 259d87146b07 to
ArmVirtPkg.
Contributed-under: TianoCore Contribution Agreement 1.0
Reviewed-by: Laszlo Ersek
Signed-off-by: Vikas C Sajjan
---
ArmVirtPkg/ArmVirt.dsc.inc | 16
This series adds ramdisk driver support to ArmVirtPkg platforms.
It actually ports OvmfPkg commit 259d87146b07 to ArmVirtPkg.
changes since v3:
- Added missing FileExplorerLib, which was causing build error for
ArmVirtXen.
changes since v2:
- Made changes as suggested by Laszlo Ersek
Since ArmVirt.dsc.inc is included in all the ArmVirt dsc files,
move inclusion of AcpiTableDxe.inf to ArmVirt.dsc.inc.
Contributed-under: TianoCore Contribution Agreement 1.0
Reviewed-by: Laszlo Ersek
Signed-off-by: Vikas C Sajjan
---
ArmVirtPkg/ArmVirt.dsc.inc | 6 ++
ArmVirtPkg/ArmV
Ard,
Your patch rock! Thank you let us know and contribute this optimization.
Steven Shi
Intel\SSG\STO\UEFI Firmware
Tel: +86 021-61166522
iNet: 821-6522
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Gao, Liming
> Sent: Monday, August
On 22 August 2016 at 10:01, Sajjan, Vikas C wrote:
>
> Hi Ard,
>
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Monday, August 22, 2016 12:46 PM
> To: Sajjan, Vikas C
> Cc: Laszlo Ersek ; edk2-de...@ml01.01.org;
> leif.lindh...@linaro.org; Wei, Don
Reviewed-by: Liming Gao
> -Original Message-
> From: Zhu, Yonghong
> Sent: Monday, August 22, 2016 3:24 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming
> Subject: [Patch] BaseTools: add capsule image header for auth FMP capsule
> file
>
> in last commit 91ae29, it missed to add the
>
Ard:
Thanks for the detail information. Your patch makes sense. Reviewed-by:
Liming Gao
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ard Biesheuvel
> Sent: Monday, August 22, 2016 3:26 PM
> To: Gao, Liming
> Cc: edk2-devel-01
> Sub
Reviewed-by: jiewen@intel.com
> -Original Message-
> From: Yarlagadda, Satya P
> Sent: Thursday, August 18, 2016 1:21 PM
> To: edk2-devel@lists.01.org
> Cc: Ma, Maurice ; Yao, Jiewen
> ; Mudusuru, Giri P
> Subject: [PATCH] IntelFsp2Pkg: Add function to get FspInfoHeader before
> stack
Reviewed by: jiewen@intel.com
> -Original Message-
> From: Mudusuru, Giri P
> Sent: Monday, August 22, 2016 1:19 PM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen
> Subject: [edk2] [PATCH] IntelSiliconPkg: Fixed VBT size bug, typos and
> updated comments
>
> 1) Fixed the VBT size fr
Hi Ard,
-Original Message-
From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
Sent: Monday, August 22, 2016 12:46 PM
To: Sajjan, Vikas C
Cc: Laszlo Ersek ; edk2-de...@ml01.01.org;
leif.lindh...@linaro.org; Wei, Dong (Office of the CTO)
Subject: Re: [PATCH v3 0/2] Add Ramdisk supp
Reviewed-by: Yonghong Zhu
Best Regards,
Zhu Yonghong
-Original Message-
From: Gao, Liming
Sent: Monday, August 22, 2016 11:09 AM
To: edk2-devel@lists.01.org
Cc: Ard Biesheuvel ; Zhu, Yonghong
Subject: [Patch] BaseTools PeCoffLib: Fix the issue to get RelocationsStripped
from TE ima
On 22 August 2016 at 09:07, Gao, Liming wrote:
> Ard:
> This is done to convert PE/COFF image to TE image. TE image Relocation
> VirtualAddress will be set to non-zero. You can find the following logic in
> GenFw.c line 2515. Then, PeCoffLib can restore RELOCS_STRIPPED flag from TE
> image.
>
On 22 August 2016 at 09:22, Gao, Liming wrote:
> Ard:
>Sorry. I miss this patch. GccBase.lds is used by GCC44~GCC5. GCC44~GCC49
> doesn't enable LTO, GCC5 enables LTO. That means RELA section is not used
> with LTO or without LTO. Right?
The allocatable RELA section is only emitted when lin
in last commit 91ae29, it missed to add the
EFI_FIRMWARE_MANAGEMENT_CAPSULE_IMAGE_HEADER for the auth FMP capsule.
Cc: Liming Gao
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Yonghong Zhu
---
BaseTools/Source/Python/GenFds/Capsule.py | 15 +++
BaseTools
Ard:
Sorry. I miss this patch. GccBase.lds is used by GCC44~GCC5. GCC44~GCC49
doesn't enable LTO, GCC5 enables LTO. That means RELA section is not used with
LTO or without LTO. Right?
Thanks
Liming
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent:
On 22 August 2016 at 09:07, Ard Biesheuvel wrote:
> On 20 August 2016 at 05:01, Sajjan, Vikas C wrote:
>>
>>
>> -Original Message-
>> From: Laszlo Ersek [mailto:ler...@redhat.com]
>> Sent: Friday, August 19, 2016 6:58 PM
>> To: Sajjan, Vikas C ; edk2-de...@ml01.01.org;
>> ard.biesheu...@
On 10 August 2016 at 10:55, Ard Biesheuvel wrote:
> The CLANG38 toolchain creates a PIE binary at link time. This is
> necessary since the LTO code generation may otherwise result in
> code that cannot execute correctly when loaded above 2 GB.
>
> PIE executables contain a RELA section consisting
On 20 August 2016 at 05:01, Sajjan, Vikas C wrote:
>
>
> -Original Message-
> From: Laszlo Ersek [mailto:ler...@redhat.com]
> Sent: Friday, August 19, 2016 6:58 PM
> To: Sajjan, Vikas C ; edk2-de...@ml01.01.org;
> ard.biesheu...@linaro.org; leif.lindh...@linaro.org
> Cc: Wei, Dong (Office
Ard:
This is done to convert PE/COFF image to TE image. TE image Relocation
VirtualAddress will be set to non-zero. You can find the following logic in
GenFw.c line 2515. Then, PeCoffLib can restore RELOCS_STRIPPED flag from TE
image.
if (((PeHdr->Pe32.FileHeader.Characteristics &
EFI_IMA
50 matches
Mail list logo