All of lore.kernel.org
 help / color / mirror / Atom feed
From: Amos Kong <akong@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: dallan@redhat.com, gleb@redhat.com, seabios@seabios.org,
	qemu-devel@nongnu.org, alex.williamson@redhat.com,
	kevin@koconnor.net
Subject: Re: [Qemu-devel] [Resend][Seabios PATCH] don't boot from un-selected devices
Date: Wed, 19 Dec 2012 16:12:22 +0800	[thread overview]
Message-ID: <20121219081222.GA3078@t430s.nay.redhat.com> (raw)
In-Reply-To: <50D16EE5.1050105@redhat.com>

On Wed, Dec 19, 2012 at 08:38:13AM +0100, Gerd Hoffmann wrote:
> On 12/19/12 08:24, Amos Kong wrote:
> > Current seabios will try to boot from selected devices first,
> > if they are all failed, seabios will also try to boot from
> > un-selected devices.
> > 
> > For example:
> > @ qemu-kvm -boot order=n,menu=on ...
> > 
> > Guest will boot from network first, if it's failed, guest will try to
> > boot from other un-selected devices (floppy, cdrom, disk) one by one.
> > 
> > Sometimes, user don't want to boot from some devices. This patch changes
> > seabios to boot only from selected devices.
> 
> I think (i.e. I didn't test) the patch breaks booting if the user didn't
> explicitly specify a boot device.

Hi Gerd,

| #define DEFAULT_PRIO           9999

If we don't select some devices, their priority will always be 9999
 
| static int DefaultFloppyPrio = 101;
| static int DefaultCDPrio     = 102;
| static int DefaultHDPrio     = 103;
| static int DefaultBEVPrio    = 104;
 
The priority of selected devices will be changed to above values
first, the priority might be re-changed.

However, we can judge if device is selected or not by checking if the
priority is 9999.

. Boot guest without qemu boot option, guest will try to boot from
  network/hd/floppy/cdrom.

. If seabios ignores the order parameter of qemu, guest will also try
  to boot from network/hd/floppy/cdrom.

-- 
		Amos.

  reply	other threads:[~2012-12-19  8:12 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-19  7:22 [Qemu-devel] [PATCH] don't boot from un-selected devices Amos Kong
2012-12-19  7:24 ` [Qemu-devel] [Resend][Seabios PATCH] " Amos Kong
2012-12-19  7:38   ` Gerd Hoffmann
2012-12-19  8:12     ` Amos Kong [this message]
2012-12-19  9:32   ` Gleb Natapov
2012-12-25  3:58     ` Amos Kong
2012-12-25  6:37       ` Gleb Natapov
2012-12-25 10:37         ` Amos Kong
2012-12-26  0:55           ` [Qemu-devel] [SeaBIOS PATCH v2] boot: support strict boot and make it configurable Amos Kong
2013-01-09  4:00             ` Kevin O'Connor
2013-01-09  5:03               ` Amos Kong
2013-01-12 22:01                 ` Kevin O'Connor
2013-01-09  5:17               ` [Qemu-devel] [SeaBIOS PATCH v3] boot: add a new type to halt booting Amos Kong
2012-12-25 15:59     ` [Qemu-devel] [Resend][Seabios PATCH] don't boot from un-selected devices Ronen Hod
2012-12-25 17:15       ` Gleb Natapov
2012-12-19 16:39   ` Kevin O'Connor

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=20121219081222.GA3078@t430s.nay.redhat.com \
    --to=akong@redhat.com \
    --cc=alex.williamson@redhat.com \
    --cc=dallan@redhat.com \
    --cc=gleb@redhat.com \
    --cc=kevin@koconnor.net \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=seabios@seabios.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.