Re: [edk2] [PATCH] BaseTools/tools_def AARCH64: enable frame pointers for RELEASE builds

2017-09-16 Thread Leif Lindholm
On Fri, Sep 15, 2017 at 04:13:56PM -0700, Ard Biesheuvel wrote: > Commit 8f0b62a5dac0 ("BaseTools/tools_def AARCH64: enable frame pointers > for DEBUG builds") removed the -fomit-frame-pointer switch from the > shared CFLAGS between AARCH64 DEBUG and RELEASE build, and move it to > the RELEASE

Re: [edk2] [PATCH 0/4] IntelSiliconPkg: move VTd related modules to feature dir.

2017-09-16 Thread Zeng, Star
Thanks for making this. Reviewed-by: Star Zeng Thanks, Star -Original Message- From: Yao, Jiewen Sent: Saturday, September 16, 2017 9:40 AM To: edk2-devel@lists.01.org Cc: Zeng, Star Subject: [PATCH 0/4] IntelSiliconPkg: move VTd related

[edk2] [PATCH 0/3] UDF partition driver fix

2017-09-16 Thread Paulo Alcantara
This series fixes an UDF issue with Partition driver as discussed in ML: https://lists.01.org/pipermail/edk2-devel/2017-September/014694.html Thanks! Paulo Repo: https://github.com/pcacjr/edk2.git Branch: udf-partition-fix Paulo Alcantara (3): MdePkg: Add UDF volume structure definitions

[edk2] [PATCH 1/3] MdePkg: Add UDF volume structure definitions

2017-09-16 Thread Paulo Alcantara
This patch adds a fewe more UDF structures in order to detect Logical Volume and Partition descriptors during Main Volume Descriptor Sequence in Partition driver. Cc: Michael D Kinney Cc: Liming Gao Cc: Laszlo Ersek Cc: Ruiyu

[edk2] [PATCH 2/3] MdeModulePkg/PartitionDxe: Fix creation of UDF logical partition

2017-09-16 Thread Paulo Alcantara
Do not use entire block device size for the UDF logical partition, instead reserve the appropriate space (LVD space) for it. Cc: Eric Dong Cc: Ruiyu Ni Cc: Star Zeng Cc: Laszlo Ersek Reported-by: Ruiyu Ni

[edk2] [PATCH 3/3] MdeModulePkg/UdfDxe: Rework driver to support PartitionDxe changes

2017-09-16 Thread Paulo Alcantara
This patch reworks the driver to support Partition driver changes. Cc: Eric Dong Cc: Paulo Alcantara Cc: Ruiyu Ni Cc: Star Zeng Cc: Laszlo Ersek Contributed-under: TianoCore Contribution

Re: [edk2] [PATCH 0/3] UDF partition driver fix

2017-09-16 Thread Yao, Jiewen
Thank you Paulo, to provide a fix for this driver. I do not have comment for this specific patch. I would defer the review work to Star and Ruiyu. I do have some general question for the new UDF support and I would like to know more detail about the quality level. As we are seeing some issues

Re: [edk2] [PATCH] Fix not able to change serial attributes

2017-09-16 Thread Laszlo Ersek
On 09/16/17 03:21, Zeng, Star wrote: > Pankaj, > > Thanks for the contribution. > Could you help update the title to be like MdeModulePkg SerialDxe: XX? > > In fact, I agree the fix as it matches the code at > MdeModulePkg/Bus/Pci/PciSioSerialDxe/SerialIo.c and >

Re: [edk2] Functionality issues in UDF support

2017-09-16 Thread Laszlo Ersek
Jiewen, On 09/16/17 01:38, Yao, Jiewen wrote: > Hi Laszlo and Ruiyu > I can think 1 possible alternative, for your consideration only. > > 1) Move the feature to OvmfPkg. > > As such, it won't block us at this moment. > > Once the UDF solution has good quality, we can move it back to

Re: [edk2] Functionality issues in UDF support

2017-09-16 Thread Laszlo Ersek
Hi Jiewen, thank you for looking into this. I'm replying below quite at length; please bear with me. At the end I'm going to propose a new alternative to the PCD: On 09/16/17 01:58, Yao, Jiewen wrote: > Hi Laszlo/Ruiyu/Star > I went through the whole conversation and have some thought: > > For

Re: [edk2] [PATCH 0/3] UDF partition driver fix

2017-09-16 Thread Laszlo Ersek
Hi Paulo, On 09/16/17 23:36, Paulo Alcantara wrote: > This series fixes an UDF issue with Partition driver as discussed in ML: > > https://lists.01.org/pipermail/edk2-devel/2017-September/014694.html > > Thanks! > Paulo > > Repo: https://github.com/pcacjr/edk2.git > Branch: udf-partition-fix