public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Jeremy Katz <katzj-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Anthony Liguori <anthony-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
Cc: kvm-devel <kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Subject: Re: [PATCH] Add support for a basic boot menu to the	bios
Date: Wed, 12 Sep 2007 16:25:17 -0400	[thread overview]
Message-ID: <1189628717.16586.13.camel@localhost.localdomain> (raw)
In-Reply-To: <46E849DE.3040103-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>

On Wed, 2007-09-12 at 15:19 -0500, Anthony Liguori wrote:
> Very cool!

Long-term itch of mine scratched :-)

> My mailer doesn't want to quote your patch, but I noticed the following:
> 
> > +
> > +    /* set the default based on the keypress or menu */
[snip]
> But the CMOS memory isn't persisted in QEMU.  Another nice patch 
> (although certainly not required IMHO for this to go in), would be to 
> make nvram optionally persistent for QEMU so these settings would 
> persist across boots.

Yeah, the lack of persistence is sort of a "feature" in this case.  If
you use the similar menus on most real hardware, they don't persist (you
have to go into the bios setup screens to make it persist).  That said,
if things were to persist, then we'd probably want to change the patch
to set something else which was looked at prior to the CMOS

Jeremy


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

  parent reply	other threads:[~2007-09-12 20:25 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-12 19:55 [PATCH] Add support for a basic boot menu to the bios Jeremy Katz
     [not found] ` <1189626953.16586.6.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2007-09-12 20:19   ` Anthony Liguori
     [not found]     ` <46E849DE.3040103-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
2007-09-12 20:25       ` Jeremy Katz [this message]
2007-09-14 20:14   ` Anthony Liguori
2007-09-16 20:00   ` Avi Kivity
     [not found]     ` <46ED8B5B.40903-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-09-17 15:23       ` Luca
     [not found]         ` <68676e00709170823m3247c3eaj7d268166ecc8c013-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2007-09-17 17:30           ` Avi Kivity
2007-09-19 20:08       ` Jeremy Katz
     [not found]         ` <1190232486.10222.37.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2007-09-20  6:02           ` Avi Kivity

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=1189628717.16586.13.camel@localhost.localdomain \
    --to=katzj-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=anthony-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.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