From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.groups.io with SMTP id smtpd.web10.56875.1622550032224268740 for ; Tue, 01 Jun 2021 05:20:32 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=PEQzZUgW; spf=pass (domain: redhat.com, ip: 170.10.133.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1622550031; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=+3HQxHbJtTw4LIZ9K8JWwJ/cYA2erWOT5flDwt1Nfhc=; b=PEQzZUgWYI3qDpz8GrvAGIpCWLka1IiR3+JwhGmBtAXzQcng3zJcFMrw1WfWVcaP6UC+Y3 clxE6GkIMflikD5UVQA0tp1mEZT8iGS8Us5TdM/zI3J2E+RhoIXWPYhVHaY5N/hXwoY5xQ TDLqOHkEhuyBpc5CE9BECzQr6W7Eo+8= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-586-yg6njMrjMR27w9cTBAC8Gw-1; Tue, 01 Jun 2021 08:20:29 -0400 X-MC-Unique: yg6njMrjMR27w9cTBAC8Gw-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 30F13501E3; Tue, 1 Jun 2021 12:20:27 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-114-181.ams2.redhat.com [10.36.114.181]) by smtp.corp.redhat.com (Postfix) with ESMTPS id D4AF460C0F; Tue, 1 Jun 2021 12:20:24 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH v1] OvmfPkg: Add build options for 8MB and 16MB X64 OVMF images To: devel@edk2.groups.io, brian.johnson@hpe.com, dbautista@newmexicoconsortium.org Cc: Brijesh Singh , Erdem Aktas , James Bottomley , Jiewen Yao , Min Xu , Tom Lendacky References: <7bfd4b82fc725302beb37e13c4a89d389c34ec34.1622048433.git.dbautista@newmexicoconsortium.org> <44af78f5-dc76-3d86-6021-488dcfaff52c@hpe.com> From: "Laszlo Ersek" Message-ID: Date: Tue, 1 Jun 2021 14:20:23 +0200 MIME-Version: 1.0 In-Reply-To: <44af78f5-dc76-3d86-6021-488dcfaff52c@hpe.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit On 05/28/21 15:33, Brian J. Johnson wrote: > On 5/27/21 5:12 AM, Laszlo Ersek wrote: >> On 05/26/21 19:08, Devon Bautista wrote: >>> Currently, the largest volume size for building OVMF images is 4MB. With >>> the growth of the Linuxboot project, maintainers have had to maintain a >>> fork containing this patch which allows larger image sizes in order for >>> Linuxboot developers/users to have enough space to experiment with >>> and test including their own Linux kernel in the DXE section of OVMF >>> firmware. Testing using OVMF is valuable since it allows testing in QEMU >>> and thus does not require any hardware to do so. >>> >>> This patch allows specifying '-D FD_SIZE_8MB' or '-D FD_SIZE_16MB' to >>> the OVMF build script in order to add the ability to build 8MB or 16MB >>> x86_64 (X64) OVMF images, respectively. >>> >>> Signed-off-by: Devon Bautista >>> --- >>>   OvmfPkg/OvmfPkgDefines.fdf.inc | 34 ++++++++++++++++++++++++++++++++++ >>>   OvmfPkg/OvmfPkgX64.dsc         | 10 +++++++++- >>>   OvmfPkg/VarStore.fdf.inc       | 16 ++++++++-------- >>>   3 files changed, 51 insertions(+), 9 deletions(-) >> (4) Dumping a bunch of magic numbers on reviewers is unhelpful. I'll >> need to sit down with a calculator and go through the patch with a >> magnifying glass. Please support that work by creating a commit message >> (summary table) similar to the one in commit b24fca05751f ("OvmfPkg: >> introduce 4MB flash image (mainly) for Windows HCK", 2017-05-05). >> > > I've found it very helpful to create a spreadsheet to calculate the > offsets based on the region sizes, and add it to the tree.  It can also > calculate the related PCDs, if any.  That makes it a lot easier to > verify the numbers, and to make changes in the future. This sounds really nice -- I'd appreciate such a spreadsheet in the OvmfPkg directory tree somewhere. My concern is that most (all?) spreadsheet formats are compressed archives one way or another (ZIP archives of multiple files, or gzipped XML files, or some such), and such formats are not nice to track in git. I'd like a (structured) plaintext representation of the spreadsheet to live in edk2 git. Same as we prefer plaintext SVG files for graphics / diagrams in edk2, to my knowledge. Thanks! Laszlo