From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=fail (domain: intel.com, ip: , mailfrom: hao.a.wu@intel.com) Received: from mga17.intel.com (mga17.intel.com []) by groups.io with SMTP; Sun, 14 Apr 2019 18:23:17 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Apr 2019 18:23:17 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,351,1549958400"; d="scan'208";a="223523265" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by orsmga001.jf.intel.com with ESMTP; 14 Apr 2019 18:23:16 -0700 Received: from fmsmsx153.amr.corp.intel.com (10.18.125.6) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sun, 14 Apr 2019 18:23:16 -0700 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by FMSMSX153.amr.corp.intel.com (10.18.125.6) with Microsoft SMTP Server (TLS) id 14.3.408.0; Sun, 14 Apr 2019 18:23:16 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.92]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.39]) with mapi id 14.03.0415.000; Mon, 15 Apr 2019 09:23:13 +0800 From: "Wu, Hao A" To: Ard Biesheuvel , "devel@edk2.groups.io" CC: "Kinney, Michael D" , Andrew Fish , Leif Lindholm , "Zeng, Star" , "Dong, Eric" , "Ni, Ray" , "Gao, Liming" , "Carsey, Jaben" , "Shi, Steven" , "Wang, Jian J" Subject: Re: [PATCH v6 2/7] MdeModulePkg/DxeCore: invoke the emulator protocol for foreign images Thread-Topic: [PATCH v6 2/7] MdeModulePkg/DxeCore: invoke the emulator protocol for foreign images Thread-Index: AQHU8vupbqW7c7XgW06KfMrQ245VVaY8bTwQ Date: Mon, 15 Apr 2019 01:23:13 +0000 Message-ID: References: <20190414195233.30045-1-ard.biesheuvel@linaro.org> <20190414195233.30045-3-ard.biesheuvel@linaro.org> In-Reply-To: <20190414195233.30045-3-ard.biesheuvel@linaro.org> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: hao.a.wu@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable > -----Original Message----- > From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org] > Sent: Monday, April 15, 2019 3:52 AM > To: devel@edk2.groups.io > Cc: Ard Biesheuvel; Kinney, Michael D; Andrew Fish; Leif Lindholm; Zeng, = Star; > Dong, Eric; Ni, Ray; Gao, Liming; Carsey, Jaben; Shi, Steven; Wang, Jian = J; Wu, > Hao A > Subject: [PATCH v6 2/7] MdeModulePkg/DxeCore: invoke the emulator > protocol for foreign images >=20 > When encountering PE/COFF images that cannot be supported natively, > attempt to locate an instance of the PE/COFF image emulator protocol, > and if it supports the image, proceed with loading it and register it > with the emulator. >=20 > Signed-off-by: Ard Biesheuvel > Reviewed-by: Michael D Kinney > --- > MdeModulePkg/Core/Dxe/DxeMain.h | 3 + > MdeModulePkg/Core/Dxe/DxeMain.inf | 1 + > MdeModulePkg/Core/Dxe/Image/Image.c | 171 ++++++++++++++++++-- > 3 files changed, 164 insertions(+), 11 deletions(-) >=20 > diff --git a/MdeModulePkg/Core/Dxe/DxeMain.h > b/MdeModulePkg/Core/Dxe/DxeMain.h > index 2dec9da5e35b..48ec30a48aa2 100644 > --- a/MdeModulePkg/Core/Dxe/DxeMain.h > +++ b/MdeModulePkg/Core/Dxe/DxeMain.h > @@ -53,6 +53,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY > KIND, EITHER EXPRESS OR IMPLIED. > #include > #include > #include > +#include > #include > #include > #include > @@ -228,6 +229,8 @@ typedef struct { > UINT16 Machine; > /// EBC Protocol pointer > EFI_EBC_PROTOCOL *Ebc; > + /// PE/COFF Image Emulator Protocol pointer > + EDKII_PECOFF_IMAGE_EMULATOR_PROTOCOL *PeCoffEmu; > /// Runtime image list > EFI_RUNTIME_IMAGE_ENTRY *RuntimeData; > /// Pointer to Loaded Image Device Path Protocol > diff --git a/MdeModulePkg/Core/Dxe/DxeMain.inf > b/MdeModulePkg/Core/Dxe/DxeMain.inf > index 10375443c0f4..ce6fc19be5e4 100644 > --- a/MdeModulePkg/Core/Dxe/DxeMain.inf > +++ b/MdeModulePkg/Core/Dxe/DxeMain.inf > @@ -162,6 +162,7 @@ > gEfiHiiPackageListProtocolGuid ## SOMETIMES_PRODUCES > gEfiEbcProtocolGuid ## SOMETIMES_CONSUMES > gEfiSmmBase2ProtocolGuid ## SOMETIMES_CONSUMES > + gEdkiiPeCoffImageEmulatorProtocolGuid ## SOMETIMES_CONSUMES >=20 > # Arch Protocols > gEfiBdsArchProtocolGuid ## CONSUMES > diff --git a/MdeModulePkg/Core/Dxe/Image/Image.c > b/MdeModulePkg/Core/Dxe/Image/Image.c > index eddca140ee1a..57330636b5f3 100644 > --- a/MdeModulePkg/Core/Dxe/Image/Image.c > +++ b/MdeModulePkg/Core/Dxe/Image/Image.c > @@ -29,6 +29,15 @@ LOAD_PE32_IMAGE_PRIVATE_DATA > mLoadPe32PrivateData =3D { > } > }; >=20 > +typedef struct { > + LIST_ENTRY Link; > + EDKII_PECOFF_IMAGE_EMULATOR_PROTOCOL *Emulator; > + UINT16 MachineType; > +} EMULATOR_ENTRY; > + > +STATIC LIST_ENTRY mAvailableEmulators; > +STATIC EFI_EVENT mPeCoffEmuProtocolRegistrationEv= ent; > +STATIC VOID *mPeCoffEmuProtocolNotifyRegistr= ation; >=20 > // > // This code is needed to build the Image handle for the DXE Core > @@ -67,6 +76,7 @@ LOADED_IMAGE_PRIVATE_DATA mCorePrivateImage =3D { > NULL, // JumpContext > 0, // Machine > NULL, // Ebc > + NULL, // PeCoffEmu > NULL, // RuntimeData > NULL // LoadedImageDevicePath > }; > @@ -118,6 +128,61 @@ GetMachineTypeName ( > return L""; > } >=20 > +/** > + Notification event handler registered by CoreInitializeImageServices (= ) to > + keep track of which PE/COFF image emulators are available. > + > + @param Event The Event that is being processed, not used. > + @param Context Event Context, not used. > + > +**/ > +STATIC > +VOID > +EFIAPI > +PeCoffEmuProtocolNotify ( > + IN EFI_EVENT Event, > + IN VOID *Context > + ) > +{ > + EFI_STATUS Status; > + UINTN BufferSize; > + EFI_HANDLE EmuHandle; > + EMULATOR_ENTRY *Entry; > + > + EmuHandle =3D NULL; > + > + while (TRUE) { > + BufferSize =3D sizeof (EmuHandle); > + Status =3D CoreLocateHandle ( > + ByRegisterNotify, > + NULL, > + mPeCoffEmuProtocolNotifyRegistration, > + &BufferSize, > + &EmuHandle > + ); > + if (EFI_ERROR (Status)) { > + // > + // If no more notification events exit > + // > + return; > + } > + > + Entry =3D AllocateZeroPool (sizeof (*Entry)); > + ASSERT (Entry !=3D NULL); > + > + Status =3D CoreHandleProtocol ( > + EmuHandle, > + &gEdkiiPeCoffImageEmulatorProtocolGuid, > + (VOID **)&Entry->Emulator > + ); > + ASSERT_EFI_ERROR (Status); > + > + Entry->MachineType =3D Entry->Emulator->MachineType; > + > + InsertTailList (&mAvailableEmulators, &Entry->Link); > + } > +} > + > /** > Add the Image Services to EFI Boot Services Table and install the prot= ocol > interfaces for this image. > @@ -192,6 +257,30 @@ CoreInitializeImageServices ( > gDxeCoreImageHandle =3D Image->Handle; > gDxeCoreLoadedImage =3D &Image->Info; >=20 > + // > + // Create the PE/COFF emulator protocol registration event > + // > + Status =3D CoreCreateEvent ( > + EVT_NOTIFY_SIGNAL, > + TPL_CALLBACK, > + PeCoffEmuProtocolNotify, > + NULL, > + &mPeCoffEmuProtocolRegistrationEvent > + ); > + ASSERT_EFI_ERROR(Status); > + > + // > + // Register for protocol notifications on this event > + // > + Status =3D CoreRegisterProtocolNotify ( > + &gEdkiiPeCoffImageEmulatorProtocolGuid, > + mPeCoffEmuProtocolRegistrationEvent, > + &mPeCoffEmuProtocolNotifyRegistration > + ); > + ASSERT_EFI_ERROR(Status); > + > + InitializeListHead (&mAvailableEmulators); > + > if (FeaturePcdGet (PcdFrameworkCompatibilitySupport)) { > // > // Export DXE Core PE Loader functionality for backward compatibilit= y. > @@ -425,6 +514,49 @@ GetPeCoffImageFixLoadingAssignedAddress( > DEBUG ((EFI_D_INFO|EFI_D_LOAD, "LOADING MODULE FIXED INFO: Loading > module at fixed address 0x%11p. Status =3D %r \n", (VOID > *)(UINTN)(ImageContext->ImageAddress), Status)); > return Status; > } > + > +/** > + Decides whether a PE/COFF image can execute on this system, either > natively > + or via emulation/interpretation. In the latter case, the PeCoffEmu mem= ber > + of the LOADED_IMAGE_PRIVATE_DATA struct pointer is populated with a > pointer > + to the emulator protocol that supports this image. > + > + @param[in] Image LOADED_IMAGE_PRIVATE_DATA struct pointer Hello Ard, Sorry for the delayed response. Could you help to update the above comments to: '@param[in, out] Image ...' here to please the Ecc checker when you push the series? Best Regards, Hao Wu > + > + @retval TRUE The image is supported > + @retval FALSE The image is not supported > + > +**/ > +STATIC > +BOOLEAN > +CoreIsImageTypeSupported ( > + IN OUT LOADED_IMAGE_PRIVATE_DATA *Image > + ) > +{ > + LIST_ENTRY *Link; > + EMULATOR_ENTRY *Entry; > + > + for (Link =3D GetFirstNode (&mAvailableEmulators); > + !IsNull (&mAvailableEmulators, Link); > + Link =3D GetNextNode (&mAvailableEmulators, Link)) { > + > + Entry =3D BASE_CR (Link, EMULATOR_ENTRY, Link); > + if (Entry->MachineType !=3D Image->ImageContext.Machine) { > + continue; > + } > + > + if (Entry->Emulator->IsImageSupported (Entry->Emulator, > + Image->ImageContext.ImageType, > + Image->Info.FilePath)) { > + Image->PeCoffEmu =3D Entry->Emulator; > + return TRUE; > + } > + } > + > + return EFI_IMAGE_MACHINE_TYPE_SUPPORTED (Image- > >ImageContext.Machine) || > + EFI_IMAGE_MACHINE_CROSS_TYPE_SUPPORTED (Image- > >ImageContext.Machine); > +} > + > /** > Loads, relocates, and invokes a PE/COFF image >=20 > @@ -473,16 +605,15 @@ CoreLoadPeImage ( > return Status; > } >=20 > - if (!EFI_IMAGE_MACHINE_TYPE_SUPPORTED (Image- > >ImageContext.Machine)) { > - if (!EFI_IMAGE_MACHINE_CROSS_TYPE_SUPPORTED (Image- > >ImageContext.Machine)) { > - // > - // The PE/COFF loader can support loading image types that can be > executed. > - // If we loaded an image type that we can not execute return > EFI_UNSUPORTED. > - // > - DEBUG ((EFI_D_ERROR, "Image type %s can't be loaded ", > GetMachineTypeName(Image->ImageContext.Machine))); > - DEBUG ((EFI_D_ERROR, "on %s UEFI system.\n", > GetMachineTypeName(mDxeCoreImageMachineType))); > - return EFI_UNSUPPORTED; > - } > + if (!CoreIsImageTypeSupported (Image)) { > + // > + // The PE/COFF loader can support loading image types that can be > executed. > + // If we loaded an image type that we can not execute return > EFI_UNSUPPORTED. > + // > + DEBUG ((DEBUG_ERROR, "Image type %s can't be loaded on %s UEFI > system.\n", > + GetMachineTypeName (Image->ImageContext.Machine), > + GetMachineTypeName (mDxeCoreImageMachineType))); > + return EFI_UNSUPPORTED; > } >=20 > // > @@ -687,6 +818,16 @@ CoreLoadPeImage ( > if (EFI_ERROR(Status)) { > goto Done; > } > + } else if (Image->PeCoffEmu !=3D NULL) { > + Status =3D Image->PeCoffEmu->RegisterImage (Image->PeCoffEmu, > + Image->ImageBasePage, > + EFI_PAGES_TO_SIZE (Image->NumberOfPages= ), > + &Image->EntryPoint); > + if (EFI_ERROR (Status)) { > + DEBUG ((DEBUG_LOAD | DEBUG_ERROR, > + "CoreLoadPeImage: Failed to register foreign image with emulator= .\n")); > + goto Done; > + } > } >=20 > // > @@ -874,6 +1015,13 @@ CoreUnloadAndCloseImage ( > Image->Ebc->UnloadImage (Image->Ebc, Image->Handle); > } >=20 > + if (Image->PeCoffEmu !=3D NULL) { > + // > + // If the PE/COFF Emulator protocol exists we must unregister the im= age. > + // > + Image->PeCoffEmu->UnregisterImage (Image->PeCoffEmu, Image- > >ImageBasePage); > + } > + > // > // Unload image, free Image->ImageContext->ModHandle > // > @@ -1599,7 +1747,8 @@ CoreStartImage ( > // > // The image to be started must have the machine type supported by > DxeCore. > // > - if (!EFI_IMAGE_MACHINE_TYPE_SUPPORTED (Image->Machine)) { > + if (!EFI_IMAGE_MACHINE_TYPE_SUPPORTED (Image->Machine) && > + Image->PeCoffEmu =3D=3D NULL) { > // > // Do not ASSERT here, because image might be loaded via > EFI_IMAGE_MACHINE_CROSS_TYPE_SUPPORTED > // But it can not be started. > -- > 2.17.1