From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.100; helo=mga07.intel.com; envelope-from=jordan.l.justen@intel.com; receiver=edk2-devel@lists.01.org Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (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 641E82035686F for ; Sat, 11 Nov 2017 12:34:18 -0800 (PST) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Nov 2017 12:38:22 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,379,1505804400"; d="scan'208";a="148638140" Received: from sacolema-mobl2.amr.corp.intel.com (HELO localhost) ([10.254.117.1]) by orsmga004.jf.intel.com with ESMTP; 11 Nov 2017 12:38:22 -0800 MIME-Version: 1.0 To: Laszlo Ersek , edk2-devel-01 Message-ID: <151043270153.17841.16763408160801933614@jljusten-skl> From: Jordan Justen In-Reply-To: <20171110154908.306-1-lersek@redhat.com> Cc: Ard Biesheuvel , Ruiyu Ni References: <20171110154908.306-1-lersek@redhat.com> User-Agent: alot/0.6 Date: Sat, 11 Nov 2017 12:38:21 -0800 Subject: Re: [PATCH 0/4] OvmfPkg: measure temp stack usage, restore temp RAM to 64KB 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: , X-List-Received-Date: Sat, 11 Nov 2017 20:34:18 -0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On 2017-11-10 07:49:04, Laszlo Ersek wrote: > The first three patches enable the PEI_CORE to report OVMF's temp > SEC/PEI stack and heap usage. > = > - This depends on the new fixed PCD "PcdInitValueInTempStack", > recently added for > > ("INIT_CAR_VALUE should be defined in a central location"). > = > - Ard recently implemented the same in ArmPlatformPkg, for > ("measure temp > SEC/PEI stack usage"). > = > The last (fourth) patch adapts OVMF to the larger MtrrLib stack demand > that originates from commit 2bbd7e2fbd4b ("UefiCpuPkg/MtrrLib: Update > algorithm to calculate optimal settings", 2017-09-27). OVMF's temp RAM > size is restored to the historical / original 64KB. > = > This work is tracked in > ("measure temp > SEC/PEI stack usage; grow temp SEC/PEI RAM"), with links to related > mailing list discussions. > = > Repo: https://github.com/lersek/edk2.git > Branch: temp_ram_tweaks > = > Cc: Ard Biesheuvel > Cc: Jordan Justen > Cc: Ruiyu Ni > = > Thanks > Laszlo > = > Laszlo Ersek (4): > OvmfPkg/Sec/Ia32: free up EAX for other uses while setting up the > stack > OvmfPkg/Sec/Ia32: seed the temporary RAM with PcdInitValueInTempStack > OvmfPkg/Sec/X64: seed the temporary RAM with PcdInitValueInTempStack I'd like to try a different option for these 3. Can you hold off a bit before pushing this series? -Jordan > OvmfPkg: restore temporary SEC/PEI RAM size to 64KB > = > OvmfPkg/OvmfPkgIa32.fdf | 2 +- > OvmfPkg/OvmfPkgIa32X64.fdf | 2 +- > OvmfPkg/OvmfPkgX64.fdf | 2 +- > OvmfPkg/Sec/SecMain.inf | 1 + > OvmfPkg/Sec/Ia32/SecEntry.nasm | 19 ++++++++++++++++--- > OvmfPkg/Sec/X64/SecEntry.nasm | 15 +++++++++++++++ > 6 files changed, 35 insertions(+), 6 deletions(-) > = > -- = > 2.14.1.3.gb7cf6e02401b >=20