All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin O'Connor <kevin@koconnor.net>
To: Igor Mammedov <imammedo@redhat.com>
Cc: "Thomas Huth" <thuth@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	seabios@seabios.org, "László Érsek" <lersek@redhat.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [SeaBIOS]  Minimum RAM size for PC machines?
Date: Wed, 22 Mar 2017 11:18:10 -0400	[thread overview]
Message-ID: <20170322151810.GA17564@morn.lan> (raw)
In-Reply-To: <20170322132549.3d7e2a49@nial.brq.redhat.com>

On Wed, Mar 22, 2017 at 01:25:49PM +0100, Igor Mammedov wrote:
> On Wed, 22 Mar 2017 11:03:44 +0100
> Thomas Huth <thuth@redhat.com> wrote:
> > On 22.03.2017 10:08, Markus Armbruster wrote:
> > > Are we now ready to accept a simple & stupid patch that actually helps
> > > users, say letting boards that care declare minimum and maximum RAM
> > > size?  And make PC reject RAM size less than 1MiB, even though "someone"
> > > might conceivably have firmware that works with less?  
> > 
> > I'd say enforce a minimum RAM size on the normal "pc" and "q35" machine,
> > but still allow smaller sizes on the "isapc" machine. So if "someone"
> > comes around and claims to have a legacy firmware that wants less memory
> > than 1MiB, just point them to the isapc machine.
> > Just my 0.02 €.
> 
> We can print warning that minimum size will be enforced in 1-2
> releases since 2.9/10 and when it's enforced send users that need
> less to an old qemu version.

I did not think that QEMU ever worked on an x86 machine with less than
1MiB of ram.  Certainly SeaBIOS has never supported it.  I'm pretty
sure that Bochs BIOS (qemu < v0.12) did not support it either - its
ram detection system assumed a minimum of 1MiB.

My $.02 - I agree with Markus - place a minimum limit of 1MiB on x86
machines.  If someone is writing a new firmware, updating QEMU to
change the limit is the very least of their concerns.

-Kevin

      reply	other threads:[~2017-03-22 15:18 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-22  9:08 [Qemu-devel] Minimum RAM size for PC machines? Markus Armbruster
2017-03-22 10:03 ` Thomas Huth
2017-03-22 10:19   ` David Hildenbrand
2017-03-23  8:19     ` Gerd Hoffmann
2017-03-23  9:18       ` David Hildenbrand
2017-03-22 12:25   ` Igor Mammedov
2017-03-22 15:18     ` Kevin O'Connor [this message]

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=20170322151810.GA17564@morn.lan \
    --to=kevin@koconnor.net \
    --cc=armbru@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=lersek@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=seabios@seabios.org \
    --cc=thuth@redhat.com \
    /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.