[edk2] [PATCH v2 16/22] DynamicTablesPkg: Arm ACPI FADT Generator

2019-01-24 Thread Sami Mujawar
The FADT generator collates the relevant information required for generating a FADT table from configuration manager using the configuration manager protocol. It then updates a template FADT table structure. This table data is used by the Table Manager to install the FADT table.

Re: [edk2] Network Stack Budgeting

2019-01-24 Thread Tomas Pilar (tpilar)
On 24/01/2019 16:49, Laszlo Ersek wrote: > On 01/24/19 14:25, Tomas Pilar (tpilar) wrote: >> Hmm, >> >> Mnp->Configure() will eventually call MnpStartSnp(). >> >> A grep for Mnp->Configure shows that: >> * ArpDxe performs this on DriverBinding.Start() >> * Ip6Dxe performs this on

[edk2] [PATCH] MdePkg/BasePeCoffLib: skip runtime relocation if relocation info is invalid.

2019-01-24 Thread Neo Hsueh
1.Skip runtime relocation for PE images that provide invalid relocation infomation (ex: RelocDir->Size = 0) to fix a hang observed while booting Windows. 2.Add a magic number check for PE32+ image. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Neo Hsueh Cc: Michael D

[edk2] [PATCH v2 11/22] DynamicTablesPkg: Table Helper Library

2019-01-24 Thread Sami Mujawar
A helper library that implements common functionality for use by table generators. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sami Mujawar --- DynamicTablesPkg/DynamicTables.dsc.inc| 20 +++

[edk2] [PATCH v2 12/22] DynamicTablesPkg: Dynamic Table Factory Protocol

2019-01-24 Thread Sami Mujawar
This patch introduces the dynamic table factory protocol that provides an interface to register and retrieve registered generators. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sami Mujawar --- DynamicTablesPkg/DynamicTablesPkg.dec | 3 +

[edk2] [PATCH v2 03/22] DynamicTablesPkg: Acpi Table Generator

2019-01-24 Thread Sami Mujawar
This patch introduces the required interfaces and definitions for implementing an ACPI table generator. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sami Mujawar --- DynamicTablesPkg/Include/AcpiTableGenerator.h | 360 1 file changed, 360

Re: [edk2] Network Stack Budgeting

2019-01-24 Thread Laszlo Ersek
On 01/24/19 14:25, Tomas Pilar (tpilar) wrote: > Hmm, > > Mnp->Configure() will eventually call MnpStartSnp(). > > A grep for Mnp->Configure shows that: > * ArpDxe performs this on DriverBinding.Start() > * Ip6Dxe performs this on DriverBinding.Start() > > Ipv4 and DnsDhcp do this as a part of

Re: [edk2] [question] A piece of log.txt showed in uefi shell is not the same as showed in notepad++

2019-01-24 Thread Andrew Fish via edk2-devel
Krishna, From an EFI point of view BackSpace 0x08, LF 0x0A, and CR 0xD are defined for a ConOut device. From Ascii Control Codes: 0x0C - Form Feed 0x0E - Shift Out The Shift Out is used with graphics characters. How did you dump the log? If EFI sends data to a serial terminal then it would

[edk2] [PATCH v2 17/22] DynamicTablesPkg: Arm ACPI MADT Generator

2019-01-24 Thread Sami Mujawar
The MADT generator uses the configuration manager protocol to obtain information about the Arm interrupt controllers (GICC, GICD, etc.) and generates the ACPI MADT table. This table data is then used by the Table Manager to install the MADT table. The Table Manager then invokes the generator

Re: [edk2] Text attachments

2019-01-24 Thread Laszlo Ersek
On 01/24/19 12:38, Tomas Pilar (tpilar) wrote: > Hi, > > Does the mailing list support plaintext attachments? No, it doesn't forward any attachments. People that you CC directly will get the attachment, but the list will not reflect any attachments to subscribers, nor will the list archive carry

[edk2] [PATCH v2 10/22] DynamicTablesPkg: Configuration Manager Helper

2019-01-24 Thread Sami Mujawar
This patch defines a helper macro 'GET_OBJECT_LIST()' that expands to a function that uses the configuration manager protocol to retrieve configuration manager object(s). Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sami Mujawar ---

Re: [edk2] [PATCH] MdeModulePkg VariableStandaloneMm: Add PcdEmuVariableNvModeEnable in inf

