Re: [edk2] Why is USB_BOOT_IO_BLOCKS set to 128?

2017-01-18 Thread Heyi Guo
Hi Feng, This is a BMC emulated USB CDROM on a server platform, so there is no model or serial number for it. For USB CDROM, we found that the block size is 2KB and then the transfer length will be 256KB (2KB * 128). Is this what we expect? Thanks and regards, Heyi 在 1/19/2017 9:00 AM, T

Re: [edk2] [PATCH 3/3] OvmfPkg: Allow HTTP connections if HTTP Boot enabled

2017-01-18 Thread Gary Lin
On Thu, Jan 19, 2017 at 01:18:03PM +0800, Jiaxin Wu wrote: > Overwrite the value of PcdAllowHttpConnections to allow HTTP > connections if HTTP Boot enabled (-D HTTP_BOOT_ENABLE). > When setting PcdAllowHttpConnections to FALSE, I got this with OVMF: HttpBootCheckUriScheme: HTTP is disabled. Http

Re: [edk2] [PATCH V2 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

2017-01-18 Thread Bhupesh SHARMA
Hi Leif, Thanks for the review. On Mon, Jan 16, 2017 at 10:50 PM, Leif Lindholm wrote: > On Tue, Dec 27, 2016 at 02:45:42AM +0530, Bhupesh Sharma wrote: >> ARM TZASC-380 IP provides a mechanism to split memory regions being >> protected via it into eight equal-sized sub-regions. A bit-setting >>

[edk2] [PATCH 3/3] OvmfPkg: Allow HTTP connections if HTTP Boot enabled

2017-01-18 Thread Jiaxin Wu
Overwrite the value of PcdAllowHttpConnections to allow HTTP connections if HTTP Boot enabled (-D HTTP_BOOT_ENABLE). Cc: Laszlo Ersek Cc: Justen Jordan L Cc: Gary Lin Cc: Ye Ting Cc: Fu Siyuan Cc: Ruiyu Ni Cc: Kinney Michael D Contributed-under: TianoCore Contribution Agreement 1.0 Signed-o

[edk2] [PATCH v3 0/3] Enable the HTTP connections switch

2017-01-18 Thread Jiaxin Wu
v3: * Append patch for OVMF. Cc: Laszlo Ersek Cc: Justen Jordan L Cc: Gary Lin Cc: Ye Ting Cc: Fu Siyuan Cc: Kinney Michael D Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Wu Jiaxin Jiaxin Wu (3): NetworkPkg: Add PCD to enable the HTTP connections switch Nt32P

[edk2] [PATCH v3 2/3] Nt32Pkg.dsc: Add flag to control HTTP connections

2017-01-18 Thread Jiaxin Wu
v3: * Correct the commits grammar v2: * Rename the flag. This flag is to overwrite the value of PcdAllowHttpConnections, then the platform can make a decision whether to allow HTTP connections or not. Cc: Ye Ting Cc: Fu Siyuan Cc: Ruiyu Ni Cc: Laszlo Ersek Cc: Kinney Michael D Contributed-u

[edk2] [PATCH v3 1/3] NetworkPkg: Add PCD to enable the HTTP connections switch

2017-01-18 Thread Jiaxin Wu
v3: * Correct the commits grammar v2: * Rename the PCD to PcdAllowHttpConnections. * Refine the PCD descriptions. If the value of PcdAllowHttpConnections is TRUE, HTTP connections are allowed. Both the "https://"; and "http://"; URI schemes are permitted. Otherwise, HTTP connections are denied. O

[edk2] [PATCH V2 3/3] MdeModulePkg: Variable: Update PCR[7] measure for new TCG spec

2017-01-18 Thread Zhang, Chao B
Measure DBT into PCR[7] when it is updated between initial measure and ExitBootService. Measure "SecureBoot" change after PK update. Spec version : TCG PC Client PFP 00.37. http://www.trustedcomputinggroup.org/wp-content/uploads/PC-ClientSpecific_Platform_Profile_for_TPM_2p0_Systems_v21.pdf Cc: S

[edk2] [PATCH V2 2/3] SecurityPkg: Tcg2Dxe: Measure DBT into PCR[7]

2017-01-18 Thread Zhang, Chao B
Measure DBT into PCR[7] in initial measurement phase according to TCG PC Client PFP 00.37. http://www.trustedcomputinggroup.org/wp-content/uploads/PC-ClientSpecific_Platform_Profile_for_TPM_2p0_Systems_v21.pdf Cc: Star Zeng Cc: Yao Jiewen Contributed-under: TianoCore Contribution Agreement 1.0

[edk2] [PATCH V2 1/3] SecurityPkg: DxeImageVerificationLib: Update PCR[7] measure logic

2017-01-18 Thread Zhang, Chao B
Update PCR[7] measure logic according to TCG PC Client PFP 00.37. Only entries in DB that is used for image authentication need to be measured. http://www.trustedcomputinggroup.org/wp-content/uploads/PC-ClientSpecific_Platform_Profile_for_TPM_2p0_Systems_v21.pdf Cc: Star Zeng Cc: Yao Jiewen Cont

[edk2] [PATCH 3/3] MdeModulePkg: Variable: Update PCR[7] measure for new TCG spec

2017-01-18 Thread Zhang, Chao B
Measure DBX into PCR[7] when it is updated between initial measure and ExitBootService. Measure "SecureBoot" change after PK update. Spec version : TCG PC Client PFP 00.37. http://www.trustedcomputinggroup.org/wp-content/uploads/PC-ClientSpecific_Platform_Profile_for_TPM_2p0_Systems_v21.pdf Cc: S

[edk2] [PATCH 2/3] SecurityPkg: Tcg2Dxe: Measure DBX into PCR[7]

2017-01-18 Thread Zhang, Chao B
Measure DBX into PCR[7] in initial measurement phase according to TCG PC Client PFP 00.37. http://www.trustedcomputinggroup.org/wp-content/uploads/PC-ClientSpecific_Platform_Profile_for_TPM_2p0_Systems_v21.pdf Cc: Star Zeng Cc: Yao Jiewen Contributed-under: TianoCore Contribution Agreement 1.0

[edk2] [PATCH 1/3] SecurityPkg: DxeImageVerificationLib: Update PCR[7] measure logic

2017-01-18 Thread Zhang, Chao B
Update PCR[7] measure logic according to TCG PC Client PFP 00.37. Only entries in DB that is used for image authentication need to be measured. http://www.trustedcomputinggroup.org/wp-content/uploads/PC-ClientSpecific_Platform_Profile_for_TPM_2p0_Systems_v21.pdf Cc: Star Zeng Cc: Yao Jiewen Cont

Re: [edk2] [PATCH v2 2/2] Nt32Pkg.dsc: Add flag to control HTTP connections

2017-01-18 Thread Wu, Jiaxin
> > >> If everyone agrees, then Jiaxin, can you please append a third patch for > > >> OvmfPkg, which sets PcdAllowHttpConnections to TRUE whenever > > >> HTTP_BOOT_ENABLE is TRUE? > > >> > > > > > > Laszlo, > > > > > > As I talked above and according your requirement, we have the below > update ch

Re: [edk2] [PATCH 2/3] OvmfPkg: correct the set of modules included for the IPv6 stack

2017-01-18 Thread Wu, Jiaxin
> > > Laszlo, > > > > > > I also agree with (a). > > > > > > For IpSec, we can do the below update later: > > > > > > 1), Include it under NETWORK_IP6_ENABLE directly but with the limit usage > for IPv4. > > > > > > Or > > > > > > 2), Define new flag "IPSEC_ENABLE" for both of them. > > > > > > I p

