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:ler...@redhat.com] Sent: Saturday, June 18, 2016 5:52 AM To: Bi, Dandan <dandan...@intel.com>; edk2-de...@ml01.01.org Cc: Dong, Eric <eric.d...@intel.com>; Gao, Liming <liming....@intel.com> 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 <liming....@intel.com> > Cc: Eric Dong <eric.d...@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Dandan Bi <dandan...@intel.com> > Reviewed-by: Eric Dong <eric.d...@intel.com> > > 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 edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