2019-01-24 Thread Bi, Dandan
Reviewed-by: Bi Dandan Thanks, Dandan > -Original Message- > From: Zeng, Star > Sent: Friday, January 25, 2019 9:23 AM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Wang, Jian J ; > Wu, Hao A ; Ard Biesheuvel > ; Bi, Dandan > Subject: [PATCH] MdeModulePkg VariableStandaloneMm: Add >

Re: [edk2] [question] A piece of log.txt showed in uefi shell is not the same as showed in notepad++

2019-01-24 Thread krishnaLee
Andrew Fish, Yes,I know it ,from a terminal's view,the 0x0C/0x0D/0x0E/...,some invisible chars like this have special means, and I also think from a text-editor's view, I think the text-editor just do not show them,and should make sure all visible chars can visible, but I think I am wrong. >

Re: [edk2] [PATCH] MdeModulePkg VariableStandaloneMm: Add PcdEmuVariableNvModeEnable in inf

2019-01-24 Thread Zeng, Star
Thanks. I just pushed it at 9084239f3440369fcb5ff0476d9bdd9442a03258 since it is a build failure. Star -Original Message- From: Wang, Jian J Sent: Friday, January 25, 2019 9:42 AM To: Zeng, Star ; edk2-devel@lists.01.org Cc: Wu, Hao A ; Ard Biesheuvel ; Bi, Dandan Subject: RE:

Re: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support

2019-01-24 Thread Wu, Hao A
> -Original Message- > From: Chen, Chen A > Sent: Friday, January 25, 2019 3:45 PM > To: Wu, Hao A; Gao, Liming; edk2-devel@lists.01.org > Cc: Kinney, Michael D; Zhang, Chao B > Subject: RE: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for > enable GPT support > > > >

Re: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support

2019-01-24 Thread Chen, Chen A
-Original Message- From: Wu, Hao A Sent: Friday, January 25, 2019 3:27 PM To: Chen, Chen A ; Gao, Liming ; edk2-devel@lists.01.org Cc: Kinney, Michael D ; Zhang, Chao B Subject: RE: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support > -Original

[edk2] [PATCH] MdeModulePkg VariableStandaloneMm: Add PcdEmuVariableNvModeEnable in inf

2019-01-24 Thread Star Zeng
It was missed in 7cd69959463ac9c761163ed8e8a93907b68e70da when rebasing the patches after 688b2cad7b712493f2cf8b6948ab795545e13961 added VariableStandaloneMm. Cc: Jian J Wang Cc: Hao Wu Cc: Ard Biesheuvel Cc: Dandan Bi Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by:

Re: [edk2] [PATCH v2] MinPlatformPkg: Support TCO base locked by FSP

2019-01-24 Thread Kubacki, Michael A
Reviewed-by: Michael Kubacki > -Original Message- > From: Chiu, Chasel > Sent: Wednesday, January 16, 2019 11:16 PM > To: edk2-devel@lists.01.org > Cc: Kubacki, Michael A ; Yao, Jiewen > ; Chiu, Chasel > Subject: [PATCH v2] MinPlatformPkg: Support TCO base locked by FSP > > REF:

Re: [edk2] [PATCH] MdeModulePkg VariableStandaloneMm: Add PcdEmuVariableNvModeEnable in inf

2019-01-24 Thread Wang, Jian J
Reviewed-by: Jian J Wang > -Original Message- > From: Zeng, Star > Sent: Friday, January 25, 2019 9:23 AM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Wang, Jian J ; > Wu, Hao A ; Ard Biesheuvel ; > Bi, Dandan > Subject: [PATCH] MdeModulePkg VariableStandaloneMm: Add >

Re: [edk2] [PATCH] IntelFsp2Pkg: Add function to get bootloader stack pointer

2019-01-24 Thread Zeng, Star
On 2019/1/23 13:36, Chasel, Chiu wrote: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1391 FSP on-going enhancement for stack utilization required to know bootloader stack pointer and this pointer can be retrieved by first input parameter address when FSP-M entry API invoked by

Re: [edk2] [PATCH 1/3] MdeModulePkg/CapsuleApp: Refine code logic of parsing parameter.

2019-01-24 Thread Yao, Jiewen
Hey I don't think MdeModulePkg can depend on ShellPkg. That is why we do not use ShellLib in the first version. Do we change the package dependency rule? Thank you Yao Jiewen > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Chen A Chen >

