qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>, qemu-devel@nongnu.org
Subject: Re: [PULL 2/3] seabios: turn off CONFIG_APMBIOS for 128k build
Date: Tue, 12 Sep 2023 15:55:03 +0200	[thread overview]
Message-ID: <5c35d3f5-91ab-4340-0363-04494bb6881b@redhat.com> (raw)
In-Reply-To: <20230912105359.563101-3-kraxel@redhat.com>

On 9/12/23 12:53, Gerd Hoffmann wrote:
> Needed to make the build fit into 128k.
> 
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>   roms/config.seabios-128k | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/roms/config.seabios-128k b/roms/config.seabios-128k
> index d18c802c46e9..bc3ba7f65bcd 100644
> --- a/roms/config.seabios-128k
> +++ b/roms/config.seabios-128k
> @@ -19,3 +19,4 @@ CONFIG_USE_SMM=n
>   CONFIG_VGAHOOKS=n
>   CONFIG_HOST_BIOS_GEOMETRY=n
>   CONFIG_ACPI_PARSE=n
> +CONFIG_APMBIOS=n

Why don't we instead drop ``pc-i440fx-1.4`` up to ``pc-i440fx-1.7``, 
which have been deprecated since 7.0?  We cannot get rid of the 128k ROM 
because isapc uses it, but we can remove PCI and ACPI from it, and solve 
the size problem once and for all.

Paolo



  reply	other threads:[~2023-09-12 13:55 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-12 10:53 [PULL 0/3] Firmware/seabios 20230912 patches Gerd Hoffmann
2023-09-12 10:53 ` [PULL 1/3] seabios: update submodule to git snapshot Gerd Hoffmann
2023-09-12 10:53 ` [PULL 2/3] seabios: turn off CONFIG_APMBIOS for 128k build Gerd Hoffmann
2023-09-12 13:55   ` Paolo Bonzini [this message]
2023-09-18  9:50     ` Gerd Hoffmann
2023-09-12 10:53 ` [PULL 3/3] seabios: update binaries to git snapshot Gerd Hoffmann
2023-09-12 12:56 ` [PULL 0/3] Firmware/seabios 20230912 patches Stefan Hajnoczi
2023-09-18  9:59   ` Gerd Hoffmann
2023-09-18 12:52     ` Stefan Hajnoczi
2023-09-18 13:05       ` Daniel P. Berrangé
2023-09-18 13:40         ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5c35d3f5-91ab-4340-0363-04494bb6881b@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).