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: [email protected] <[email protected]> On Behalf Of Ard Biesheuvel > Sent: Wednesday, December 11, 2019 6:40 PM > To: Ni, Ray <[email protected]> > Cc: Laszlo Ersek <[email protected]>; edk2-devel-groups-io > <[email protected]>; Leif Lindholm > <[email protected]>; Gao, Zhichao <[email protected]>; Ma, Maurice > <[email protected]>; Dong, Guo > <[email protected]>; You, Benjamin <[email protected]> > 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 <[email protected]> wrote: > > > > On Mon, 9 Dec 2019 at 03:12, Ni, Ray <[email protected]> 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: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
