From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=66.187.233.73; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id F34282063D741 for ; Tue, 22 May 2018 12:11:36 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 2F3B8401EF08; Tue, 22 May 2018 19:11:36 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-18.rdu2.redhat.com [10.10.120.18]) by smtp.corp.redhat.com (Postfix) with ESMTP id BCD48111DD04; Tue, 22 May 2018 19:11:34 +0000 (UTC) To: Ard Biesheuvel , "Kinney, Michael D" Cc: "edk2-devel@lists.01.org" , Leif Lindholm , "Gao, Liming" , "Zeng, Star" , "Dong, Eric" , "Bi, Dandan" References: <20180522140850.30369-1-ard.biesheuvel@linaro.org> <20180522140850.30369-4-ard.biesheuvel@linaro.org> From: Laszlo Ersek Message-ID: Date: Tue, 22 May 2018 21:11:33 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.78 on 10.11.54.3 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.5]); Tue, 22 May 2018 19:11:36 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.5]); Tue, 22 May 2018 19:11:36 +0000 (UTC) for IP:'10.11.54.3' DOMAIN:'int-mx03.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'lersek@redhat.com' RCPT:'' Subject: Re: [PATCH 3/6] MdePkg/UefiLib: introduce EfiAllocatePeiAccessiblePages 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: , X-List-Received-Date: Tue, 22 May 2018 19:11:37 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 05/22/18 19:47, Ard Biesheuvel wrote: > OK, to summarize: > - move the implementation of EfiAllocatePeiAccessiblePages() to > DxeServicesLib (and perhaps rename it to something more appropriate > for its new home) > - only restrict the X64 version to below 4 GB if EfiMemoryTop and > EfiFreeMemoryTop are both below 4 GB. If this works with pure X64 OVMF (including S3 without SMM), I'd be very happy with it. Also we should regression test whether IA32X64 OVMF continues working (including S3 without SMM, and S3 with SMM). I'm glad to help with the testing once patches are posted. OVMF installs the permanent PEI RAM in PublishPeiMemory() [OvmfPkg/PlatformPei/MemDetect.c]. It keeps things under 4GB, so I think this approach should work. Thanks Ard & Mike! Laszlo