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.
Thanks, Dandan -----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