Re: [edk2] [PATCH 1/3] MdeModulePkg/CapsuleApp: Refine code logic of parsing parameter.

2019-01-24 Thread Zhang, Chao B
Jiewen & Liming: It is a problem. Unlike UiApp. CapsuleApp is supposed to run in Shell. ShellLib provides standard parameter parsing support. Any suggestion on this? From: Yao, Jiewen Sent: Friday, January 25, 2019 2:25 PM To: Chen, Chen A ; edk2-devel@lists.01.org Cc: Wu, Hao A ; Zhang,

Re: [edk2] [PATCH] ShellPkg/TftpDynamicCommand: Return proper status

2019-01-24 Thread Li, Songpeng
Reviewed-by: Songpeng Li Sorry, the previous email did not include edk2 mailing list. Best, Songpeng > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Vladimir Olovyannikov via edk2-devel > Sent: Thursday, January 24, 2019 2:26 AM > To:

Re: [edk2] [PATCH 1/2] FatPkg: Break down Part.c file.

2019-01-24 Thread Wu, Hao A
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Chen A Chen > Sent: Thursday, January 17, 2019 10:03 AM > To: edk2-devel@lists.01.org > Cc: Ni, Ray; Zhang, Chao B > Subject: [edk2] [PATCH 1/2] FatPkg: Break down Part.c file. > > BZ:

[edk2] [PATCH 3/3] MdeModulePkg/CapsuleApp: Enhance CapsuleApp to support Capsule-on-Disk.

2019-01-24 Thread Chen A Chen
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1482 CapsuleApp is used for trigger capsule update. Add -OD option in CapsuleApp to support doing capsule update via storage. Add -F and -L options to support dumping information feature. Cc: Jian J Wang Cc: Hao Wu Cc: Zhang Chao B

[edk2] [PATCH 2/3] MdePkg/UefiSpec.h: Add definition to support Capsule-on-Disk feature.

2019-01-24 Thread Chen A Chen
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1482 UEFI Spec define this definition to support Capsule-on-Disk. Cc: Michael D Kinney Cc: Liming Gao Cc: Zhang Chao B Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chen A Chen --- MdePkg/Include/Uefi/UefiSpec.h | 5

[edk2] [PATCH 1/3] MdeModulePkg/CapsuleApp: Refine code logic of parsing parameter.

2019-01-24 Thread Chen A Chen
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1482 No change functionality, use ShellLib to parsing command line. Cc: Jian J Wang Cc: Hao Wu Cc: Zhang Chao B Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chen A Chen ---

[edk2] [PATCH edk2-platforms] Platform/ARM/SgiPkg: Include HobLib, MmServicesTableLib and MemoryAllocationLib

2019-01-24 Thread Jagadeesh Ujja
Include the HobLib, MmServicesTableLib and MemoryAllocationLib libraries on Sgi platforms. These will be consumed by MM_STANDALONE drivers. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jagadeesh Ujja --- Platform/ARM/SgiPkg/PlatformStandaloneMm.dsc | 5 + 1 file

Re: [edk2] [PATCH] IntelFsp2Pkg: FSP can utilize bootloader stack

2019-01-24 Thread Zeng, Star
On 2019/1/23 17:33, Chasel, Chiu wrote: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1485 Current FSP utilizes pre-allocated temporary memory from boot loader for both heap and stack. To reduce overall temporary memory usage FSP may share the same stack with boot loader and only needs a

Re: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support

2019-01-24 Thread Wu, Hao A
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Chen A Chen > Sent: Thursday, January 17, 2019 10:03 AM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D; Zhang, Chao B; Gao, Liming > Subject: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new

[edk2] [Patch 08/33] BaseTools: replace long by int

2019-01-24 Thread Feng, Bob C
From: Yunhua Feng replace long by int Because the long() was not exist in Python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng --- BaseTools/Source/Python/AutoGen/ValidCheckingInfoObject.py | 4 ++--

[edk2] [Patch 09/33] BaseTools:Solve the data sorting problem use python3

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan set PYTHONHASHSEED Specifying the value 0 will disable hash randomization. Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/toolsetup.bat | 1 + edksetup.sh | 2 +- 2

[edk2] [Patch 07/33] BaseTools: Remove unnecessary super function

2019-01-24 Thread Feng, Bob C
From: Yunhua Feng Remove unnecessary super function Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng --- BaseTools/Source/Python/AutoGen/AutoGen.py | 5 - 1 file changed, 5 deletions(-) diff --git

