From nobody Tue Dec 24 12:27:03 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 151151708303161.58131187718857; Fri, 24 Nov 2017 01:51:23 -0800 (PST) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 54BDB220F3C31; Fri, 24 Nov 2017 01:47:00 -0800 (PST) Received: from mail-wm0-x242.google.com (mail-wm0-x242.google.com [IPv6:2a00:1450:400c:c09::242]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id CB50E220F3C2A for ; Fri, 24 Nov 2017 01:46:58 -0800 (PST) Received: by mail-wm0-x242.google.com with SMTP id x63so21240256wmf.4 for ; Fri, 24 Nov 2017 01:51:16 -0800 (PST) Received: from localhost.localdomain ([160.90.99.19]) by smtp.gmail.com with ESMTPSA id l1sm2717340wmh.42.2017.11.24.01.51.10 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 24 Nov 2017 01:51:12 -0800 (PST) X-Original-To: edk2-devel@lists.01.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c09::242; helo=mail-wm0-x242.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=tfi9gxnzQv6Fq5o6IXrdS68YmdXq3VnfT2M/DIQcsDg=; b=ONC9hFi/LDVVrLQoG6Fym1cbU4KoX5xcc4i6FckVRELpagoNeGNMC4KxAMGyEdySaM D/YPPK7KtkS0SW270Uzow68eQcObNkz5BHWOiqekSD+RmwhNRyBWHWQKRxS0bRdOG50X Avmce7cCR0NsE6GpRQjxz/CGWR2MY8eCngTko= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=tfi9gxnzQv6Fq5o6IXrdS68YmdXq3VnfT2M/DIQcsDg=; b=qNcxTPu7FmQaoeRhMKg2Bg+5jsYFhKh6JJVz0uH1uQuEzi2eQhZDMWlLEmnMF9rUad 8o2qWJq3mmiWoXtM1sO/blU3YPhCGWkQ0ljo40w199gcMusUcGAD17gDv2IsZ1PVAUBz ua7Dt/em+feyy1BfZd/Cmnn9Ata0CK6t8CaWU3qzQsibVoNr+2Ea93O3JsL7gkx3f0R8 Jsok6pJZN3UbF61dNvG/Fmbh6GzdDgZKfhuBBPssi0Dr6OOG3cwwYApNFhaPrOoMfcxB Rj90PTzqn0WboNQ+DuD4XzgWjG8onjEdHh7BSZFmCEiwDkuybc7zQwEAbXrnnnDM8Ogx YJcA== X-Gm-Message-State: AJaThX46w5ZoreuGKAQPz/Gml42z7MlqDlbt7VWOA3G52UPHwDLpxmM1 dmNzObfVIl6sJYp81usVeTyYSQgp+Y4= X-Google-Smtp-Source: AGs4zMYP3IABxedsYvsUiqGUuUr3QHepxWTQiDIOjA0FnY6oJJoMbY8Q7d7w94JK2kwXzDDSHNhNSA== X-Received: by 10.28.220.67 with SMTP id t64mr10354667wmg.150.1511517074661; Fri, 24 Nov 2017 01:51:14 -0800 (PST) From: Ard Biesheuvel To: edk2-devel@lists.01.org Date: Fri, 24 Nov 2017 09:51:03 +0000 Message-Id: <20171124095103.26099-1-ard.biesheuvel@linaro.org> X-Mailer: git-send-email 2.11.0 Subject: [edk2] [PATCH] ArmVirtPkg/PrePi: don't export PE/COFF and LZMA libraries via HOBs X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: lersek@redhat.com, Ard Biesheuvel MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" The PrePi code we inherited from ArmPlatformPkg contains a rather obscure optimization, where entry points of the PE/COFF and LZMA handling routines are recorded in special HOBs, allowing DXE core to call into that code directly rather than carry its own copy of these libraries. Given that no ArmVirtPkg platforms actually include the library resolutions* that take advantage of these optimizations, let's not bother with them, and remove the associated code. * EmbeddedPkg/Library/LzmaHobCustomDecompressLib/LzmaHobCustomDecompressLib= .inf EmbeddedPkg/Library/DxeHobPeCoffLib/DxeHobPeCoffLib.inf Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel Acked-by: Laszlo Ersek --- ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf | 1 - ArmVirtPkg/PrePi/LzmaDecompress.h | 103 ----------------= ---- ArmVirtPkg/PrePi/PrePi.c | 11 --- 3 files changed, 115 deletions(-) diff --git a/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf b/ArmVirtP= kg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf index 58290d2d1b76..b3a3f5da065e 100755 --- a/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf +++ b/ArmVirtPkg/PrePi/ArmVirtPrePiUniCoreRelocatable.inf @@ -51,7 +51,6 @@ [LibraryClasses] SerialPortLib ExtractGuidedSectionLib LzmaDecompressLib - PeCoffGetEntryPointLib PrePiLib MemoryAllocationLib HobLib diff --git a/ArmVirtPkg/PrePi/LzmaDecompress.h b/ArmVirtPkg/PrePi/LzmaDecom= press.h deleted file mode 100644 index a79ff343d231..000000000000 --- a/ArmVirtPkg/PrePi/LzmaDecompress.h +++ /dev/null @@ -1,103 +0,0 @@ -/** @file - LZMA Decompress Library header file - - Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.
- This program and the accompanying materials - are licensed and made available under the terms and conditions of the BS= D License - which accompanies this distribution. The full text of the license may b= e found at - http://opensource.org/licenses/bsd-license.php - - THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, - WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMP= LIED. - -**/ - -#ifndef __LZMA_DECOMPRESS_H___ -#define __LZMA_DECOMPRESS_H___ - -/** - Examines a GUIDed section and returns the size of the decoded buffer and= the - size of an scratch buffer required to actually decode the data in a GUID= ed section. - - Examines a GUIDed section specified by InputSection. - If GUID for InputSection does not match the GUID that this handler suppo= rts, - then RETURN_UNSUPPORTED is returned. - If the required information can not be retrieved from InputSection, - then RETURN_INVALID_PARAMETER is returned. - If the GUID of InputSection does match the GUID that this handler suppor= ts, - then the size required to hold the decoded buffer is returned in OututBu= fferSize, - the size of an optional scratch buffer is returned in ScratchSize, and t= he Attributes field - from EFI_GUID_DEFINED_SECTION header of InputSection is returned in Sect= ionAttribute. - - If InputSection is NULL, then ASSERT(). - If OutputBufferSize is NULL, then ASSERT(). - If ScratchBufferSize is NULL, then ASSERT(). - If SectionAttribute is NULL, then ASSERT(). - - - @param[in] InputSection A pointer to a GUIDed section of an FFS f= ormatted file. - @param[out] OutputBufferSize A pointer to the size, in bytes, of an ou= tput buffer required - if the buffer specified by InputSection w= ere decoded. - @param[out] ScratchBufferSize A pointer to the size, in bytes, required= as scratch space - if the buffer specified by InputSection w= ere decoded. - @param[out] SectionAttribute A pointer to the attributes of the GUIDed= section. See the Attributes - field of EFI_GUID_DEFINED_SECTION in the = PI Specification. - - @retval RETURN_SUCCESS The information about InputSection wa= s returned. - @retval RETURN_UNSUPPORTED The section specified by InputSection= does not match the GUID this handler supports. - @retval RETURN_INVALID_PARAMETER The information can not be retrieved = from the section specified by InputSection. - -**/ -RETURN_STATUS -EFIAPI -LzmaGuidedSectionGetInfo ( - IN CONST VOID *InputSection, - OUT UINT32 *OutputBufferSize, - OUT UINT32 *ScratchBufferSize, - OUT UINT16 *SectionAttribute - ); - -/** - Decompress a LZAM compressed GUIDed section into a caller allocated outp= ut buffer. - - Decodes the GUIDed section specified by InputSection. - If GUID for InputSection does not match the GUID that this handler suppo= rts, then RETURN_UNSUPPORTED is returned. - If the data in InputSection can not be decoded, then RETURN_INVALID_PARA= METER is returned. - If the GUID of InputSection does match the GUID that this handler suppor= ts, then InputSection - is decoded into the buffer specified by OutputBuffer and the authenticat= ion status of this - decode operation is returned in AuthenticationStatus. If the decoded bu= ffer is identical to the - data in InputSection, then OutputBuffer is set to point at the data in I= nputSection. Otherwise, - the decoded data will be placed in caller allocated buffer specified by = OutputBuffer. - - If InputSection is NULL, then ASSERT(). - If OutputBuffer is NULL, then ASSERT(). - If ScratchBuffer is NULL and this decode operation requires a scratch bu= ffer, then ASSERT(). - If AuthenticationStatus is NULL, then ASSERT(). - - - @param[in] InputSection A pointer to a GUIDed section of an FFS format= ted file. - @param[out] OutputBuffer A pointer to a buffer that contains the result= of a decode operation. - @param[out] ScratchBuffer A caller allocated buffer that may be required= by this function - as a scratch buffer to perform the decode oper= ation. - @param[out] AuthenticationStatus - A pointer to the authentication status of the = decoded output buffer. - See the definition of authentication status in= the EFI_PEI_GUIDED_SECTION_EXTRACTION_PPI - section of the PI Specification. EFI_AUTH_STAT= US_PLATFORM_OVERRIDE must - never be set by this handler. - - @retval RETURN_SUCCESS The buffer specified by InputSection = was decoded. - @retval RETURN_UNSUPPORTED The section specified by InputSection= does not match the GUID this handler supports. - @retval RETURN_INVALID_PARAMETER The section specified by InputSection= can not be decoded. - -**/ -RETURN_STATUS -EFIAPI -LzmaGuidedSectionExtraction ( - IN CONST VOID *InputSection, - OUT VOID **OutputBuffer, - OUT VOID *ScratchBuffer, OPTIONAL - OUT UINT32 *AuthenticationStatus - ); - -#endif // __LZMADECOMPRESS_H__ - diff --git a/ArmVirtPkg/PrePi/PrePi.c b/ArmVirtPkg/PrePi/PrePi.c index fce4ab9428a5..f6abe2f2016b 100755 --- a/ArmVirtPkg/PrePi/PrePi.c +++ b/ArmVirtPkg/PrePi/PrePi.c @@ -17,7 +17,6 @@ =20 #include #include -#include #include #include #include @@ -25,10 +24,8 @@ =20 #include #include -#include =20 #include "PrePi.h" -#include "LzmaDecompress.h" =20 VOID EFIAPI @@ -98,14 +95,6 @@ PrePiMain ( // SEC phase needs to run library constructors by hand. ProcessLibraryConstructorList (); =20 - // Build HOBs to pass up our version of stuff the DXE Core needs to save= space - BuildPeCoffLoaderHob (); - BuildExtractSectionHob ( - &gLzmaCustomDecompressGuid, - LzmaGuidedSectionGetInfo, - LzmaGuidedSectionExtraction - ); - // Assume the FV that contains the SEC (our code) also contains a compre= ssed FV. Status =3D DecompressFirstFv (); ASSERT_EFI_ERROR (Status); --=20 2.11.0 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel