Re: [edk2] [Patch] MdeModulePkg/Core/Dxe: Remove extra connects for UEFI Applications

2017-11-09 Thread Kinney, Michael D
Star, Thanks for noticing the inconsistency. I agree that Image->Type is a better choice. Mike > -Original Message- > From: Zeng, Star > Sent: Thursday, November 9, 2017 5:10 AM > To: Kinney, Michael D ; > edk2-devel@lists.01.org > Cc: Dong, Eric

Re: [edk2] [Patch] MdeModulePkg/Core/Dxe: Remove extra connects for UEFI Applications

2017-11-09 Thread Zeng, Star
Reviewed-by: Star Zeng BTW: I see the code is using Image->ImageContext.ImageType at some places and Image->Type at other place, it seems a little inconsistent. Thanks, Star -Original Message- From: Kinney, Michael D Sent: Thursday, November 9, 2017 1:55 AM To:

[edk2] [Patch] MdeModulePkg/Core/Dxe: Remove extra connects for UEFI Applications

2017-11-08 Thread Michael D Kinney
https://bugzilla.tianocore.org/show_bug.cgi?id=765 The UEFI Specification Boot Services chapter, StartImage() service, EFF 1.10 Extension requires extra calls to ConnectController() if a UEFI Driver produces handles. The DXE Core is performing these extra calls to ConnectController() without