[edk2] [Patch 03/33] BaseTools: Rename iteritems to items

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan replace the list iteritems by items in Python3. Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/GenMake.py| 6 +++---

[edk2] [Patch 11/33] BaseTools:Similar to octal data rectification

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan In python3, if Value is octal data, the int(Value, 0) report an error Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/GenC.py | 2 ++

[edk2] [Patch 12/33] BaseTools/UPT:merge UPT Tool use Python2 and Python3

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan In UPT Tool,merge python2 and python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/UPT/Core/IpiDb.py | 4 ++--

[edk2] [Patch 01/33] BaseTool:Rename xrange() to range()

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan Because the xrange() was not exist in Python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/GenPcdDb.py | 22 +++---

[edk2] [Patch 10/33] BaseTools: Update argparse arguments since it not have version now

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan argparse.ArgumentParser not have version parameter Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/Pkcs7Sign/Pkcs7Sign.py | 3 ++-

[edk2] [Patch 13/33] BaseTools: update Test scripts support python3

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan update Test scripts support python2 and python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Tests/CToolsTests.py | 2 +- BaseTools/Tests/CheckUnicodeSourceFiles.py

[edk2] [Patch 06/33] BaseTools: nametuple not have verbose parameter in python3

2019-01-24 Thread Feng, Bob C
From: Yunhua Feng nametuple not have verbose parameter in python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng --- BaseTools/Source/Python/Workspace/BuildClassObject.py | 2 +- 1 file changed, 1

[edk2] [Patch 05/33] BaseTools: use OrderedDict instead of sdict

2019-01-24 Thread Feng, Bob C
From: Yunhua Feng use OrderedDict instead of sdict, and delete sdict Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yunhua Feng --- BaseTools/Source/Python/Common/Misc.py | 127

[edk2] [Patch 02/33] BaseTools:use iterate list to replace the itertools

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan itertools.imap() replace map(), itertools.ifilter() replace filter Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/build/build.py | 6 ++ 1 file changed, 2

[edk2] [Patch 00/33] BaseTools python3 migration patch set

2019-01-24 Thread Feng, Bob C
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=55 This patch set is to enable python3 on BaseTools. Basetools code will be compatible with both python3 and python2. We will have two envs PYTHON3_ENABLE and PYTHON_COMMAND. The behavior can be combined as the below to support this usage. If

Re: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support

2019-01-24 Thread Chen, Chen A
-Original Message- From: Wu, Hao A Sent: Friday, January 25, 2019 11:04 AM To: Chen, Chen A ; edk2-devel@lists.01.org Cc: Kinney, Michael D ; Zhang, Chao B ; Gao, Liming Subject: RE: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support > -Original

Re: [edk2] [PATCH] IntelFsp2Pkg: Add function to get bootloader stack pointer

2019-01-24 Thread Desimone, Nathaniel L
Reviewed-by: Nate DeSimone -Original Message- From: Chiu, Chasel Sent: Tuesday, January 22, 2019 9:36 PM To: edk2-devel@lists.01.org Cc: Desimone, Nathaniel L ; Zeng, Star ; Chiu, Chasel Subject: [PATCH] IntelFsp2Pkg: Add function to get bootloader stack pointer REF:

[edk2] [Patch 31/33] BaseTools: Handle the bytes and str difference

2019-01-24 Thread Feng, Bob C
Deal with bytes and str is different, remove the unicode(), correct open file parameter. Using utcfromtimestamp instead of fromtimestamp. Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan ---

[edk2] [Patch 23/33] BaseTools: Similar to octal data rectification

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan In python3, if Value is octal data, the int(Value, 0) report an error Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/GenC.py | 2 +-

[edk2] [Patch 24/33] BaseTools: Update windows and linux run scripts file to use Python3

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan Modify windows script, PosixLike script, edksetup.sh, edksetup.bat to use Python3 based on PYTHON3_ENABLE environment. Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan ---

[edk2] [Patch 28/33] BaseTools:Fixed Rsa issue and a set define issue.

2019-01-24 Thread Feng, Bob C
ValueError: non-hexadecimal number found in fromhex() arg at position 0 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/Rsa2048Sha256Sign/Rsa2048Sha256Sign.py | 4 ++-- 1 file changed,

[edk2] [Patch 21/33] BaseTools: change the Division Operator

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan PEP 238 -- Changing the Division Operator x/y to return a reasonable approximation of the mathematical result of the division ("true division") x//y to return the floor ("floor division") Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore

