Re: [edk2] [PATCH 0/3] OvmfPkg: enable 64-bit PCI MMIO

On 03/10/2016 10:05 PM, Laszlo Ersek wrote: > This is the last wave(-let) I have for now. > > Public branch: > . > > Thomas, it would be great if you could test it. Cool stuff, thanks Laszlo! I'll build and test it, although I'm not

[edk2] [Patch] CryptoPkg: Fix the potential system hang issue

This patch is used to fix the potential system hang caused by the NULL 'time' parameter usage. Cc: David Woodhouse Cc: Long Qin Cc: Ye Ting Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu

Re: [edk2] [PATCH v3] NetworkPkg: Add URI configuration form to HTTP boot driver.

On Thu, Mar 10, 2016 at 12:31:18PM +0100, Laszlo Ersek wrote: > On 03/10/16 11:02, Gary Lin wrote: > > On Thu, Mar 10, 2016 at 10:20:12AM +0100, Laszlo Ersek wrote: > >> On 03/10/16 08:49, Gary Lin wrote: > > >>> I found that it's related to iPXE. If I disable iPXE with > >>> > >>> "-netdev

Re: [edk2] [Patch] NetworkPkg: fix function comments in HttpBootDxe.

Reviewed-by: Jiaxin Wu > -Original Message- > From: Fu, Siyuan > Sent: Thursday, March 10, 2016 10:10 AM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; Zhang, Lubo > Subject: [Patch] NetworkPkg: fix function comments in

Re: [edk2] [PATCH v6] MdePkg: Add UEFI2.6 HII Image Ex and Image Decoder protocol definition.

Cecil: Please run BaseTools\Scripts\PatchCheck.py for your patch and fix the invalid coding format. And, please create this patch based on the latest edk2 trunk. I also meet with the issue to apply this patch. Thanks Liming > -Original Message- > From: edk2-devel

Re: [edk2] [PATCH v6] MdePkg: Add UEFI2.6 HII Image Ex and Image Decoder protocol definition.

Hi, Cecil & Eric I can't apply this git patch at all. what code revision are you using to create/review the patch? I want to emphasize the review path should be based on latest git revision. Please follow it. Thanks Feng -Original Message- From: Dong, Eric Sent: Wednesday, March 9,

[edk2] [PATCH] ArmVirtPkg/VirtFdtDxe: set /chosen/linux, pci-probe-only to 1 in DTB

Unlike Linux on x86, which typically honors the PCI configuration performed by the firmware, Linux on ARM assumes that the PCI subsystem needs to be configured from scratch. This is not entirely unreasonable given the historical background of embedded systems using very basic bootloaders, but is

Re: [edk2] [Patch] NetworkPkg: fix function comments in HttpBootDxe.

Reviewed-by: Zhang, Lubo -Original Message- From: Fu, Siyuan Sent: Thursday, March 10, 2016 10:10 AM To: edk2-devel@lists.01.org Cc: Wu, Jiaxin ; Zhang, Lubo Subject: [Patch] NetworkPkg: fix function comments in

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

Thanks for comments. /Jiaxin > -Original Message- > From: Wu, Jiaxin > Sent: Friday, March 11, 2016 10:09 AM > To: David Woodhouse ; edk2-de...@ml01.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Long, > Qin >

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

> -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Thursday, March 10, 2016 7:31 PM > To: Wu, Jiaxin ; edk2-de...@ml01.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Long, > Qin >

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

You are right. The change for EDKII_openssl patch is unreasonable. Actually, this change was based on the openssl version 1.0.2e. The issue has been fixed in the later openssl version. So, just ignore it. I will create another patch for whole 'openssl\ssl' feature requirement to resolve this

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

Hi David, Good point. I agree the change for 'time' should be standalone commit. This change is to avoid the potential system hang caused by the NULL 'time' parameter usage. Thanks. Jiaxin > -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Thursday,

[edk2] [PATCH 0/3] OvmfPkg: enable 64-bit PCI MMIO

This is the last wave(-let) I have for now. Public branch: . Thomas, it would be great if you could test it. Cc: Gerd Hoffmann Cc: Jordan Justen Cc: Marcel Apfelbaum Cc:

[edk2] [PATCH 3/3] OvmfPkg: PciHostBridgeLib: install 64-bit PCI host aperture

