All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [PULL 2/3] seabios: turn off CONFIG_APMBIOS for 128k build
Date: Mon, 18 Sep 2023 11:50:48 +0200	[thread overview]
Message-ID: <oukf6qzukn5fdu3indrx3fuytkwl4aj7pncipmnchexcg6ffte@ewnnijlr4oab> (raw)
In-Reply-To: <5c35d3f5-91ab-4340-0363-04494bb6881b@redhat.com>

On Tue, Sep 12, 2023 at 03:55:03PM +0200, Paolo Bonzini wrote:
> 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.

I happily update the seabios config once the old machine types are gone.

take care,
  Gerd



  reply	other threads:[~2023-09-18  9:51 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
2023-09-18  9:50     ` Gerd Hoffmann [this message]
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=oukf6qzukn5fdu3indrx3fuytkwl4aj7pncipmnchexcg6ffte@ewnnijlr4oab \
    --to=kraxel@redhat.com \
    --cc=pbonzini@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.