[edk2] [Patch 25/33] BaseTools:Update build tool to print python version information

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan print PYTHON3_ENABLE and PYTHON_COMMAND in build tool Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/build/build.py | 6 ++ 1 file changed, 6 insertions(+) diff --git

[edk2] [Patch 29/33] BaseTools:ord() don't match in py2 and py3

2019-01-24 Thread Feng, Bob C
In python2, the FvHeaderBuffer Type is a str In python3, the FvHeaderBuffer Type is a bytes Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/GenFds/FvImageSection.py | 10 -- 1 file changed, 8

[edk2] [Patch 20/33] BaseTools:File open failed for VPD MapFile

2019-01-24 Thread Feng, Bob C
correct open MapFile support python2 and python3 Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/BPDG/GenVpd.py | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git

[edk2] [Patch 19/33] BaseTools:Double carriage return inserted from Trim.py on Python3

2019-01-24 Thread Feng, Bob C
From: Zhiju Fan https://bugzilla.tianocore.org/show_bug.cgi?id=1379 Line 208 of BaseTools/Source/Python/Trim/Trim.py uses 'NewLines.append(os.linesep)' to insert a new line into the list that will be written to the output file. This causes the '\r\n' inserted with os.linesep to be written as

[edk2] [Patch 27/33] BaseTools: Update PYTHON env to PYTHON_COMMAND

2019-01-24 Thread Feng, Bob C
From: Liming Gao Update PYTHON env to PYTHON_COMMAND. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Liming Gao --- BaseTools/Bin/CYGWIN_NT-5.1-i686/Ecc| 6 +++--- BaseTools/Bin/CYGWIN_NT-5.1-i686/GenDepex | 6 +++---

[edk2] [Patch 30/33] BaseTools: the list and iterator translation

2019-01-24 Thread Feng, Bob C
In python3,The keys of the dictionary not a list,It needs to be converted Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/AutoGen.py | 26 +-

[edk2] [Patch 18/33] BaseTools:TestTools character encoding issue

2019-01-24 Thread Feng, Bob C
From: Zhiju Fan Specifies encoding when opening a file using codecs Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Tests/TestTools.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[edk2] [Patch 26/33] BaseTools:Linux Python highest version check.

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan Linux Python highest version check. The path of Python interpreter assign values to PYTHON_COMMAND Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- edksetup.sh | 16 +++- 1 file changed, 11

[edk2] [Patch 17/33] BaseTools: Make sure AllPcdList valid.

2019-01-24 Thread Feng, Bob C
This patch is to make sure the AllPcdList is always evaluated. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng Cc: Liming Gao --- BaseTools/Source/Python/AutoGen/AutoGen.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

[edk2] [Patch 16/33] BaseTools:Fv BaseAddress must set If it not set

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan If ForceRebase is not set, and FV is specified in FD region, it should have FvBaseAddress Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/GenFds/FvImageSection.py | 2 ++

[edk2] [Patch 22/33] BaseTools:There is extra blank line in datalog

2019-01-24 Thread Feng, Bob C
From: Zhiju Fan There should be no blank line across every line in datalog if open it with Notepad++. Cc: Bob Feng Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/build/BuildReport.py | 2 +- 1 file changed, 1

[edk2] [Patch 14/33] BaseTools/Scripts: Porting PackageDocumentTools code to use Python3

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan Porting PackageDocumentTools code to support python2 and python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Scripts/ConvertFceToStructurePcd.py

[edk2] [Patch 04/33] BaseTools: replace get_bytes_le() to bytes_le

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan UUID does not have the get_bytes_le() in python3 Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/Common/Misc.py | 2 +-

[edk2] [Patch 15/33] Basetools: It went wrong when use os.linesep

2019-01-24 Thread Feng, Bob C
From: Zhijux Fan in python2 and python3,use of line breaks Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/AutoGen.py | 10 +-

[edk2] Do python3 supported in edk2 shell now?

2019-01-24 Thread wang xiaofeng
HI All, I only see AppPkg/Applications/Python/2.7.10 in latest edk2 tree. Do edk2 plan to support python3 in shell ? Thanks! ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for enable GPT support

