qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gleb Natapov <gleb@redhat.com>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: qemu-devel@nongnu.org, Alex Graf <agraf@suse.de>,
	Anthony Liguori <aliguori@us.ibm.com>,
	Avik Sil <aviksil@linux.vnet.ibm.com>,
	David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [Qemu-devel] nvram and boot order
Date: Wed, 17 Oct 2012 15:48:42 +0200	[thread overview]
Message-ID: <20121017134842.GR20788@redhat.com> (raw)
In-Reply-To: <1350418073.4678.58.camel@pasglop>

On Wed, Oct 17, 2012 at 07:07:53AM +1100, Benjamin Herrenschmidt wrote:
> On Tue, 2012-10-16 at 14:55 -0500, Anthony Liguori wrote:
> > 
> > 4) If -boot is specified, the parameter should alter the contents of
> >    NVRAM to change the boot order to what is specified by -boot.
> > 
> > 5) If ,bootorder is specified, it should take predence over -boot.
> > 
> > 6) ,bootorder= should also alter the contents of NVRAM to determine
> > the
> >    boot order
> 
> That's where I disagree. At least for us... I don't see why -boot or
> -bootorder should alter the nvram content.
> 
> The plan is to have the nvram content essentially in control of SLOF
> (ie. the BIOS in x86 land). Qemu doesn't know much of anything appart
> from providing the storage for it.
> 
> I see -boot and -bootorder as ways to *temporarily* override whatever
> setting was put in nvram by the guest. The nvram is typically populated
> by the distro installer (though sometimes by hand by the user). One may
> want to just temporarily boot from a CD (rescue for example, or to test
> a live CD or something ...), that doesn't mean the permanent setting
> should be altered.
> 
> I would pass -boot and -bootorder to SLOF like we pass the current
> bootlist today and let it deal with it, I wouldn't touch the nvram.
> 
Agree with you and David. This is not QEMU busyness to touch nvram
content. For all we know it may be encrypted by a key knows only to
the firmware.

--
			Gleb.

  parent reply	other threads:[~2012-10-17 13:50 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-16 19:55 [Qemu-devel] nvram and boot order Anthony Liguori
2012-10-16 20:07 ` Benjamin Herrenschmidt
2012-10-16 20:55   ` Anthony Liguori
2012-10-26  6:27     ` David Gibson
2012-10-17 13:48   ` Gleb Natapov [this message]
2012-10-16 20:13 ` Peter Maydell
2012-10-16 23:12   ` Anthony Liguori
2012-10-17  1:01     ` David Gibson
2012-10-17  0:58 ` David Gibson
2012-10-17 18:17   ` Anthony Liguori
2012-10-18  0:09     ` David Gibson
2012-10-18  1:18       ` Benjamin Herrenschmidt
2012-10-18  6:32         ` Alexander Graf
2012-10-19  8:24           ` David Gibson
2012-10-19  8:40             ` Alexander Graf
2012-10-25  4:14               ` David Gibson
2012-10-19 14:21             ` Anthony Liguori
2012-10-19 14:36               ` Peter Maydell
2012-10-19 18:11               ` Benjamin Herrenschmidt
2012-10-19 17:29           ` Blue Swirl

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=20121017134842.GR20788@redhat.com \
    --to=gleb@redhat.com \
    --cc=agraf@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=aviksil@linux.vnet.ibm.com \
    --cc=benh@kernel.crashing.org \
    --cc=david@gibson.dropbear.id.au \
    --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).