Re: [edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

2017-01-18 Thread Zeng, Star
Reviewed-by: Star Zeng I can help push the patch at tomorrow if no other comments are received. Thanks, Star -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Thomas Huth Sent: Wednesday, January 18, 2017 9:40 PM To: Ni, Ruiyu ; edk2-de...@ml01.01

Re: [edk2] Why is USB_BOOT_IO_BLOCKS set to 128?

2017-01-18 Thread Tian, Feng
Hi, Heyi Just I said before, it's just an experience value. The transfer length limitation only comes from real h/w and the largest transfer length shipped by SCSI Read10/Write10 cmd. A SCSI Read10/Write10 cmd supports max 0x blocks. But according to our experience, some h/w devices don't

Re: [edk2] [PATCH] ShellPkg SmbiosView: Add missing decoding of SlotType AGP8X

2017-01-18 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Zeng, Star > Sent: Monday, January 16, 2017 7:23 PM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Carsey, Jaben ; > Ni, Ruiyu > Subject: [PATCH] ShellPkg SmbiosView: Add missing decoding of SlotType AGP8X > > REF: https://bugzill

Re: [edk2] [PATCH] ShellPkg SmbiosView: Add missing decoding of SMBIOS spec 3.0.0

2017-01-18 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Zeng, Star > Sent: Monday, January 16, 2017 7:24 PM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Carsey, Jaben ; > Ni, Ruiyu > Subject: [PATCH] ShellPkg SmbiosView: Add missing decoding of SMBIOS spec > 3.0.0 > > REF: https://bu

[edk2] DSC macro expansion

2017-01-18 Thread Huber, Brandon
Hi All, There seems to be a discrepancy between the DSC spec and the EDKII implementation regarding what macros are allowed in !include statements within a DSC file. >From the spec (v1.26 section 2.2.5): Macros, defined in this file, are not permitted in the path or file name of the !include

[edk2] [PATCH] ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe: Fixed crash on Juno R0

2017-01-18 Thread Daniil Egranov
The Marvell Yukon MAC address load supported only on Juno R1 and R2. It disabled for Juno R0 due to PCI issues on this board. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Daniil Egranov --- ArmPlatformPkg/ArmJunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 9 +++-- 1 file

Re: [edk2] [PATCH] ArmPlatformPkg/ArmVExpressPkg: Fix memory attributes for NOR Flash

2017-01-18 Thread Supreeth Venkatesh
On Wed, 2017-01-18 at 22:05 +, Leif Lindholm wrote: > Hi Achin, > > On Wed, Jan 18, 2017 at 08:24:06PM +, achin.gu...@arm.com wrote: > > > > From: Achin Gupta > > > > The NOR flash banks were being mapped in the translation tables > > with the same > > memory attributes as RAM in the sy

Re: [edk2] [PATCH] ArmPlatformPkg/ArmVExpressPkg: Fix memory attributes for NOR Flash

2017-01-18 Thread Leif Lindholm
Hi Achin, On Wed, Jan 18, 2017 at 08:24:06PM +, achin.gu...@arm.com wrote: > From: Achin Gupta > > The NOR flash banks were being mapped in the translation tables with the same > memory attributes as RAM in the system. These attributes mark the region as > Normal Memory and could additionall

[edk2] Google Summer of Code 2017 - Requesting Input

2017-01-18 Thread Richardson, Brian
Mentor applications for the Google Summer of Code project start Thursday, Jan 19. I will submit projects on behalf of TianoCore, so I would like developer input on proposals. We would need to propose specific projects, and be prepared to mentor students working on those projects. Student Applic

Re: [edk2] [patch] ShellPkg/HandleParsingLib: Fix coding style issue

2017-01-18 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Bi, Dandan > Sent: Sunday, January 15, 2017 6:04 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu > Subject: [patch] ShellPkg/HandleParsingLib: Fix coding style issue > Importance: High > > Cc: Jaben Carsey > Cc: Ru

Re: [edk2] [PATCH 0/3] Add SMBIOS spec 3.1.0 support

2017-01-18 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey For the ShellPkg content. > -Original Message- > From: Zeng, Star > Sent: Monday, January 16, 2017 7:25 PM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Gao, Liming ; > Kinney, Michael D ; Tian, Feng > ; Ni, Ruiyu ; Carsey, Jaben > > Subject: [PATCH 0/3] Add

Re: [edk2] [PATCH] ArmPlatformPkg/ArmVExpressPkg: Fix memory attributes for NOR Flash

2017-01-18 Thread Ard Biesheuvel
On 18 January 2017 at 20:24, wrote: > From: Achin Gupta > > The NOR flash banks were being mapped in the translation tables with the same > memory attributes as RAM in the system. These attributes mark the region as > Normal Memory and could additionally be cacheable or non-cacheable. > > Either

[edk2] [PATCH] ArmPlatformPkg/ArmVExpressPkg: Fix memory attributes for NOR Flash

2017-01-18 Thread achin.gupta
From: Achin Gupta The NOR flash banks were being mapped in the translation tables with the same memory attributes as RAM in the system. These attributes mark the region as Normal Memory and could additionally be cacheable or non-cacheable. Either type of attributes are unsuitable for NOR Flash s

[edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

2017-01-18 Thread Thomas Huth
Correct the reported by the codespell utility in some files of PcAtChipsetPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth --- PcAtChipsetPkg/8254TimerDxe/Timer.c | 6 +++--- PcAtChipsetPkg/8254TimerDxe/Timer.h | 2

[edk2] [Patch][edk2-platforms/devel-MinnowBoard3 2/2] Restore P2SB MMIO

2017-01-18 Thread zwei4
Restore P2SB's, as well as GPIO controllers', MMIO resources to original values which have been overridden by PCI bus driver. Signed-off-by: zwei4 Cc: Mike Wu Cc: Benjamin You Cc: Mang Guo Cc: Shifei Lu Cc: Ruth Li Cc: Junjie He --- .../BroxtonSoC/BroxtonSiPkg/SouthCluster/ScInit/Dxe/ScI

Re: [edk2] [PATCH v2 2/2] Nt32Pkg.dsc: Add flag to control HTTP connections

2017-01-18 Thread Gary Lin
On Wed, Jan 18, 2017 at 09:30:52AM +0100, Laszlo Ersek wrote: > On 01/18/17 03:16, Wu, Jiaxin wrote: > >> Subject: Re: [edk2] [PATCH v2 2/2] Nt32Pkg.dsc: Add flag to control HTTP > >> connections > >> > >> CC Jordan and Gary > >> > >> On 01/17/17 04:33, Jiaxin Wu wrote: > >>> v2: > >>> * Rename the

Re: [edk2] [PATCH 2/3] OvmfPkg: correct the set of modules included for the IPv6 stack

2017-01-18 Thread Gary Lin
On Wed, Jan 18, 2017 at 09:17:28AM +0100, Laszlo Ersek wrote: > On 01/18/17 01:47, Wu, Jiaxin wrote: > >> Subject: Re: [edk2] [PATCH 2/3] OvmfPkg: correct the set of modules > >> included > >> for the IPv6 stack > >> > >> On 01/17/17 05:52, Gary Lin wrote: > >>> Always use IScsiDxe from NetworkPkg

Re: [edk2] [PATCH 0/3] Enable HTTPS Boot in OVMF

2017-01-18 Thread Gary Lin
On Tue, Jan 17, 2017 at 09:13:23PM +0100, Laszlo Ersek wrote: > On 01/17/17 09:25, Jordan Justen wrote: > > On 2017-01-16 20:52:29, Gary Lin wrote: > >> This series adjusts the build files of OVMF in order to support HTTPS. > >> It introdues a new build option, TLS_ENABLE, and makes OpenSSL a hard

Re: [edk2] SCT 2.3.1 v1.3

2017-01-18 Thread Laszlo Ersek
(Dropping the and email addresses; please never cross-post the public edk2-devel list with the confidential spec development / working group lists on UEFI.org!) On 01/17/17 22:22, Daniel Samuelraj wrote: > Hi, > > What SCT v1.3 states for HII Config Access Protocol seems not in align > with UEFI

Re: [edk2] [PATCH v2 2/2] Nt32Pkg.dsc: Add flag to control HTTP connections

2017-01-18 Thread Laszlo Ersek
On 01/18/17 03:16, Wu, Jiaxin wrote: >> Subject: Re: [edk2] [PATCH v2 2/2] Nt32Pkg.dsc: Add flag to control HTTP >> connections >> >> CC Jordan and Gary >> >> On 01/17/17 04:33, Jiaxin Wu wrote: >>> v2: >>> * Rename the flag. >>> >>> This flag is used to overwrite the PcdAllowHttpConnections >>> va

Re: [edk2] [PATCH 2/3] OvmfPkg: correct the set of modules included for the IPv6 stack

2017-01-18 Thread Laszlo Ersek
On 01/18/17 01:47, Wu, Jiaxin wrote: >> Subject: Re: [edk2] [PATCH 2/3] OvmfPkg: correct the set of modules included >> for the IPv6 stack >> >> On 01/17/17 05:52, Gary Lin wrote: >>> Always use IScsiDxe from NetworkPkg when IPv6 is enabled since it provides >>> the complete ISCSI support. >>> >>>