From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, HTML_MESSAGE,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9EAFBC43461 for ; Tue, 15 Sep 2020 19:10:59 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id DDDBD20770 for ; Tue, 15 Sep 2020 19:10:58 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=hp.com header.i=@hp.com header.b="SF5oDL09" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DDDBD20770 Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=hp.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:56220 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kIGLx-0006lk-Sx for qemu-devel@archiver.kernel.org; Tue, 15 Sep 2020 15:10:57 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42782) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kIGKY-00060d-HW for qemu-devel@nongnu.org; Tue, 15 Sep 2020 15:09:30 -0400 Received: from us-smtp-delivery-162.mimecast.com ([216.205.24.162]:41958) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1kIGKV-0001IG-0a for qemu-devel@nongnu.org; Tue, 15 Sep 2020 15:09:29 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hp.com; s=mimecast20180716; t=1600196965; 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: in-reply-to:in-reply-to:references:references; bh=4zglPiw/v8P+wjMHe07/ttJh8pTNUPZJdbB10frjf98=; b=SF5oDL09Xl5r6NYLKpy6sPPg8cUL4Bm2E9i8XarUPhAJxcvxdOcodAlrfEMgTfbWbd5V1Q yHL2eklGgzTzVhWAyJMM8lMAaP5hfB4ZrMM06eyhPunCzMaQeQpE1/wM8Enlj3Eg/McIIO /Z1eV1+jMczhIG1ciw22ku65oieX4jQ= Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12lp2174.outbound.protection.outlook.com [104.47.55.174]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-471-UbHys-DgPWKeYvFTvyYjCQ-1; Tue, 15 Sep 2020 15:09:21 -0400 X-MC-Unique: UbHys-DgPWKeYvFTvyYjCQ-1 Received: from CS1PR8401MB0327.NAMPRD84.PROD.OUTLOOK.COM (2a01:111:e400:7507::9) by CS1PR8401MB0965.NAMPRD84.PROD.OUTLOOK.COM (2a01:111:e400:7508::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3370.16; Tue, 15 Sep 2020 19:09:20 +0000 Received: from CS1PR8401MB0327.NAMPRD84.PROD.OUTLOOK.COM ([fe80::40df:1118:3bef:736c]) by CS1PR8401MB0327.NAMPRD84.PROD.OUTLOOK.COM ([fe80::40df:1118:3bef:736c%12]) with mapi id 15.20.3370.019; Tue, 15 Sep 2020 19:09:20 +0000 From: "McMillan, Erich" To: "qemu-devel@nongnu.org" CC: "mst@redhat.com" , "marcel.apfelbaum@gmail.com" , "qemu-trivial@nongnu.org" Subject: Re: PATCH: Increase System Firmware Max Size Thread-Topic: PATCH: Increase System Firmware Max Size Thread-Index: AQHWh9wbvH3PA15ju0WtYVYwRPIQ7qlqFivM Date: Tue, 15 Sep 2020 19:09:20 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [76.199.17.212] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: fa50ddc9-d421-476b-acaa-08d859aad9df x-ms-traffictypediagnostic: CS1PR8401MB0965: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:10000; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: T0dCzCREVFMo1P6GyK/Jn9YHYUlEtZzDs5xZfrgsY1S4jRNDmZqHJYT4Do/vIRqM2IYSlx+apA3FytA1ah6FtJNdIh+Wb8hf5dag8pMcjMcVITG329BK/eTrjkAoKM2cWD1AZqSajUb5ozwrH2OtzCChlgsVTlqgMBdBdivgWVM1DmXThlryNmZPIVnPG9mgU4IwCn3eCVua0oJfJfyNz/+93q9uwrvukfFj4N6gJfW367Nyg0J5GZcv84NHxUWhG7W1HnbE0GzFCEdnTrPwZAgI50BRTcFe7XLNnbrQvCAeZHf3jDi7yXxdGAqgmVwWIe3PGM8kLlg6CAmT344YKQ== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CS1PR8401MB0327.NAMPRD84.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(396003)(136003)(366004)(39860400002)(346002)(376002)(316002)(7696005)(54906003)(478600001)(2906002)(9686003)(55016002)(8936002)(4326008)(6506007)(53546011)(71200400001)(76116006)(26005)(66946007)(6916009)(186003)(19627405001)(52536014)(66476007)(64756008)(66556008)(8676002)(5660300002)(86362001)(33656002)(66446008)(83380400001); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata: w1QwxFQsi1Lj5e+dV7pkAnWu8WxXLvT7sbPcI8WiisJrgvo90oILuGETl8XED3S77fEX+RtTG8s4YbiktHz+F1kaQw1Nb/yqQ4e6wTeTi/z+FUYLKzEdArp2bOQFtlMijwsqfFu+ewPWWFx7rFFCesJ9lMnZK/VP6J5ho1avoqOQ5GFsC382eCqaDkWSj+WgzJ+Y5Kc/EoteBwSPiokV3T5txBf2jJPxeVrkuhmUHeepypMfP5GqHm52AX/5z5Jsdb0IlTQsy0DUDSkJkX5TGPLyRveum00TZ+YpUvg7Iut/NIr225ZU4lvk5an59usotqMZ+orit3IBDE5fg1l9DokkonIaA6ZKHw8wsT5gLX/HvC0nnizcXlC7nFZIpgLQUuK6Z0HkrLyr6XMD7PnO4Uyk2QBaNCS1gEjnKHODlF3jaKuMLkdhA4kfMLFakueD55oVgQpYnwXquavWKnYb6Qzi0eqPWaHVN3GWiGNEPEIWVU7zA4uHPEIGY+BqUVQshv2SDuslUimwixD2nsE5VA+9WmA0h/pae1KyzPOKcxH4+jqL9nluTpew2DE8IhQPddrAMA7ArNY6Zi5wFmhf03KbSTXe55N7mue/lxEMF6RVWKXPa1HhBvICUKy78JWTtf2uJDXcs1zlQEREe55lpA== x-ms-exchange-transport-forked: True MIME-Version: 1.0 X-OriginatorOrg: hp.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CS1PR8401MB0327.NAMPRD84.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: fa50ddc9-d421-476b-acaa-08d859aad9df X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Sep 2020 19:09:20.6222 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: ca7981a2-785a-463d-b82a-3db87dfc3ce6 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: L1X4fJvlhQNC6V7SV8yFqOe8e+jlHraJInwysCKUScKiG8dyVrTZkWwUWGvtGPKIDy9Er8JspLUMO+U/al9LEg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CS1PR8401MB0965 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA62A171 smtp.mailfrom=erich.mcmillan@hp.com X-Mimecast-Spam-Score: 0.003 X-Mimecast-Originator: hp.com Content-Type: multipart/alternative; boundary="_000_CS1PR8401MB0327959D96C84FB32E071E49F3200CS1PR8401MB0327_" Content-Language: en-US Received-SPF: pass client-ip=216.205.24.162; envelope-from=erich.mcmillan@hp.com; helo=us-smtp-delivery-162.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/09/15 15:09:25 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -50 X-Spam_score: -5.1 X-Spam_bar: ----- X-Spam_report: (-5.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-2.999, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" --_000_CS1PR8401MB0327959D96C84FB32E071E49F3200CS1PR8401MB0327_ Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable Hi all, I've rewritten the FLASH_SIZE_LIMIT as a command line parameter as requeste= d, but I'd like some feedback. My current concerns are: 1. I'm not too happy using an global variable in this manner, but I'm no= t sure the appropriate way to share this information between vl.c and pc_sy= sfw.c. Is there a way for other .c modules to query the QemuOpts, or is thi= s not preferred. 2. Would appreciate feedback on the help strings, I think the formatting= is correct based on -m (memory size) flag. 3. If global variable is acceptable then is it appropriate for it to be = shared via loader.h, this seemed the most appropriate place to put it witho= ut adding new includes to either vl.c or pc_sysfw.c. Thank you, Erich diff --git a/softmmu/vl.c b/softmmu/vl.c index 05d1a4cb6bf863b6ac1df8f94694c073fa4f8d90..a34995819fa14ef728d82ab179e= f3a2e468e2365 100644 --- a/softmmu/vl.c +++ b/softmmu/vl.c @@ -442,6 +442,20 @@ static QemuOptsList qemu_mem_opts =3D { }, }; +static QemuOptsList qemu_max_fw_size_opts =3D { + .name =3D "maxfirmwaresize", + .implied_opt_name =3D "fwsize", + .head =3D QTAILQ_HEAD_INITIALIZER(qemu_max_fw_size_opts.head), + .merge_lists =3D true, + .desc =3D { + { + .name =3D "size", + .type =3D QEMU_OPT_SIZE, + }, + { /* end of list */ } + }, +}; + static QemuOptsList qemu_icount_opts =3D { .name =3D "icount", .implied_opt_name =3D "shift", @@ -2559,6 +2573,23 @@ static bool object_create_delayed(const char *type, = QemuOpts *opts) return !object_create_initial(type, opts); } +static void set_max_firmware_size(uint64_t *maxfwsize) +{ + const char *max_fw_size_str; + QemuOpts *opts =3D qemu_find_opts_singleton("maxfirmwaresize"); + + max_fw_size_str =3D qemu_opt_get(opts, "size"); + + if (max_fw_size_str) { + if (!*max_fw_size_str) { + error_report("missing 'size' option value"); + exit(EXIT_FAILURE); + } + + *maxfwsize =3D qemu_opt_get_size(opts, "size", 8 * MiB); + } +} + static bool set_memory_options(uint64_t *ram_slots, ram_addr_t *maxram_siz= e, MachineClass *mc) @@ -2887,6 +2918,7 @@ void qemu_init(int argc, char **argv, char **envp) qemu_add_opts(&qemu_machine_opts); qemu_add_opts(&qemu_accel_opts); qemu_add_opts(&qemu_mem_opts); + qemu_add_opts(&qemu_max_fw_size_opts); qemu_add_opts(&qemu_smp_opts); qemu_add_opts(&qemu_boot_opts); qemu_add_opts(&qemu_add_fd_opts); @@ -3143,6 +3175,10 @@ void qemu_init(int argc, char **argv, char **envp) exit(EXIT_FAILURE); } break; + case QEMU_OPTION_maxfirmwaresize: + opts =3D qemu_opts_parse_noisily(qemu_find_opts("maxfirmwa= resize"), + optarg, true); + break; #ifdef CONFIG_TPM case QEMU_OPTION_tpmdev: if (tpm_config_parse(qemu_find_opts("tpmdev"), optarg) < 0= ) { @@ -3831,6 +3867,8 @@ void qemu_init(int argc, char **argv, char **envp) have_custom_ram_size =3D set_memory_options(&ram_slots, &maxram_size, machine_class); + set_max_firmware_size(&MaxCombinedFirmwareSize); + os_daemonize(); /* diff --git a/include/hw/loader.h b/include/hw/loader.h index a9eeea39521d2d5b5e9b73e0fcbd4d4ce9292046..9e982cd60f8f2173a3160f56316= 7e48b7ca88aa9 100644 --- a/include/hw/loader.h +++ b/include/hw/loader.h @@ -318,4 +318,13 @@ int rom_add_option(const char *file, int32_t bootindex= ); * overflow on real hardware too. */ #define UBOOT_MAX_GUNZIP_BYTES (64 << 20) +/* + * We don't have a theoretically justifiable exact lower bound on the base + * address of any flash mapping. In practice, the IO-APIC MMIO range is + * [0xFEE00000..0xFEE01000] -- see IO_APIC_DEFAULT_ADDRESS --, leaving fre= e + * only 18MB-4KB below 4G. For now, restrict the cumulative mapping to 8MB= in + * size, but allow user to specify larger size via command line. + */ +uint64_t MaxCombinedFirmwareSize =3D (8 * MiB); + #endif diff --git a/hw/i386/pc_sysfw.c b/hw/i386/pc_sysfw.c index b8d8ef59eb17c6ace8194fc69c3d27809becfbc0..012c28a3b4de1c1618404faefd6= 3a99267636935 100644 --- a/hw/i386/pc_sysfw.c +++ b/hw/i386/pc_sysfw.c @@ -39,14 +39,8 @@ #include "hw/block/flash.h" #include "sysemu/kvm.h" -/* - * We don't have a theoretically justifiable exact lower bound on the base - * address of any flash mapping. In practice, the IO-APIC MMIO range is - * [0xFEE00000..0xFEE01000] -- see IO_APIC_DEFAULT_ADDRESS --, leaving fre= e - * only 18MB-4KB below 4G. For now, restrict the cumulative mapping to 8MB= in - * size. - */ -#define FLASH_SIZE_LIMIT (8 * MiB) + +extern MaxCombinedFirmwareSize; #define FLASH_SECTOR_SIZE 4096 @@ -177,10 +171,10 @@ static void pc_system_flash_map(PCMachineState *pcms, } if ((hwaddr)size !=3D size || total_size > HWADDR_MAX - size - || total_size + size > FLASH_SIZE_LIMIT) { + || total_size + size > MaxCombinedFirmwareSize) { error_report("combined size of system firmware exceeds " "%" PRIu64 " bytes", - FLASH_SIZE_LIMIT); + MaxCombinedFirmwareSize); exit(1); } ________________________________ From: McMillan, Erich Sent: Thursday, September 10, 2020 8:45 PM To: qemu-devel@nongnu.org Cc: mst@redhat.com ; marcel.apfelbaum@gmail.com ; qemu-trivial@nongnu.org Subject: PATCH: Increase System Firmware Max Size Hi all, (this is my first Qemu patch submission, please let me know if my formattin= g/content needs to be fixed). We have a need for increased firmware size, currently we are building Qemu = with the following change to test our Uefi Firmware and it works well for u= s. Hope that this change can be made to open source. Thank you. ------- Increase allowed system firmware size to 16M per comment suggesting up to 1= 8M is permissible. Signed-off-by: Erich McMillan diff --git a/hw/i386/pc_sysfw.c b/hw/i386/pc_sysfw.c index b8d8ef59eb17c6ace8194fc69c3d27809becfbc0..f6f7cd744d0690cee0355fbd19f= fdcdb71ea75ca 100644 --- a/hw/i386/pc_sysfw.c +++ b/hw/i386/pc_sysfw.c @@ -46,7 +46,7 @@ * only 18MB-4KB below 4G. For now, restrict the cumulative mapping to 8MB= in * size. */ -#define FLASH_SIZE_LIMIT (8 * MiB) +#define FLASH_SIZE_LIMIT (16 * MiB) #define FLASH_SECTOR_SIZE 4096 ------- --_000_CS1PR8401MB0327959D96C84FB32E071E49F3200CS1PR8401MB0327_ Content-Type: text/html; charset=WINDOWS-1252 Content-Transfer-Encoding: quoted-printable
Hi all,

I've rewritten the FLASH_SIZE_LIMIT as a command line parameter as requeste= d, but I'd like some feedback. My current concerns are:
  1. I'm not too happy using an global variable in this manner, but I'm not = sure the appropriate way to share this information between vl.c and pc_sysf= w.c. Is there a way for other .c modules to query the QemuOpts, or is this = not preferred.
  2. Would appreciate feedback on the help strings, I thi= nk the formatting is correct based on -m (memory size) flag.
  3. If glo= bal variable is acceptable then is it appropriate for it to be shared via l= oader.h, this seemed the most appropriate place to put it without adding ne= w includes to either vl.c or pc_sysfw.c.
Thank you,
Erich

diff --git a/softmmu/vl.c b/softmmu/vl.c
index 05d1a4cb6bf863b6ac1df8f94694c073fa4f8d90..a34995819fa14ef728d82a= b179ef3a2e468e2365 100644
--- a/softmmu/vl.c
+++ b/softmmu/vl.c
@@ -442,6 +442,20 @@ static QemuOptsList qemu_mem_opts =3D {
     },
 };
 
+static QemuOptsList qemu_max_fw_size_opts =3D {
+    .name =3D "maxfirmwaresize",
+    .implied_opt_name =3D "fwsize",
+    .head =3D QTAILQ_HEAD_INITIALIZER(qemu_max_fw_size_opts= .head),
+    .merge_lists =3D true,
+    .desc =3D {
+        {
+            .name =3D "size",=
+            .type =3D QEMU_OPT_SIZE,
+        },
+        { /* end of list */ }
+    },
+};
+
 static QemuOptsList qemu_icount_opts =3D {
     .name =3D "icount",
     .implied_opt_name =3D "shift",
@@ -2559,6 +2573,23 @@ static bool object_create_delayed(const char *t= ype, QemuOpts *opts)
     return !object_create_initial(type, opts);
 }
 
+static void set_max_firmware_size(uint64_t *maxfwsize)
+{
+    const char *max_fw_size_str;
+    QemuOpts *opts =3D qemu_find_opts_singleton("maxfi= rmwaresize");
+
+    max_fw_size_str =3D qemu_opt_get(opts, "size"= );
+
+    if (max_fw_size_str) {
+        if (!*max_fw_size_str) {
+            error_report("missing = 'size' option value");
+            exit(EXIT_FAILURE);
+        }
+
+        *maxfwsize =3D qemu_opt_get_size(opts, &q= uot;size", 8 * MiB);
+    }
+}
+
 
 static bool set_memory_options(uint64_t *ram_slots, ram_addr_t *= maxram_size,
                    =             MachineClass *mc)
@@ -2887,6 +2918,7 @@ void qemu_init(int argc, char **argv, char **env= p)
     qemu_add_opts(&qemu_machine_opts);
     qemu_add_opts(&qemu_accel_opts);
     qemu_add_opts(&qemu_mem_opts);
+    qemu_add_opts(&qemu_max_fw_size_opts);
     qemu_add_opts(&qemu_smp_opts);
     qemu_add_opts(&qemu_boot_opts);
     qemu_add_opts(&qemu_add_fd_opts);
@@ -3143,6 +3175,10 @@ void qemu_init(int argc, char **argv, char **en= vp)
                    =  exit(EXIT_FAILURE);
                 }
                 break;
+            case QEMU_OPTION_maxfirmwar= esize:
+                opts =3D qemu= _opts_parse_noisily(qemu_find_opts("maxfirmwaresize"),
+                    = ;                     &nb= sp;     optarg, true);
+                break;
 #ifdef CONFIG_TPM
             case QEMU_OPTION_tpmde= v:
                 if (tpm_= config_parse(qemu_find_opts("tpmdev"), optarg) < 0) {
@@ -3831,6 +3867,8 @@ void qemu_init(int argc, char **argv, char **env= p)
     have_custom_ram_size =3D set_memory_options(&r= am_slots, &maxram_size,
                    =                      = ;      machine_class);
 
+    set_max_firmware_size(&MaxCombinedFirmwareSize);
+
     os_daemonize();
 
     /*
diff --git a/include/hw/loader.h b/include/hw/loader.h
index a9eeea39521d2d5b5e9b73e0fcbd4d4ce9292046..9e982cd60f8f2173a3160f= 563167e48b7ca88aa9 100644
--- a/include/hw/loader.h
+++ b/include/hw/loader.h
@@ -318,4 +318,13 @@ int rom_add_option(const char *file, int32_t boot= index);
  * overflow on real hardware too. */
 #define UBOOT_MAX_GUNZIP_BYTES (64 << 20)
 
+/*
+ * We don't have a theoretically justifiable exact lower bound on the= base
+ * address of any flash mapping. In practice, the IO-APIC MMIO range = is
+ * [0xFEE00000..0xFEE01000] -- see IO_APIC_DEFAULT_ADDRESS --, leavin= g free
+ * only 18MB-4KB below 4G. For now, restrict the cumulative mapping t= o 8MB in
+ * size, but allow user to specify larger size via command line.
+ */
+uint64_t MaxCombinedFirmwareSize =3D (8 * MiB);
+
 #endif
diff --git a/hw/i386/pc_sysfw.c b/hw/i386/pc_sysfw.c
index b8d8ef59eb17c6ace8194fc69c3d27809becfbc0..012c28a3b4de1c1618404f= aefd63a99267636935 100644
--- a/hw/i386/pc_sysfw.c
+++ b/hw/i386/pc_sysfw.c
@@ -39,14 +39,8 @@
 #include "hw/block/flash.h"
 #include "sysemu/kvm.h"
 
-/*
- * We don't have a theoretically justifiable exact lower bound on the= base
- * address of any flash mapping. In practice, the IO-APIC MMIO range = is
- * [0xFEE00000..0xFEE01000] -- see IO_APIC_DEFAULT_ADDRESS --, leavin= g free
- * only 18MB-4KB below 4G. For now, restrict the cumulative mapping t= o 8MB in
- * size.
- */
-#define FLASH_SIZE_LIMIT (8 * MiB)
+
+extern MaxCombinedFirmwareSize;
 
 #define FLASH_SECTOR_SIZE 4096
 
@@ -177,10 +171,10 @@ static void pc_system_flash_map(PCMachineState *= pcms,
         }
         if ((hwaddr)size !=3D size
             || total_size > HWA= DDR_MAX - size
-            || total_size + size > F= LASH_SIZE_LIMIT) {
+            || total_size + size > M= axCombinedFirmwareSize) {
             error_report("com= bined size of system firmware exceeds "
                    =       "%" PRIu64 " bytes",
-                    = ;     FLASH_SIZE_LIMIT);
+                    = ;     MaxCombinedFirmwareSize);
             exit(1);
         }
 



From: McMillan, Erich
Sent: Thursday, September 10, 2020 8:45 PM
To: qemu-devel@nongnu.org <qemu-devel@nongnu.org>
Cc: mst@redhat.com <mst@redhat.com>; marcel.apfelbaum@gmail.co= m <marcel.apfelbaum@gmail.com>; qemu-trivial@nongnu.org <qemu-triv= ial@nongnu.org>
Subject: PATCH: Increase System Firmware Max Size
 
Hi all,

(this is my first Qemu patch submission, please let me know if my formattin= g/content needs to be fixed).
We have a need for increased firmware size, currently we are building Qemu = with the following change to test our Uefi Firmware and it works well for u= s. Hope that this change can be made to open source. Thank you.
-------
Increase allowed system firmware size to 16M per comment suggesting up to 1= 8M is permissible.

 Signed-off-by: Erich McMillan <erich.mcmillan@hp.com>

diff --git a/hw/i386/pc_sysfw.c b/hw/i386/pc_sysfw.c
index b8d8ef59eb17c6ace8194fc69c3d27809becfbc0..f6f7cd744d0690cee0355f= bd19ffdcdb71ea75ca 100644
--- a/hw/i386/pc_sysfw.c
+++ b/hw/i386/pc_sysfw.c
@@ -46,7 +46,7 @@
  * only 18MB-4KB below 4G. For now, restrict the cumulative mapp= ing to 8MB in
  * size.
  */
-#define FLASH_SIZE_LIMIT (8 * MiB)
+#define FLASH_SIZE_LIMIT (16 * MiB)
 
 #define FLASH_SECTOR_SIZE 4096
-------
 
--_000_CS1PR8401MB0327959D96C84FB32E071E49F3200CS1PR8401MB0327_--