Hi Liming,
> -Original Message-
> From: Gao, Liming
> Sent: Wednesday, June 5, 2019 1:57 PM
> To: devel@edk2.groups.io; Lu, XiaoyuX
> Cc: Bi, Dandan ; Wang, Jian J
> Subject: RE: [edk2-devel] [PATCH v1 1/1] CryptoPkg/IntrinsicLib: Fix
> CLANG38 IA32 build problem
>
> Xiaoyu:
>
> >-
On Fri, 31 May 2019 at 01:06, Ma, Maurice wrote:
>
> Reviewed-by: Maurice Ma
>
This patch has now been pushed, while I don't think it is a bugfix,
and we are in the middle of the hard freeze period.
> > -Original Message-
> > From: Dong, Guo
> > Sent: Thursday, May 30, 2019 2:52
> > To
Xiaoyu:
>-Original Message-
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Xiaoyu Lu
>Sent: Wednesday, June 05, 2019 1:25 PM
>To: devel@edk2.groups.io
>Cc: Lu, XiaoyuX ; Bi, Dandan ;
>Wang, Jian J
>Subject: [edk2-devel] [PATCH v1 1/1] CryptoPkg/IntrinsicLib: Fix C
NumApsExecuting is only used when InitFlag == ApInitConfig for
counting the processor count.
The patch changes Ia32 version of waking up vector assembly code
to align to x64 version of waking up vector assembly code.
After the change both versions of waking up vector increase
NumApsExecuting when
The patch fixes the bug that the memory under 1MB is modified by
firmware in S3 boot.
Root cause is a racing condition in MpInitLib:
1. BSP: WakeUpByInitSipiSipi is set by NotifyOnS3SmmInitDonePpi()
2. BSP: WakeUpAP() wakes all APs to run certain procedure.
2.1. AllocateResetVector() uses <1MB m
v2:
Answer Laszlo's comments:
1. Add 1/2 patch to change IA32 waking up vector assembly code to increase
NumApsExecuting only for ApInitConfig.
2. Change 2/2 patch to remove the unnecessary debug messages.
Ray Ni (2):
UefiCpuPkg/MpInitLib: increase NumApsExecuting only for ApIni
Rebecca,
You may help to fix it. We don't have FreeBSD machine and are not familiar with
build env in it.
Regards,
Jian
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> rebe...@bluestop.org
> Sent: Wednesday, June 05, 2019 9:52 AM
> To: dev
When use clang-3.8 to build the NetworkPkg, compiler optimization
may use memcpy for memory copy. For example:
CryptoPkg/Library/OpensslLib/openssl/ssl/ssl_rsa.c:918: undefined
reference to `memcpy'`
Compiler optimization is sophisticated, but we can work around it
use __attribute__((__used__))
This change will be added after 201905 stable tag.
>-Original Message-
>From: Zhu, Yonghong
>Sent: Wednesday, June 05, 2019 8:51 AM
>To: devel@edk2.groups.io
>Cc: Feng, Bob C ; Gao, Liming
>Subject: [Patch] Maintainers.txt: Remove Yonghong from BaseTools Reviewer
>
>As Yonghong has some
I know it's not an officially supported platform, but a recent update
has broken building CryptoPkg under FreeBSD. I've tried building both
Armada80x0McBin (shown below) and OvmfPkg, and both fail.
"/usr/local/bin/aarch64-unknown-freebsd13.0-gcc"
@/home/bcran/workspace/Build/Armada80x0McBin-AARC
From: Bret Barkelew
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853
Code change form Mu project:
https://github.com/microsoft/mu_basecore/blob/release/201903/
MdeModulePkg/Universal/CapsulePei/UefiCapsule.c#L801
Separate the capsule check function from GetCapsuleDescriptors
to AreCapsu
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1853
AreCapsulesStaged do not need to return the status, only boolean
result is useful. So directly return a boolean value.
Cannot initialize the variable at its definition.
GetScatterGatherHeadEntries: use allocated buffer instead of fixed
arra
v1:
Sperate the capsule check function from GetCapsuleDescriptors
and name it to AreCapsulesStaged.
Rename GetCapsuleDescriptors to GetScatterGatherHeadEntries.
And optimize its to remove the duplicated code.
V2:
Clarify the changes of the patch and descript in the commit
message.
V3:
Add the cod
Update the list of additional licenses in Readme.md. For additional
licenses from a git submodule, provide a link to the license file
in the remote git repository. This makes the links in Readme.md
when viewed from the edk2 repository GitHub landing page
https://github.com/tianocore/edk2/ functio
As Yonghong has some other focus, remove him from the reviewer.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yonghong Zhu
---
Maintainers.txt | 1 -
1 file changed, 1 deletion(-)
diff --git a/Maintainers.txt b/Maintainers.txt
index 010ec31..b41bd59 100644
--- a/Maintainers.txt
+++ b/Maintainers
Hi everyone,
Hao pushed the patch because:
1. it's a bug fix
2. it got a R-b.
I don't think it's his fault.
For #1, it's gap in process.
For #2, it's my fault. Because even the patch title says EmulatorPkg but the
patch itself changes MdeModulePkg, I am still the person who can approve
changes
Reviewed-by: Liming Gao
>-Original Message-
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Laszlo Ersek
>Sent: Wednesday, June 05, 2019 2:59 AM
>To: edk2-devel-groups-io
>Cc: Ard Biesheuvel ; Wu, Hao A
>; Wang, Jian J ; Gao, Liming
>; Ni, Ray ; Zeng, Star
>
>Subj
Reviewed-by: Liming Gao
>-Original Message-
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Wang, Jian J
>Sent: Wednesday, June 05, 2019 5:19 AM
>To: devel@edk2.groups.io
>Cc: Bi, Dandan ; Lu, XiaoyuX
>Subject: [edk2-devel] [PATCH] CryptoPkg/OpensslLib: fix VS2017
Hi,
Just to clarify - this is a single patch, wrongly numbered by mistake.
Best regards,
Marcin
śr., 5 cze 2019 o 01:52 Marcin Wojtas napisał(a):
>
> Linux FWTS test of the ARM SBSA compliance suite
> fail on using unsupported _CID in UART node. Remove it.
>
> Contributed-under: TianoCore Contr
Another one is to fix Maintainers.txt.
[edk2-devel] [Patch] Maintainers.txt: Remove Network maintainers for
MdeModulePkg/Universal/Network
>-Original Message-
>From: Wang, Jian J
>Sent: Wednesday, June 05, 2019 5:21 AM
>To: devel@edk2.groups.io; Kinney, Michael D ;
>Gao, Liming
>Cc: lei
Linux FWTS test of the ARM SBSA compliance suite
fail on using unsupported _CID in UART node. Remove it.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas
---
Silicon/Marvell/Armada7k8k/AcpiTables/Armada70x0Db/Dsdt.asl| 1 -
Silicon/Marvell/Armada7k8k/AcpiT
When assigning a physical PCIe device to a QEMU/KVM guest, PciBusDxe may
find that the extended config space is not (fully) implemented. In
LocatePciExpressCapabilityRegBlock(), "CapabilityEntry" may be read as
0x_ at a given config space offset, after which the loop gets
stuck spinning on
There's a fix for build failure.
*[edk2-devel] [PATCH] CryptoPkg/OpensslLib: fix VS2017 build failure
Regards,
Jian
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Michael D Kinney
> Sent: Wednesday, June 05, 2019 4:29 AM
> To: Gao, Liming
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1878
This issue is specific to VS2017 which tries to resolve symbol referenced
by a symbol not really referenced eventually.
ossl_init_load_crypto_strings
-> err_load_crypto_strings_int (not really referenced)
-> ERR_load_OSSL_STORE_strings
Beca
wt., 4 cze 2019 o 22:14 Leif Lindholm napisał(a):
>
> On Tue, Jun 04, 2019 at 01:39:07PM -0500, Jeremy Linton wrote:
> > Hi Marcin,
> >
> > On 6/4/19 12:53 PM, Marcin Wojtas via Groups.Io wrote:
> > > Fill in the basic requirements of the SMBIOS specification by specifying
> > > the minimum requir
Thanks Laszlo. I will send a V3 with the updated links.
Mike
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io]
> On Behalf Of Laszlo Ersek
> Sent: Tuesday, June 4, 2019 11:58 AM
> To: devel@edk2.groups.io; Kinney, Michael D
>
> Subject: Re: [edk2-devel] [Pa
Hello,
The patches for edk2-stable201905 were reviewed this morning.
The following 2 patches are in scope for the hard freeze.
* [edk2-devel] [PATCH for-edk2-stable201905] Revert "EmulatorPkg: don't display
the cpu current speed"
* [edk2-devel] [Patch V2] edk2: Update additional licenses in Read
On Tue, Jun 04, 2019 at 01:39:07PM -0500, Jeremy Linton wrote:
> Hi Marcin,
>
> On 6/4/19 12:53 PM, Marcin Wojtas via Groups.Io wrote:
> > Fill in the basic requirements of the SMBIOS specification by specifying
> > the minimum required structures. The basic fixup is performed.
> > CPU/DRAM freque
This reverts commit 7cea4d71a8a87a93924a07ab32348332f5881ef9.
Said commit was not suitable for pushing during the edk2-stable201905 hard
feature freeze; it was pushed only by mistake. The subject line referenced
EmulatorPkg, but the patch changed MdeModulePkg/UiApp, regressing the
display of the C
Hi Mike,
On 06/04/19 20:40, Michael D Kinney wrote:
> Update the list of additional licenses in Readme.md. For additional
> licenses from a git submodule, provide a link to the license file
> in the remote git repository. This makes the links in Readme.md
> when viewed from the edk2 repository G
:)
For future reference:
- browse to file
- click the "history" button
- click the "<>" button for the top commit ("Browse the repository at
this point in history")
- browse to the file again
(If someone has a less tedious route I'd appreciate it, but it works.)
/
Leif
On Tue, Jun 04, 201
Update the list of additional licenses in Readme.md. For additional
licenses from a git submodule, provide a link to the license file
in the remote git repository. This makes the links in Readme.md
when viewed from the edk2 repository GitHub landing page
https://github.com/tianocore/edk2/ functio
Hi Marcin,
On 6/4/19 12:53 PM, Marcin Wojtas via Groups.Io wrote:
Fill in the basic requirements of the SMBIOS specification by specifying
the minimum required structures. The basic fixup is performed.
CPU/DRAM frequency is obtained via SampleAtResetlib and the DRAM size
is calculated from the i
Leif,
I was wanting to figure out how to do the direct link
to the license material. Thanks for providing that!
I will send V2.
Mike
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io]
> On Behalf Of Leif Lindholm
> Sent: Tuesday, June 4, 2019 11:00 AM
> To:
On Tue, Jun 04, 2019 at 10:12:48AM -0700, Michael D Kinney wrote:
> * Remove refrences EdkCompatibilityPkg
> * Update link to OpenSSL to https://github.com/openssl/openssl
> * Add link to https://github.com/ucb-bar/berkeley-softfloat-3
> for ArmSoftFloatLib
Bikeshed warning:
> Signed-off-by: Mi
On 06/04/19 19:12, Michael D Kinney wrote:
> * Remove refrences EdkCompatibilityPkg
> * Update link to OpenSSL to https://github.com/openssl/openssl
> * Add link to https://github.com/ucb-bar/berkeley-softfloat-3
> for ArmSoftFloatLib
>
> Signed-off-by: Michael D Kinney
> ---
> Readme.md | 4 +
Fill in the basic requirements of the SMBIOS specification by specifying
the minimum required structures. The basic fixup is performed.
CPU/DRAM frequency is obtained via SampleAtResetlib and the DRAM size
is calculated from the information stored in the HOB list.
Add new Armada SmbiosPlatformDxe
Hi Liming,
Sure. Here is what I get after building our NIC driver binary with VS2015x86
and VS2017 toolchains:
pcfist@pcfist-pc:/mnt/c/src/uefi/udk2018$ du -h
Build/xgbe_atl/RELEASE_VS2015x86/X64/xgbe_atl.efi
Build/xgbe_atl/RELEASE_VS2017/X64/xgbe_atl.efi
36K Build/xgbe_atl/RELEASE_VS2015x
* Remove refrences EdkCompatibilityPkg
* Update link to OpenSSL to https://github.com/openssl/openssl
* Add link to https://github.com/ucb-bar/berkeley-softfloat-3
for ArmSoftFloatLib
Signed-off-by: Michael D Kinney
---
Readme.md | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
On 06/04/19 16:35, Leif Lindholm wrote:
> On Tue, Jun 04, 2019 at 10:10:59PM +0800, Liming Gao wrote:
>> MdeModulePkg/Universal/Network has been moved to NetworkPkg. So,
>> MdeModulePkg/Universal/Network can be removed in Maintainers.txt.
>>
>> Signed-off-by: Liming Gao
>> Cc: Andrew Fish
>> Cc:
I have noticed the resulting binaries are about twice as large when
using VS2017 toolchain compared to the ones built with VS2015. It appears
this is caused by /WHOLEARCHIVE linker flag used by this toolchain. This
flag was previously removed from VS2015 toolchain due to compatibility
issues. I bel
This patch removes /WHOLEARCHIVE flag from VS2017 toolchain configuration
to reduce the size of resulting binaries.
GitHub branch: https://github.com/pcfist/edk2/tree/fix_vs2017_linker_config
Roman Agafonov (1):
BaseTools: don't use WHOLEARCHIVE linker flag for VS2017 toolchain
BaseTools/Conf
(+Gerd)
On 06/04/19 13:28, Pavan Kumar Aravapalli wrote:
> Hi Laszlo,
>
> Thank you for your quick response, and apologies for the your pervious mail
> thread which i could not observed as I only monitor the mail [
> pavankuma...@accelerite.com ]inbox. And I am not receiving mail reply's to my
Can you show the size data with VS2017 and VS2015 for the same code?
Thanks
Liming
> -Original Message-
> From: Roman Agafonov [mailto:roman.agafo...@aquantia.com]
> Sent: Tuesday, June 4, 2019 11:41 PM
> To: devel@edk2.groups.io
> Cc: Feng, Bob C ; Gao, Liming ;
> Zhu, Yonghong
> Subjec
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#41870): https://edk2.groups.io/g/devel/message/41870
Mute This Topic: https://groups.io/mt/31927809/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#41869): https://edk2.groups.io/g/devel/message/41869
Mute This Topic: https://groups.io/mt/31927806/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#41868): https://edk2.groups.io/g/devel/message/41868
Mute This Topic: https://groups.io/mt/31927802/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/
*Reminder:* TianoCore Design / Bug Triage - EMEA
*When:* Wednesday, 5 June 2019, 8:00am to 9:00am, (GMT-07:00) America/Los
Angeles
*Where:* https://zoom.us/j/695893389
View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=488405 )
*Organizer:* Stephano Cetola stephano.cet...@linux.int
On Tue, Jun 04, 2019 at 10:10:59PM +0800, Liming Gao wrote:
> MdeModulePkg/Universal/Network has been moved to NetworkPkg. So,
> MdeModulePkg/Universal/Network can be removed in Maintainers.txt.
>
> Signed-off-by: Liming Gao
> Cc: Andrew Fish
> Cc: Laszlo Ersek
> Cc: Leif Lindholm
> Cc: Michae
Ard:
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ard
> Biesheuvel
> Sent: Tuesday, June 4, 2019 9:42 PM
> To: Gao, Liming
> Cc: devel@edk2.groups.io; Ni, Ray ; Leif Lindholm
> ; Laszlo Ersek ;
> Kinney, Michael D ; Liu, Zhiguang
> ; Just
MdeModulePkg/Universal/Network has been moved to NetworkPkg. So,
MdeModulePkg/Universal/Network can be removed in Maintainers.txt.
Signed-off-by: Liming Gao
Cc: Andrew Fish
Cc: Laszlo Ersek
Cc: Leif Lindholm
Cc: Michael D Kinney
---
Because MdeModulePkg/Universal/Network has been moved to Net
https://bugzilla.tianocore.org/show_bug.cgi?id=1876
Add a PCD for skipping Hdd password prompt.
If device is in the locked status while attempting to skip
password prompt, device will keep locked and system
continue to boot.
If device is in the unlocked status while attempting to skip
password prom
On Tue, 4 Jun 2019 at 15:40, Gao, Liming wrote:
>
> Ard:
>
> > -Original Message-
> > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ard
> > Biesheuvel
> > Sent: Tuesday, June 4, 2019 7:53 PM
> > To: edk2-devel-groups-io ; Ni, Ray
> > ; Leif Lindholm ; Gao, Liming
Ard:
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ard
> Biesheuvel
> Sent: Tuesday, June 4, 2019 7:53 PM
> To: edk2-devel-groups-io ; Ni, Ray ;
> Leif Lindholm ; Gao, Liming
> ; Laszlo Ersek ; Kinney, Michael D
>
> Cc: Liu, Zhiguang ; Jus
On Tue, 4 Jun 2019 at 10:50, Ni, Ray wrote:
>
> Zhiguang,
> I made a mistake here.
> I thought you changed the EmulatorPkg to not display the CPU speed
> but actually what you changed impacts all platforms.
> With this change, OVMF cannot show the CPU speed either.
>
> Can you please roll back thi
Pavan,
On 06/03/19 21:21, Pavan Kumar Aravapalli wrote:
> * We have CentOS flavoured qcow2 image which is used to boot Guest
> VM's in Apache CloudStack. We are trying to enable these CentOS
> qcow2 images with UEFI support.
I told you weeks ago to take this question to the CentOS mailing li
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Laszlo
> Ersek
> Sent: Monday, June 3, 2019 11:21 PM
> To: devel@edk2.groups.io; Ni, Ray
> Cc: Dong, Eric ; Sathyanarayanan, Nandagopal
>
> Subject: Re: [edk2-devel] [PATCH] UefiCpuPkg/MpInitLib: Decrease
> NumApsExecuting on
Zhiguang,
I made a mistake here.
I thought you changed the EmulatorPkg to not display the CPU speed
but actually what you changed impacts all platforms.
With this change, OVMF cannot show the CPU speed either.
Can you please roll back this changes (if it was pushed in)?
To close the Bugzilla, I t
Eric,
I have envisioned less granular packages for advanced features. One of the
goals for MinPlatform is to improve usability during porting. The idea is that
you would do basic board porting with minimal effort to get your system
functional. Then you would enable additional features by add
59 matches
Mail list logo