2019-01-24 Thread Wu, Hao A
> -Original Message- > From: Chen, Chen A > Sent: Friday, January 25, 2019 3:16 PM > To: Wu, Hao A; edk2-devel@lists.01.org > Cc: Kinney, Michael D; Zhang, Chao B; Gao, Liming > Subject: RE: [edk2] [PATCH 2/2] MdePkg/UefiGpt.h: Add new definition for > enable GPT support > > > >

Re: [edk2] A VM failed to boot when I changed the gUefiOvmfPkgTokenSpaceGuid.PcdPciMmio64Size

2019-01-24 Thread Wuzongyong (Euler Dept)
> >> (1) Either way, if you'd like to check whether this issue is specific > >> to the iPXE option ROM, you could prevent QEMU from loading the ROM > >> image into the NIC's ROM BAR, and retest. The QEMU option for that is > >> > >> -device virtio-net-pci,[some properties,]romfile='' > >> > >>

[edk2] [PATCH v1] ShellPkg/TftpDynamicCommand: Fix return status error in RunTftp()

2019-01-24 Thread Songpeng Li
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1487 ShellStatus is set to SHELL_NOT_FOUND initially in RunTftp(), but there is no operation to clean this status after download successful. This patch is to fix this issue. Cc: Jaben Carsey Cc: Ruiyu Ni Cc: Wu Jiaxin Contributed-under:

Re: [edk2] parallelism in the module-level, generated GNUmakefile's

2019-01-24 Thread Gao, Liming
Yes. BaseTools should make sure the complete dependency in the generated Makefile. If you find any missing, please submit BZs. > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, January 24, 2019 6:07 AM > To: Gao, Liming > Cc: edk2-devel-01 >

Re: [edk2] A VM failed to boot when I changed the gUefiOvmfPkgTokenSpaceGuid.PcdPciMmio64Size

2019-01-24 Thread Laszlo Ersek
On 01/24/19 03:35, Wuzongyong (Euler Dept) wrote: >> (1) Either way, if you'd like to check whether this issue is specific >> to the iPXE option ROM, you could prevent QEMU from loading the ROM >> image into the NIC's ROM BAR, and retest. The QEMU option for that is >> >> -device

Re: [edk2] Drop CSM support in OvmfPkg?