On the normal boot path (which is when PciHostBridgeDxe runs), the PCDs have been calculated; report the 64-bit PCI host aperture to PciHostBridgeDxe. In the Ia32 build, the PCD values (zeros) come directly from the DEC file, and this patch makes no difference. Cc: Gerd Hoffmann

[edk2] [PATCH 2/3] OvmfPkg: PlatformPei: determine the 64-bit PCI host aperture for X64 DXE

The main observation about the 64-bit PCI host aperture is that it is the highest part of the useful address space. It impacts the top of the GCD memory space map, and, consequently, our maximum address width calculation for the CPU HOB too. Thus, modify the GetFirstNonAddress() function to

[edk2] [PATCH 1/3] OvmfPkg: PlatformPei: factor out GetFirstNonAddress()

Factor out the expression that is currently the basis of the address width calculation into a standalone function. In the next patches we'll raise the return value under certain circumstances. Cc: Gerd Hoffmann Cc: Jordan Justen Cc: Marcel Apfelbaum

Re: [edk2] [PATCH v2 0/6] OvmfPkg: enable PCIe on Q35

On 03/10/16 19:20, Jordan Justen wrote: > On 2016-03-10 02:34:58, Laszlo Ersek wrote: >> Hi Jordan, >> >> On 03/08/16 15:19, Laszlo Ersek wrote: >>> Version 2 of . >>> >>> News for v2: following Gerd's advice, I inserted a new patch (as #2),

Re: [edk2] EDK2 Staging Repository

On 03/10/16 19:29, Kinney, Michael D wrote: >> -Original Message- >> From: Laszlo Ersek [mailto:ler...@redhat.com] >> Keeping work-in-progress modules in a separate repo has a critical >> drawback: those modules will not share the commit history of the main >> edk2 repo. > > The issue

Re: [edk2] EDK2 Staging Repository

> -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, March 10, 2016 2:03 AM > To: Kinney, Michael D ; Justen, Jordan L > ; Mangefeste, Tony ; > edk2- > de...@lists.01.org

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of David > Woodhouse > Sent: Thursday, March 10, 2016 10:01 AM > To: Long, Qin ; Wu, Jiaxin ; > edk2-de...@ml01.01.org > Cc: Ye, Ting ;

[edk2] KVM Forum 2016: Call For Participation

= KVM Forum 2016: Call For Participation August 24-26, 2016 - Westin Harbor Castle - Toronto, Canada (All submissions must be received before midnight May 1, 2016) =

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

On Thu, 2016-03-10 at 17:12 +, Long, Qin wrote: > > This patch series should be based on the old version, before the > back-porting of upstreaming patch was done. > We should have no need to add the extra patches on OpenSSL now for > OpensslTlsLib build now. Do you have a simple test case

Re: [edk2] EDK2 Staging Repository

Yep, I've heard that loud and clear. It must not be only web UI, but can be over email _if_ we move with Gerrit or something similar. -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Thursday, March 10, 2016 9:34 AM To: Mangefeste, Tony

Re: [edk2] [PATCH v2] BaseTools ConvertMasmToNasm: Support Python 3

Reviewed-by: Erik Bjorge > -Original Message- > From: Justen, Jordan L > Sent: Wednesday, March 9, 2016 5:46 PM > To: edk2-devel@lists.01.org > Cc: Justen, Jordan L ; Zhu, Yonghong > ; Gao, Liming

Re: [edk2] EDK2 Staging Repository

