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: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