2019-01-24 Thread David Woodhouse
On Thu, 2019-01-24 at 01:48 +, Ni, Ray wrote: > David, > I think we got an agreement here to move CSM components in OvmfPkg. > I prefer we firstly clone the required CSM components in OvmfPkg right no. > Finally I can remove the IntelFrameworkModulePkg/IntelFrameworkPkg in one > patch. > (I

[edk2] [PATCH v2] SD/eMMC : Fix Command Argument for SD/eMMC R/W operation.

2019-01-24 Thread Meenakshi Aggarwal
Issue : SD read failure for high capacity cards e.g. 64 GB i Reason : Command argument value exceeds 32 bit for block number 0x3787FFF and cant be fit into 32 bit wide SD host controller register. Fix : AccessMode bits [29:30] of OCR is a valid definition to calculate data address for eMMC cards.

Re: [edk2] Network Stack Budgeting

2019-01-24 Thread Laszlo Ersek
On 01/24/19 12:37, Tomas Pilar (tpilar) wrote: > > Hi Laszlo, > >> Can you capture a call stack when Snp.Start() is invoked for the very >> first time (which, IIUC, is a call that should not happen, in your >> opinion)? >> > Unfortunately I do not have access to the platform firmware itself (I

Re: [edk2] Network Stack Budgeting

2019-01-24 Thread Tomas Pilar (tpilar)
Hi Laszlo, > Can you capture a call stack when Snp.Start() is invoked for the very > first time (which, IIUC, is a call that should not happen, in your > opinion)? > Unfortunately I do not have access to the platform firmware itself (I maintain an IHV network driver that's shipped in

[edk2] Text attachments

2019-01-24 Thread Tomas Pilar (tpilar)
Hi, Does the mailing list support plaintext attachments? Cheers, Tom ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] parallelism in the module-level, generated GNUmakefile's

2019-01-24 Thread Laszlo Ersek
On 01/24/19 10:57, Gao, Liming wrote: > Yes. BaseTools should make sure the complete dependency in the generated > Makefile. If you find any missing, please submit BZs. Awesome. That's a huge help. Thank you Liming! Laszlo ___ edk2-devel mailing list

Re: [edk2] UEFI Shell + startup.nsh

2019-01-24 Thread Rafael Machado
Thanks a lot for the help Laszlo! Will take a look. (Also agree about having a signed UEFI Shell not being a good idea.) Best Regards Rafael Em qui, 24 de jan de 2019 às 10:47, Laszlo Ersek escreveu: > On 01/24/19 13:22, Rafael Machado wrote: > > Hi everyone. > > > > I have a question. > >

Re: [edk2] Network Stack Budgeting

2019-01-24 Thread Tomas Pilar (tpilar)
> This reeks. :) > > It looks like some driver in the platform sets up a protocol notify > callback for SNP, with gBS->CreateEvent() + > gBS->RegisterProtocolNotify(). Your driver's DriverBindingStart() > function is called normally from BDS, via ConnectController(). In > DriverBindingStart(),

Re: [edk2] UEFI Shell + startup.nsh

2019-01-24 Thread Paulo Alcantara
Rafael Machado writes: > I have a question. > Considering I have a PXE server that my client downloads a shell.efi app. > Considering also that I need to execute a .nsh script, but I their is no > media at the system. (no usb or storage device attached) > > Is there any way to embed a

Re: [edk2] UEFI Shell + startup.nsh

2019-01-24 Thread Laszlo Ersek
On 01/24/19 13:22, Rafael Machado wrote: > Hi everyone. > > I have a question. > Considering I have a PXE server that my client downloads a shell.efi app. > Considering also that I need to execute a .nsh script, but I their is no > media at the system. (no usb or storage device attached) > >

Re: [edk2] Drop CSM support in OvmfPkg?

2019-01-24 Thread Laszlo Ersek
On 01/24/19 10:31, David Woodhouse wrote: > On Thu, 2019-01-24 at 01:48 +, Ni, Ray wrote: >> David, >> I think we got an agreement here to move CSM components in OvmfPkg. >> I prefer we firstly clone the required CSM components in OvmfPkg right no. >> Finally I can remove the

Re: [edk2] A VM failed to boot when I changed the gUefiOvmfPkgTokenSpaceGuid.PcdPciMmio64Size

2019-01-24 Thread Laszlo Ersek
On 01/24/19 10:25, Wuzongyong (Euler Dept) wrote: (1) Either way, if you'd like to check whether this issue is specific to the iPXE option ROM, you could prevent QEMU from loading the ROM image into the NIC's ROM BAR, and retest. The QEMU option for that is -device

[edk2] UEFI Shell + startup.nsh

2019-01-24 Thread Rafael Machado
Hi everyone. I have a question. Considering I have a PXE server that my client downloads a shell.efi app. Considering also that I need to execute a .nsh script, but I their is no media at the system. (no usb or storage device attached) Is there any way to embed a startup.nsh at the shell.efi

Re: [edk2] Network Stack Budgeting

2019-01-24 Thread Tomas Pilar (tpilar)
Hmm, Mnp->Configure() will eventually call MnpStartSnp(). A grep for Mnp->Configure shows that: * ArpDxe performs this on DriverBinding.Start() * Ip6Dxe performs this on DriverBinding.Start() Ipv4 and DnsDhcp do this as a part of their Configure() they expose in the API. Cheers, Tom

[edk2] [PATCH v2 08/22] DynamicTablesPkg: Configuration Manager Objects

2019-01-24 Thread Sami Mujawar
The dynamic tables frameworks core communicates with the platform specific implementation using the configuration manager protocol interface. The dynamic tables framework core uses this interface to retrieve information required for generating the firmware tables. This information is represented

[edk2] [PATCH v2 00/22] DynamicTablesPkg: Introduce Dynamic Tables Framework

2019-01-24 Thread Sami Mujawar
Dynamic Tables Framework aims to reduce the amount of effort required for porting firmware to new platforms by simplifying the generation of firmware tables based on hardware description provided by a platform specific component. The Dynamic Tables Framework core queries the platform specific

[edk2] [PATCH v2 04/22] DynamicTablesPkg: SMBIOS Table Generator

2019-01-24 Thread Sami Mujawar
This patch introduces the required interfaces and definitions for implementing a SMBIOS table generator. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Sami Mujawar --- DynamicTablesPkg/Include/SmbiosTableGenerator.h | 240 1 file changed, 240

[edk2] [PATCH v2 09/22] DynamicTablesPkg: Configuration Manager Protocol

2019-01-24 Thread Sami Mujawar
Introduce configuration manager protocol interface that is used by the dynamic tables framework core to communicate with configuration manager. Configuration manager is a platform specific module that implements the configuration manager protocol. Table generators use this interface to retrieve

[edk2] [PATCH v2 02/22] DynamicTablesPkg: Table Generator definition

2019-01-24 Thread Sami Mujawar
A Table generator is a component that implements the logic for building a firmware table. This is typically implemented as a library and registers itself with a table factory. Table generators are further classified based on type of table it generates, a namespace that signifies if the

[edk2] [PATCH v2 06/22] DynamicTablesPkg: Standard NameSpace Objects

2019-01-24 Thread Sami Mujawar
The dynamic tables frameworks core communicates with the platform specific implementation using the configuration manager protocol interface. The dynamic tables framework core uses this interface to retrieve information required for generating the firmware tables. This information is represented

[edk2] [PATCH v2 01/22] DynamicTablesPkg: Dynamic Tables Framework

2019-01-24 Thread Sami Mujawar
The dynamic tables framework is designed to generate standardised firmware tables that describe the hardware information at run-time. A goal of standardised firmware is to have a common firmware for a platform capable of booting both Windows and Linux operating systems. Traditionally the firmware

[edk2] [PATCH v2 07/22] DynamicTablesPkg: Arm NameSpace Objects

2019-01-24 Thread Sami Mujawar
The dynamic tables frameworks core communicates with the platform specific implementation using the configuration manager protocol interface. The dynamic tables framework core uses this interface to retrieve information required for generating the firmware tables. This information is represented

[edk2] [PATCH v2 14/22] DynamicTablesPkg: Dynamic Table Manager Dxe

2019-01-24 Thread Sami Mujawar
The dynamic table manager implements the top level component that drives the table generation and installation process. It uses the configuration manager protocol to get the list of tables to be installed from the configuration manager. It iterates through the list of tables, requests the table

[edk2] [PATCH v2 13/22] DynamicTablesPkg: Dynamic Table Factory Dxe

2019-01-24 Thread Sami Mujawar
The dynamic table factory dxe implements the dynamic table factory protocol. It also implements the ACPI, SMBIOS and DT table factories. The table generators register themselves with the respective table factories and the factories are responsible for instantiating instances of the generators to

[edk2] [PATCH v2 20/22] DynamicTablesPkg: Arm DBG2 Table Generator

2019-01-24 Thread Sami Mujawar
The DBG2 generator uses the configuration manager protocol to obtain the debug serial port information from the platform configuration manager. It then updates a template DBG2 table structure. This table data is used by the Table Manager to install the DBG2 table. Contributed-under: TianoCore

[edk2] [PATCH v2 22/22] DynamicTablesPkg: Arm IORT Table Generator

2019-01-24 Thread Sami Mujawar
The IORT generator uses the configuration manager protocol to obtain information about the PCI Root Complex, SMMU, GIC ITS, Performance Monitoring counters etc. and generates the IORT table. The mappings between the components are represented using tokens. The generator invokes the configuration

[edk2] [PATCH v2 19/22] DynamicTablesPkg: Arm SPCR Table Generator

2019-01-24 Thread Sami Mujawar
The SPCR generator uses the configuration manager protocol to obtain the serial port information from the platform configuration manager. It then updates a template SPCR table structure. This table data is used by the Table Manager to install the SPCR table. Contributed-under: TianoCore

[edk2] [PATCH v2 18/22] DynamicTablesPkg: Arm ACPI GTDT Generator

2019-01-24 Thread Sami Mujawar
The GTDT generator uses the configuration manager protocol to obtain information about the architectural and platform timers available on the platform and generates the ACPI GTDT table. This table data is then used by the Table Manager to install the GTDT table. The Table Manager then invokes the

[edk2] [PATCH v2 21/22] DynamicTablesPkg: Arm PCI MCFG Table Generator

2019-01-24 Thread Sami Mujawar
The MCFG generator uses the configuration manager protocol to obtain the PCI Configuration space information from the platform configuration manager and builds the MCFG table. This table data is then used by the Table Manager to install the MCFG table. The Table Manager then invokes the

[edk2] [PATCH edk2-platforms v3 0/4] Platform/ARM: Platform support for Dynamic Tables Framework

2019-01-24 Thread Sami Mujawar
Dynamic Tables Framework aims to reduce the amount of effort required for porting firmware to new platforms by simplifying the generation of firmware tables based on hardware description provided by a platform specific component. The Dynamic Tables Framework core queries the platform specific

  1   2   >