On 06/20/16 02:50, Bi, Dandan wrote:
> Hi Laszlo,
> 
> Yes, this is a performance improvement!
> We remove the connect-all  action in BootManagerUiLib,
> so improve the performance when enter Boot Manager form.

That's cool; I've noticed a brief pause there.

Thanks!
Laszlo

> -----Original Message-----
> From: Laszlo Ersek [mailto:[email protected]] 
> Sent: Saturday, June 18, 2016 5:52 AM
> To: Bi, Dandan <[email protected]>; [email protected]
> Cc: Dong, Eric <[email protected]>; Gao, Liming <[email protected]>
> Subject: Re: [edk2] [patch 0/2] Connect all devices in UiApp if needed
> 
> On 06/16/16 03:37, Dandan Bi wrote:
>> Do the connect all action in UiApp if needed, and no need to do the 
>> connect action in other UI library.
>>
>> Cc: Liming Gao <[email protected]>
>> Cc: Eric Dong <[email protected]>
>> Contributed-under: TianoCore Contribution Agreement 1.0
>> Signed-off-by: Dandan Bi <[email protected]>
>> Reviewed-by: Eric Dong <[email protected]>
>>
>> Dandan Bi (2):
>>   MdeModulePkg/UiApp: Connect all devices in UiApp if needed
>>   MdeModulePkg/BootManagerUiLib: No need to connect all devices
>>
>>  MdeModulePkg/Application/UiApp/FrontPage.c          | 6 +++---
>>  MdeModulePkg/Application/UiApp/FrontPage.h          | 3 +--
>>  MdeModulePkg/Library/BootManagerUiLib/BootManager.c | 7 ++-----
>>  3 files changed, 6 insertions(+), 10 deletions(-)
>>
> 
> Is this a performance improvement primarily?
> 
> Thanks
> Laszlo
> 

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to