Ard, I still cannot understand it. Since Driver#### are processed (process = LoadImage + StartImage) after EndOfDxe, they are not deferred. It means the Driver#### entrypoints are called directly. Inside the entrypoints, driverbinding protocols are installed.
After that, EfiBootManagerConnectAllDefaultConsoles () uses driver model logic to start the proper GOP driver. Thanks, Ray > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ard Biesheuvel > Sent: Wednesday, December 11, 2019 6:40 PM > To: Ni, Ray <ray...@intel.com> > Cc: Laszlo Ersek <ler...@redhat.com>; edk2-devel-groups-io > <devel@edk2.groups.io>; Leif Lindholm > <leif.lindh...@linaro.org>; Gao, Zhichao <zhichao....@intel.com>; Ma, Maurice > <maurice...@intel.com>; Dong, Guo > <guo.d...@intel.com>; You, Benjamin <benjamin....@intel.com> > Subject: Re: [edk2-devel] [RFC PATCH 2/2] MdeModulePkg/BdsDxe: allow consoles > on drivers loaded via Driver#### > > On Mon, 9 Dec 2019 at 09:42, Ard Biesheuvel <ard.biesheu...@linaro.org> wrote: > > > > On Mon, 9 Dec 2019 at 03:12, Ni, Ray <ray...@intel.com> wrote: > > > > > > > Exactly. This flow is identical to how option ROMs are processed if > > > > they are discovered before EndOfDxe signalling completes (which is why > > > > the Juno platform was broken without the call to > > > > EfiBootManagerDispatchDeferredImages() in > > > > PlatformBootManagerBeforeConsole()) > > > > > > > > > > Ard, > > > I checked ArmPkg's PlatformBootManagerLib and found it doesn't > > > call *DispatchDeferredImages() after signaling EndOfDxe. > > > > > > > It does. We just added this in 0f9395d7c5cc6ae2beaa2d87008fe158d04a8069 > > > > > The deferred image dispatch mechanism assumes the platform > > > needs to call the *DispatchDeferredImages() after signaling EndOfDxe. > > > > > > > Indeed. > > > > > I don't understand why the deferred image can be loaded with your patch. > > > They are still deferred because the loading time is before EndOfDxe. > > > > > > > Yes, but because PlatformBootManagerBeforeConsole () does all of this, > > the only way to get Driver#### to work for consoles on GOP drivers, we > > need to move it before that call. > > Any further comments on this patch? > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52156): https://edk2.groups.io/g/devel/message/52156 Mute This Topic: https://groups.io/mt/67470372/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
