devel@edk2.groups.io

[PATCHV2] MdePkg: Add DmaRemappingReportingTable.h

DMA Remapping Reporting (DMAR) ACPI table definitions from Intel(R) Virtualization Technology for Directed I/O (VT-D) Architecture Specification v2.4 dated June 2016. This replaces the

By Giri P Mudusuru <giri.p.mudusuru@...> · #283 ·

Re: [staging/HTTPS-TLS][PATCH 0/4] Replace the TLS definitions with the standardized one

Jiaxin / Qin, I'm unaware of what criteria is required for a cipher to be in this TlsCipherMappingTable. I had presumed that it would be b/c UEFI supported the cipher for TLS operation. If

By Palmer, Thomas <thomas.palmer@...> · #282 ·

Re: [PATCH] IntelFsp2Pkg: Locate FSP Info Header dynamically

Reviewed-by: Giri P Mudusuru <giri.p.mudusuru@...> Please update the comments for the opcode to be in line similar to the

By Mudusuru, Giri P <giri.p.mudusuru@...> · #281 ·

Re: [PATCH] IntelFsp2Pkg: Locate FSP Info Header dynamically

It looks good to me. Reviewed by: Maurice Ma <maurice.ma@...> Regards, Maurice

By Ma, Maurice <maurice.ma@...> · #280 ·

Re: [PATCH] ArmPkg/Library: Add ArmReadSctlr for aarch64

Thanks Ard.

By Supreeth Venkatesh · #279 ·

Re: [PATCH] MdePkg: Add DmaRemappingReportingTable.h

Giri, Based on the information in this thread, I am in favor of adding to MdePkg. If the number of files in MdePkg/Include/IndustryStandard grows too large then some additional directory

By Michael D Kinney · #278 ·

Re: [PATCH] add top-level .gitattributes file, dealing with .depex

Jordan -- As a company that delivers a lot of mixed binary/source builds, we see .depex as actually important for ease of maintenance. The .fdf syntax can work, as you mention, but it is actually

By Tim Lewis · #277 ·

Re: [PATCH] add top-level .gitattributes file, dealing with .depex

Jordan, I agree that we should not be adding .efi binaries or .depex binary files to our source repos. There are several repos for EDK II, and some of those do accept binaries. Specifically, the

By Michael D Kinney · #276 ·

Re: [PATCH] MdePkg: Add DmaRemappingReportingTable.h

Thanks for detailed conversation and I agree with direction. As it is ACPI related which has multiple vendors owning specific tables referred by ACPI spec MdePkg for now seems to be right place. That

By Mudusuru, Giri P <giri.p.mudusuru@...> · #275 ·

Re: [PATCH v5 7/8] MdePkg GCC/X64: avoid 'hidden' visibility for module entry points

As it turns out, the LTO linker does not need to visibility pragma to prevent it from emitting GOT based relocations. This makes sense, considering that the LTO linker can see that no symbol

By Ard Biesheuvel · #274 ·

Re: [PATCH v5 7/8] MdePkg GCC/X64: avoid 'hidden' visibility for module entry points

The only alternative I can think of is to add a static non-lto object to the tree that refers to _ModuleEntryPoint, similar to the way I handle the ARM intrinsics in patch #5 Which one do you hate

By Ard Biesheuvel · #273 ·

[patch] BaseTool/UPT: Not expand macro for UserExtension

All MACRO values defined by the DEFINE statements n any section (except [Userextensions] sections other than TianoCore."ExtraFiles) of the INF or DEC file must be expanded before processing of the

By hesschen <hesheng.chen@...> · #272 ·

Re: [PATCH v5 7/8] MdePkg GCC/X64: avoid 'hidden' visibility for module entry points

Hmm, looks like I spoke too soon. I don't know what I did wrong, but this does not actually work. -- Ard.

By Ard Biesheuvel · #271 ·

Re: [PATCH v5 7/8] MdePkg GCC/X64: avoid 'hidden' visibility for module entry points

It seems this does the trick as well diff --git a/BaseTools/Scripts/GccBase.lds b/BaseTools/Scripts/GccBase.lds index 281af8a9bd33..02387d4f8d6f 100644 --- a/BaseTools/Scripts/GccBase.lds +++

By Ard Biesheuvel · #270 ·

[PATCH 2/2] MdeModulePkg/EbcDxe: cleanup

- indentation - premature optimization of the thunking code, i.e., align function prototypes so we don't have to move arguments around or duplicate them on the stack, and perform tail calls where

By Ard Biesheuvel · #269 ·

[PATCH 1/2] MdeModulePkg/EbcDxe AARCH64: fix comment style

Replace '#' comments with '//' Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel <ard.biesheuvel@...> ---

By Ard Biesheuvel · #268 ·

Re: [PATCH 1/2] BaseTools/GenFw AARCH64: convert ADRP to ADR if binary size allows it

Apologies, lost track of this one. It's absolutely horrid, but extremely useful. For the series: Reviewed-by: Leif Lindholm <leif.lindholm@...>

By Leif Lindholm <leif.lindholm@...> · #267 ·

Re: [PATCH v5 7/8] MdePkg GCC/X64: avoid 'hidden' visibility for module entry points

Ard: I don't think it is good way to define GCC_VISIBILITY_PROTECTED and apply it in EntryPointLib. We only need to expose _ModuleEntryPoint. It has been specified in LINK_FLAGS in tools_def.txt.

By Liming Gao · #266 ·

Re: [PATCH 1/2] BaseTools/GenFw AARCH64: convert ADRP to ADR if binary size allows it

Ard: Your change is OK to me. I have no comment. Thanks Liming Sent: Monday, August 1, 2016 7:53 PM To: edk2-devel-01 <edk2-devel@...>; Gao, Liming <liming.gao@...>; Zhu, Yonghong

By Liming Gao · #265 ·

Re: [PATCH v5 0/8] BaseTools: add support for GCC5 in LTO mode

https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/gcc5-lto-v5 or git://git.linaro.org/people/ard.biesheuvel/uefi-next.git gcc5-lto-v5 Thanks, Ard.

By Ard Biesheuvel · #264 ·

Messages