On 03/10/16 17:55, Mangefeste, Tony wrote: > This is fantastic, because the other two things I would like for us all to > consider is Gerrit+Jenkins support. Sorry for repeating myself :(, but: Please make sure that all reviews done on the web are forwarded to the mailing list, and that I can

Re: [edk2] Tianocore Community Update 2016 #1

On 03/10/16 18:20, Mangefeste, Tony wrote: > Very good thoughts, Laszlo. > > The more I think about GitHub integration, the more I'm leaning > against it. Another reason is that we can assign groups to certain > organizational emails to allow/grant special rights to specific > classes of issues.

Re: [edk2] [PATCH] CryptoPkg/OpensslLib: Convert saved opensslconf.h to DOS line endings

Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: David Woodhouse [mailto:david.woodho...@intel.com] > Sent: Thursday, March 10, 2016 3:15 AM > To: edk2-de...@ml01.01.org > Cc: Long, Qin > Subject: [PATCH]

Re: [edk2] Tianocore Community Update 2016 #1

Very good thoughts, Laszlo. The more I think about GitHub integration, the more I'm leaning against it. Another reason is that we can assign groups to certain organizational emails to allow/grant special rights to specific classes of issues. For example, it's plausible that we would give

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

David, This patch series should be based on the old version, before the back-porting of upstreaming patch was done. We should have no need to add the extra patches on OpenSSL now for OpensslTlsLib build now. Best Regards & Thanks, LONG, Qin > -Original Message- > From: David

Re: [edk2] EDK2 Staging Repository

On 3/10/16 9:55 AM, Mangefeste, Tony wrote: This is fantastic, because the other two things I would like for us all to consider is Gerrit+Jenkins support. Why Gerrit? Everyone I've seen talk about it hate its UI. FWIW I seem to be leading a bit of a rebellion at $work, since I setup a demo

Re: [edk2] EDK2 Staging Repository

This is fantastic, because the other two things I would like for us all to consider is Gerrit+Jenkins support. I'll be talking to you to get more details. -Original Message- From: Gerd Hoffmann [mailto:kra...@redhat.com] Sent: Thursday, March 10, 2016 12:39 AM To: Bruce Cran

Re: [edk2] Tianocore Community Update 2016 #1

On 03/10/16 14:11, David Woodhouse wrote: > On Thu, 2016-03-10 at 13:58 +0100, Laszlo Ersek wrote: >> >>> Sure, actually getting vendor buy-in for that is a completely different >>> story. But let's not design the system to make it hard :) >> >> I couldn't buy in. > > That's fine. I'm not asking

Re: [edk2] [patch] SecurityPkg: Fix Tcg2 SubmitRequestToPreOSFunction() fail in second times.

No, not for TPM1.2, because TPM1.2 PP does not have RequestParameter field. Thank you Yao Jiewen -Original Message- From: Zhang, Chao B Sent: Thursday, March 10, 2016 6:07 AM To: Yao, Jiewen; edk2-de...@ml01.01.org Subject: RE: [patch] SecurityPkg: Fix Tcg2

Re: [edk2] [patch] SecurityPkg: Clear LocalAuthSession content after use.

Reviewed-by: Chao Zhang Thanks & Best regards Chao Zhang -Original Message- From: Yao, Jiewen Sent: Thursday, March 10, 2016 1:53 PM To: edk2-de...@ml01.01.org Cc: Yao, Jiewen; Zhang, Chao B Subject: [patch] SecurityPkg: Clear LocalAuthSession content after

Re: [edk2] [patch] SecurityPkg: Fix Tcg2 SubmitRequestToPreOSFunction() fail in second times.

Jiewen: The patch is good to me. Do we have the same issue for TPM1.2 ? Thanks & Best regards Chao Zhang -Original Message- From: Yao, Jiewen Sent: Thursday, March 10, 2016 1:47 PM To: edk2-de...@ml01.01.org Cc: Yao, Jiewen; Zhang, Chao B Subject: [patch] SecurityPkg: Fix

Re: [edk2] Tianocore Community Update 2016 #1

On Thu, 2016-03-10 at 13:58 +0100, Laszlo Ersek wrote: > > > Sure, actually getting vendor buy-in for that is a completely different > > story. But let's not design the system to make it hard :) > > I couldn't buy in. That's fine. I'm not asking you to. I'm just asking that we don't make it

Re: [edk2] Tianocore Community Update 2016 #1

On 03/10/16 11:58, David Woodhouse wrote: > On Thu, 2016-03-10 at 11:33 +0100, Laszlo Ersek wrote: >> >>> * Considering tying the Bugzilla login to GitHub using GitHub as the >>> provider. This would mean that anyone wishing to submit an item into >>> BZ would require a GitHub account. >> >> I

[edk2] [PATCH] CryptoPkg/OpensslLib: Convert saved opensslconf.h to DOS line endings

Until we fix the git repository to store line endings properly and then just check them out in the appropriate form for the platform, let's make process_files.sh convert the opensslconf.h to DOS line endings when it creates it. Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] [PATCH v3] NetworkPkg: Add URI configuration form to HTTP boot driver.

On 03/10/16 11:02, Gary Lin wrote: > On Thu, Mar 10, 2016 at 10:20:12AM +0100, Laszlo Ersek wrote: >> On 03/10/16 08:49, Gary Lin wrote: >>> I found that it's related to iPXE. If I disable iPXE with >>> >>> "-netdev user,id=hostnet0 -device virtio-net-pci,romfile=,netdev=hostnet0" >>> >>> then

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

On Wed, 2016-02-24 at 16:15 +0800, Jiaxin Wu wrote: > diff --git a/CryptoPkg/Library/OpensslLib/OpensslTlsLib.uni > b/CryptoPkg/Library/OpensslLib/OpensslTlsLib.uni > new file mode 100644 > index > ..384f0245db8d1d3a1d758f6db58f85f5fc155211 > GIT binary

Re: [edk2] Tianocore Community Update 2016 #1

On Thu, 2016-03-10 at 11:33 +0100, Laszlo Ersek wrote: > > > * Considering tying the Bugzilla login to GitHub using GitHub as the > > provider.  This would mean that anyone wishing to submit an item into > > BZ would require a GitHub account. > > I vote against this. I find 3rd party

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

On Wed, 2016-02-24 at 16:15 +0800, Jiaxin Wu wrote: > diff --git a/CryptoPkg/Library/BaseCryptLib/SysCall/CrtWrapper.c > b/CryptoPkg/Library/BaseCryptLib/SysCall/CrtWrapper.c > index c0ccc0e..e68bfb8 100644 > --- a/CryptoPkg/Library/BaseCryptLib/SysCall/CrtWrapper.c > +++

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

On Wed, 2016-02-24 at 16:15 +0800, Jiaxin Wu wrote: > diff --git a/CryptoPkg/Library/OpensslLib/OpensslTlsLib.inf > b/CryptoPkg/Library/OpensslLib/OpensslTlsLib.inf > new file mode 100644 > index 000..fbd2b08 > --- /dev/null > +++ b/CryptoPkg/Library/OpensslLib/OpensslTlsLib.inf > @@ -0,0

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

On Wed, 2016-02-24 at 16:15 +0800, Jiaxin Wu wrote: > > diff --git a/CryptoPkg/Library/OpensslLib/EDKII_openssl-1.0.2f.patch > b/CryptoPkg/Library/OpensslLib/EDKII_openssl-1.0.2f.patch > index c42b776..f2d8f1a 100644 > --- a/CryptoPkg/Library/OpensslLib/EDKII_openssl-1.0.2f.patch > +++

Re: [edk2] Tianocore Community Update 2016 #1

On 03/09/16 19:49, Mangefeste, Tony wrote: > I. Defect & Issue Tracking > There are several topics we're investigating, and your input is appreciated: First of all, thank you very much for going forward with Bugzilla! > * Considering tying the Bugzilla login to GitHub using GitHub as the >

Re: [edk2] [PATCH v3] NetworkPkg: Add URI configuration form to HTTP boot driver.

On Thu, Mar 10, 2016 at 10:20:12AM +0100, Laszlo Ersek wrote: > On 03/10/16 08:49, Gary Lin wrote: > > On Wed, Mar 09, 2016 at 04:54:08PM +0800, Gary Lin wrote: > >> On Fri, Mar 04, 2016 at 04:38:22PM +0800, Fu Siyuan wrote: > >>> This patch updates the HTTP boot driver to produce a setup page for

Re: [edk2] EDK2 Staging Repository

On 03/10/16 00:02, Kinney, Michael D wrote: > Jordan, > > Responses included below. > > Mike > > > >> -Original Message- >> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >> Jordan Justen >> Sent: Wednesday, March 9, 2016 1:47 PM >> To: Mangefeste, Tony

Re: [edk2] [PATCH v3] NetworkPkg: Add URI configuration form to HTTP boot driver.

On 03/10/16 08:49, Gary Lin wrote: > On Wed, Mar 09, 2016 at 04:54:08PM +0800, Gary Lin wrote: >> On Fri, Mar 04, 2016 at 04:38:22PM +0800, Fu Siyuan wrote: >>> This patch updates the HTTP boot driver to produce a setup page for the boot >>> file URI configuration. A new boot option will be

Re: [edk2] EDK2 Staging Repository

On Mi, 2016-03-09 at 16:57 -0700, Bruce Cran wrote: > On 3/9/16 2:59 PM, Jordan Justen wrote: > > > So far I have a setup that can test building BaseTools and OVMF on > > Linux. The Windows setup is a little more complicated, but I don't > > think it should be too difficult. > > I guess that's