Re: [edk2] Regarding UefiDriverEntryPoint unload handler

2017-06-14 Thread Marvin Häuser
Hey Mike, "if UnloadImage() is called and the Unload field is NULL, it returns an EFI_UNSUPPORTED." This is the part I missed, my example is impossible in this case. Thanks, Marvin. -Ursprüngliche Nachricht- Von: Kinney, Michael D [mailto:michael.d.kin...@intel.com] Gesendet: Donnersta

Re: [edk2] [Patch 1/2 V2] BaseTools: add /Gw to CC_FLAGS for VS2013 and higher tool chain tags

2017-06-14 Thread Kinney, Michael D
Series Reviewed-by: Michael D Kinney Mike -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Yonghong Zhu Sent: Wednesday, June 14, 2017 12:35 AM To: edk2-devel@lists.01.org Cc: Gao, Liming Subject: [edk2] [Patch 1/2 V2] BaseTools: add /Gw to CC_

Re: [edk2] Regarding UefiDriverEntryPoint unload handler

2017-06-14 Thread Kinney, Michael D
Hi Marvin, Yes. This is intentional. If MODULE_UNLOAD is not provided, then the module can never be unloaded. The LoadImage() services initializes the Unload field to NULL, and if UnloadImage() is called and the Unload field is NULL, it returns an EFI_UNSUPPORTED. I do not think the case yo

[edk2] Regarding UefiDriverEntryPoint unload handler

2017-06-14 Thread Marvin H?user
Dear developers, While performing some research tasks, I noticed that when UefiDriverEntryPoint's _gDriverUnloadImageCount is 0 (only MODULE_UNLOAD entries are count, DESTRUCTOR as they are used with libraries are not, as far as I can see), EFI_LOADED_IMAGE_PROTOCOL.Unload is not set, even if l

[edk2] [PATCH] MdePkg/Cper.h: Update Firmware Error Record per UEFI 2.7

2017-06-14 Thread Hao Wu
This commit updates the Firmware Error Record related definitions according to UEFI 2.7 spec Section N.2.10 Table 281: a. Adds definitions for 2 Firmware Error Record types b. Update the structure EFI_FIRMWARE_ERROR_DATA Cc: Michael D Kinney Cc: Liming Gao Contributed-under: TianoCore Contribut

Re: [edk2] [RFC] migration of OpenPlatformPkg to tianocore

2017-06-14 Thread Leif Lindholm
Hi Mike, Yes, this way I am able to build all of my packages! Many thanks! This also works with the simplified PACKAGES_PATH usage model I had initially been trying for (just point to edk2-platforms and edk2-non-osi top level directories). I have pushed updated and slightly cleaned up versions t

Re: [edk2] [PATCH v2] EmbeddedPkg/MmcDxe: Add alignment for ECSD data

2017-06-14 Thread Leif Lindholm
On Wed, Jun 14, 2017 at 10:50:02AM +0800, Jun Nie wrote: > >> Sorry, just see your email because that thread is not highlighted for > >> new email in gmail for unknown reason. > >> I have concern that "UINT64 VENDOR_SPECIFIC_FIELD[8]" cannot secure > >> the ECSD alignment because it is not the firs

[edk2] [Patch 0/2] Add HTTP Boot Callback Protocol support.

2017-06-14 Thread Fu Siyuan
Cc: Ye Ting Cc: Wu Jiaxin Fu Siyuan (2): MdePkg: Add header file for HTTP Boot Callback protocol in UEFI 2.7. NetworkPkg/HttpBootDxe: Add HTTP Boot Callback protocol support. MdePkg/Include/Protocol/HttpBootCallback.h | 100 + MdePkg/MdePkg.dec | 7 +-

[edk2] [Patch 1/2] MdePkg: Add header file for HTTP Boot Callback protocol in UEFI 2.7.

2017-06-14 Thread Fu Siyuan
Cc: Ye Ting Cc: Wu Jiaxin Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- MdePkg/Include/Protocol/HttpBootCallback.h | 100 + MdePkg/MdePkg.dec | 7 +- 2 files changed, 105 insertions(+), 2 deletions(-)

[edk2] [Patch 2/2] NetworkPkg/HttpBootDxe: Add HTTP Boot Callback protocol support.

2017-06-14 Thread Fu Siyuan
This patch updates the HTTP Boot driver to install a default HTTP Callback protocol if the platform doesn't provide one. This callback implementation will print the boot file download progress in percentage format. Cc: Ye Ting Cc: Wu Jiaxin Contributed-under: TianoCore Contribution Agreement 1.

Re: [edk2] [Patch 1/2] BaseTools: add /Gw to CC_FLAGS for VS2013 and higher tool chain tags

2017-06-14 Thread Zhu, Yonghong
Hi Mike, Thanks. I only checked it build pass, but missed "iec: Command line warning: ignoring unknown option '-Gw'" in the log. I sent out a V2 that not add /Gw for EBC. Best Regards, Zhu Yonghong -Original Message- From: Kinney, Michael D Sent: Wednesday, June 14, 2017 2:51 PM To: Z

[edk2] [Patch 1/2 V2] BaseTools: add /Gw to CC_FLAGS for VS2013 and higher tool chain tags

2017-06-14 Thread Yonghong Zhu
The /Gw flag does a better job at size optimization than use of the GLOBAL_REMOVE_IF_UNREFERENCED macro that is currently used for VS20xx tool chains to remove unreferenced global variables. This patch add /Gw to CC_FLAGS for VS2013 and higher tool chain tags. Cc: Liming Gao Contributed-under: Ti

Re: [edk2] HTTP Boot failed to download NBP file if it is .iso type

2017-06-14 Thread Gary Lin
On Tue, Jun 13, 2017 at 09:31:49AM +, Santhapur Naveen wrote: > Hi Siyuan, > > Thank you for your reply. > And regarding the OS installation, we are able to download SUSE iso (>3 GB) > from the HTTP server. But the install didn't happen. > May I ask you what could be possible reason? Is there