public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
To: Jeremy Katz <katzj-H+wXaHxf7aLQT0dZR+AlfA@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 15:19:42 -0500	[thread overview]
Message-ID: <46E849DE.3040103@codemonkey.ws> (raw)
In-Reply-To: <1189626953.16586.6.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>

Very cool!

My mailer doesn't want to quote your patch, but I noticed the following:

> +
> +    /* set the default based on the keypress or menu */
> +    switch(scan) {
> +    case 0x3D:
> +        outb_cmos(0x3d, 0x01);
> +        break;
> +    case 0x3E:
> +        outb_cmos(0x3d, 0x02);
> +        break;
> +    case 0x3F:
> +        outb_cmos(0x3d, 0x03);
> +        break;
> +    case 0x58:
> +        outb_cmos(0x3d, 0x04);
> +        break;
> +    default:
> +        break;
> +    }
> +
> +    /* Restore PIT ticks */
> +    wait_uninit();
> +}

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.

Regards,

Anthony Liguori

Jeremy Katz wrote:
> I sent this to the bochs list earlier today, but given that kvm is
> already carrying patches for the BIOS, it may be worthwhile/interesting
> to add this also as it can make the user experience substantially nicer.
>
> -- Begin forwarded message --
>
> The attached patch adds support for a relatively basic boot device
> selection menu to the bochs bios code.  
>
> Instead of immediately booting from the boot device set in the cmos, we
> wait for 3 seconds for the user to press F10; if they press it, then we
> show a basic boot menu that they can select what device to boot from.
> Otherwise, we continue on with what was setup before running the virtual
> machine.  The advantage is that users can change their boot device just
> on rebooting a virtual machine rather than having to stop and then
> restart it.
>
> This includes the wait routines added by VirtualBox
> (http://www.virtualbox.org) in their modifications to the rombios as
> they made things a bit easier.
>
> 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/
> ------------------------------------------------------------------------
>
> _______________________________________________
> kvm-devel mailing list
> kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> https://lists.sourceforge.net/lists/listinfo/kvm-devel


-------------------------------------------------------------------------
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:19 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 [this message]
     [not found]     ` <46E849DE.3040103-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
2007-09-12 20:25       ` Jeremy Katz
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=46E849DE.3040103@codemonkey.ws \
    --to=anthony-rdkfgonbjusknkdkm+me6a@public.gmane.org \
    --cc=katzj-H+wXaHxf7aLQT0dZR+AlfA@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