From patchwork Thu May 24 09:09:43 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ard Biesheuvel X-Patchwork-Id: 136723 Delivered-To: patch@linaro.org Received: by 2002:a2e:9706:0:0:0:0:0 with SMTP id r6-v6csp1938349lji; Thu, 24 May 2018 02:10:01 -0700 (PDT) X-Google-Smtp-Source: AB8JxZp8uW+xeK6y58nVT1CynYT0CBmExqpxRGEFNPzL503XgY9hD8dHwPycue6SKd0SM7blj+Zj X-Received: by 2002:a17:902:778c:: with SMTP id o12-v6mr5598476pll.199.1527153001268; Thu, 24 May 2018 02:10:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1527153001; cv=none; d=google.com; s=arc-20160816; b=zRf0JmtvmDZ/DcqRXhs7t44POYbDHS5dHRBEEFRKenEpSoH4KXubR+4JHagd0Dm/dz xZgVDeLVM3eK5B8tj8Pk8fMszwe4XI1v8LOzyTAK8vYwXykkBgqEhOxxJSA/4z9ZkK/H n2AhNd7s7alP/FWWtxH568WZ+JW3DeKipchccpmp+Vgblj8C7GftiECnJnzCAXWCiSyX FKF2VPkX9hK2Ajkew08JYDTjGPfdq3BgT28D25X3gvARxq9Q4A5w49FNoR7lfyg9gP74 Yze9zZKq1jki3UNATVWcpvBMENjGymYp4tLoEuDI+p6yU+OMikI1ItxrFkKCrvPoP/Ia ymHQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version:cc :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:references:in-reply-to:message-id:date :to:from:dkim-signature:delivered-to:arc-authentication-results; bh=x798+fWfITI+3B3mvt98T5VxTuvOZEwsd9pHOXOof1o=; b=Y3eXRaRHPfvPut6jO/Qj7M0Y9qvDtM5EEAUFT1IpRcJXsEm82KaiFYI06M9V43ZOBt D88evgzRQyiViQdKqlKrhd/Nc+RzetlXiVJPSbNyBmyCpp+arf8t3cYKqHCQ1SU0Kwby xhLXDQQK3tIuWcf+KIHrrF7Geo4UlK9PuQwdgvxLLEAniLwwIptPE9I7w3gcGm2MQk6Y A27eei7qbya6Zahda/QgssSjHOo8B0ysuI04/qao6LcrO1taCb3S//dPTsg1ng5wKbuo 55DEMURaQtoUuldUOlfhXlivMgHAHhoTUIVeKepXwmtnRxWOjTG5Hdsi/d9P1jtvX3vq GQLQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=Ym1wInpp; spf=pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 198.145.21.10 as permitted sender) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from ml01.01.org (ml01.01.org. [198.145.21.10]) by mx.google.com with ESMTPS id d15-v6si3994574pgf.390.2018.05.24.02.10.01 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 24 May 2018 02:10:01 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 198.145.21.10 as permitted sender) client-ip=198.145.21.10; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=Ym1wInpp; spf=pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 198.145.21.10 as permitted sender) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id E50F62083798F; Thu, 24 May 2018 02:09:58 -0700 (PDT) X-Original-To: edk2-devel@lists.01.org Delivered-To: edk2-devel@lists.01.org Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c0c::22e; helo=mail-wr0-x22e.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr0-x22e.google.com (mail-wr0-x22e.google.com [IPv6:2a00:1450:400c:c0c::22e]) (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 2E0E7207E6376 for ; Thu, 24 May 2018 02:09:57 -0700 (PDT) Received: by mail-wr0-x22e.google.com with SMTP id w18-v6so1691650wrn.6 for ; Thu, 24 May 2018 02:09:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=4WwapQSe19fvux92f/oRKcQdyHyZL37u0klOzxPwNzk=; b=Ym1wInppN3xAoivjVmOPYH8Efv4rZwF6FyFmLjg3525e70pccVhuinTRTMgvYwSeCf h2ijihcQmipNKw4DqKlgTaC+mDICfbQrKL1VfUlE4amUMXl5g7beZ6yCmGFw50maS0/P fwRpKeuOJcfdC/2Urm8XPE6msS2WZai3xjOTY= 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:in-reply-to :references; bh=4WwapQSe19fvux92f/oRKcQdyHyZL37u0klOzxPwNzk=; b=gSjTY4Qy9oDhXogIgDLyTTdRnywG4dN6ldwu9SboUmViT/8VE88e9iTLOO9DNoapmH Sv7KkT0Q3U0RqXRWjz1cCuesfn3tCvSXul2NOWde+JRPg4rsmaS1kRFeghe74hCs9AO2 toS2qkZYSIZPMoAfHMs93oNBx9A4BG5Qd7CGH/QKdJm4eXxIeVtjSxm6fv7rX4bOA8zx gp4QwUHi3FN11NmtNDedh8AOAAZJ08aNk8JNH3ySGRnAZacbCojwktE6tBMdvjWe+qnQ JTdEFaVSM7Jo3EINT6aQ595ZTnGIYceuWXvtSB9Rp5oURLpZUtir9+MeUMn1FZ1yl4aL pG0A== X-Gm-Message-State: ALKqPweVzNr+2EIvlTpRf++WJOLAWkqURbbUIk3lrn9PRPrCAGQeFtdG sPEBl6y/ZFfxBVHUQLwNO3C2TNK9TBw= X-Received: by 2002:adf:c32b:: with SMTP id n40-v6mr5406697wrf.91.1527152995427; Thu, 24 May 2018 02:09:55 -0700 (PDT) Received: from localhost.localdomain ([2a01:e35:3995:5470:200:1aff:fe1b:b328]) by smtp.gmail.com with ESMTPSA id x16-v6sm8505467wmc.2.2018.05.24.02.09.53 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 24 May 2018 02:09:54 -0700 (PDT) From: Ard Biesheuvel To: edk2-devel@lists.01.org Date: Thu, 24 May 2018 11:09:43 +0200 Message-Id: <20180524090945.10289-4-ard.biesheuvel@linaro.org> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180524090945.10289-1-ard.biesheuvel@linaro.org> References: <20180524090945.10289-1-ard.biesheuvel@linaro.org> Subject: [edk2] [PATCH v2 3/5] MdePkg/DxeServicesLib: introduce AllocatePeiAccessiblePages routine X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Eric Dong , Ard Biesheuvel , Liming Gao , Dandan Bi , Leif Lindholm , Michael D Kinney , Laszlo Ersek , Star Zeng MIME-Version: 1.0 Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" Add a routine to DxeServicesLib that abstracts the allocation of memory that should be accessible by PEI after a warm reboot. We will use it to replace open coded implementations that limit the address to < 4 GB, which may not be possible on non-Intel systems that have no 32-bit addressable memory at all. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- MdePkg/Include/Library/DxeServicesLib.h | 23 +++++++- MdePkg/Library/DxeServicesLib/DxeServicesLib.c | 55 ++++++++++++++++++++ MdePkg/Library/DxeServicesLib/DxeServicesLib.inf | 3 ++ 3 files changed, 80 insertions(+), 1 deletion(-) -- 2.17.0 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel Reviewed-by: Laszlo Ersek diff --git a/MdePkg/Include/Library/DxeServicesLib.h b/MdePkg/Include/Library/DxeServicesLib.h index 7c1c62236d96..20aee68af558 100644 --- a/MdePkg/Include/Library/DxeServicesLib.h +++ b/MdePkg/Include/Library/DxeServicesLib.h @@ -305,5 +305,26 @@ GetFileDevicePathFromAnyFv ( OUT EFI_DEVICE_PATH_PROTOCOL **FvFileDevicePath ); -#endif +/** + Allocates one or more 4KB pages of a given type from a memory region that is + accessible to PEI. + + Allocates the number of 4KB pages of type 'MemoryType' and returns a + pointer to the allocated buffer. The buffer returned is aligned on a 4KB + boundary. If Pages is 0, then NULL is returned. If there is not enough + memory remaining to satisfy the request, then NULL is returned. + @param[in] MemoryType The memory type to allocate + @param[in] Pages The number of 4 KB pages to allocate. + + @return A pointer to the allocated buffer or NULL if allocation fails. + +**/ +VOID * +EFIAPI +AllocatePeiAccessiblePages ( + IN EFI_MEMORY_TYPE MemoryType, + IN UINTN Pages + ); + +#endif diff --git a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c index 1827c9216fbc..6719aabe5d04 100644 --- a/MdePkg/Library/DxeServicesLib/DxeServicesLib.c +++ b/MdePkg/Library/DxeServicesLib/DxeServicesLib.c @@ -16,6 +16,7 @@ #include #include +#include #include #include #include @@ -1093,3 +1094,57 @@ Done: return Status; } + +/** + Allocates one or more 4KB pages of a given type from a memory region that is + accessible to PEI. + + Allocates the number of 4KB pages of type 'MemoryType' and returns a + pointer to the allocated buffer. The buffer returned is aligned on a 4KB + boundary. If Pages is 0, then NULL is returned. If there is not enough + memory remaining to satisfy the request, then NULL is returned. + + @param[in] MemoryType The memory type to allocate + @param[in] Pages The number of 4 KB pages to allocate. + + @return A pointer to the allocated buffer or NULL if allocation fails. + +**/ +VOID * +EFIAPI +AllocatePeiAccessiblePages ( + IN EFI_MEMORY_TYPE MemoryType, + IN UINTN Pages + ) +{ + EFI_STATUS Status; + EFI_ALLOCATE_TYPE AllocType; + EFI_PHYSICAL_ADDRESS Memory; +#ifdef MDE_CPU_X64 + EFI_HOB_HANDOFF_INFO_TABLE *PhitHob; +#endif + + if (Pages == 0) { + return NULL; + } + + AllocType = AllocateAnyPages; +#ifdef MDE_CPU_X64 + // + // On X64 systems, a X64 build of DXE may be combined with a 32-bit build of + // PEI, and so we need to check the memory limit set by PEI, and allocate + // below 4 GB if the limit is set to 4 GB or lower. + // + PhitHob = (EFI_HOB_HANDOFF_INFO_TABLE *)GetHobList (); + if (PhitHob->EfiFreeMemoryTop <= MAX_UINT32) { + AllocType = AllocateMaxAddress; + Memory = MAX_UINT32; + } +#endif + + Status = gBS->AllocatePages (AllocType, MemoryType, Pages, &Memory); + if (EFI_ERROR (Status)) { + return NULL; + } + return (VOID *)(UINTN)Memory; +} diff --git a/MdePkg/Library/DxeServicesLib/DxeServicesLib.inf b/MdePkg/Library/DxeServicesLib/DxeServicesLib.inf index bd2faf2f6f2d..b0306c09872f 100644 --- a/MdePkg/Library/DxeServicesLib/DxeServicesLib.inf +++ b/MdePkg/Library/DxeServicesLib/DxeServicesLib.inf @@ -44,6 +44,9 @@ [LibraryClasses] UefiLib UefiBootServicesTableLib +[LibraryClasses.common.X64] + HobLib + [Guids] gEfiFileInfoGuid ## SOMETIMES_CONSUMES ## UNDEFINED