qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gleb Natapov <gleb@redhat.com>
To: Kevin O'Connor <kevin@koconnor.net>
Cc: Gerhard Wiesinger <lists@wiesinger.com>,
	Gerd Hoffmann <kraxel@redhat.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Boot order problem and disable iPXE/gPXE
Date: Fri, 6 Jan 2012 07:27:25 +0200	[thread overview]
Message-ID: <20120106052725.GA3105@redhat.com> (raw)
In-Reply-To: <20120106020539.GB25140@morn.localdomain>

On Thu, Jan 05, 2012 at 09:05:39PM -0500, Kevin O'Connor wrote:
> On Thu, Jan 05, 2012 at 11:20:37AM +0200, Gleb Natapov wrote:
> > On Thu, Jan 05, 2012 at 10:09:44AM +0100, Gerhard Wiesinger wrote:
> > > But as far as I remember therefore the option ROM registers through
> > > BIOS for INT 19h booting. So Seabios should know it that this is a
> > > harddisk.
> > It registers BEV. Otherwise you wouldn't be able to boot at all. In fact
> > it registers multiple BEVs (one for each attached scsi device). But BIOS
> > knows nothing about the device behind the BEV. It just jumps to BEV
> > address to boot from it when INT 19h is called. There may be scsi cdrom
> > there or even tape.
> 
> Gerhard's option rom is actually registering a BCV, and SeaBIOS can
> know that is likely a hard drive.  So, I think Gerhard has a point.
Ah, yeah. I see now that pnp rom can register either bev or bcv. But non
of them tell us what device it actually is.

> SeaBIOS could probably fall back to the harddrive priority if it finds
> a BCV without an explicit bootindex priority.
The same option rom probably will register bcv for SCSI CD/DVD too. Also
are you sure pxe roms do not register bcv for network booting?

> 
> Gerhard - does the patch below improve things for you?
> 
> -Kevin
> 
> 
> --- a/src/boot.c
> +++ b/src/boot.c
> @@ -326,7 +326,7 @@ boot_add_bev(u16 seg, u16 bev, u16 desc, int prio)
>  void
>  boot_add_bcv(u16 seg, u16 ip, u16 desc, int prio)
>  {
> -    bootentry_add(IPL_TYPE_BCV, defPrio(prio, DEFAULT_PRIO)
> +    bootentry_add(IPL_TYPE_BCV, defPrio(prio, DefaultHDPrio)
>                    , SEGOFF(seg, ip).segoff
>                    , desc ? MAKE_FLATPTR(seg, desc) : "Legacy option rom");
>  }

--
			Gleb.

  reply	other threads:[~2012-01-06  5:27 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-04 18:02 [Qemu-devel] Boot order problem and disable iPXE/gPXE Gerhard Wiesinger
2012-01-05  3:03 ` Kevin O'Connor
2012-01-05  7:39   ` Gerhard Wiesinger
2012-01-06  1:42     ` Kevin O'Connor
2012-01-06  8:20       ` Gerhard Wiesinger
2012-01-06  8:59         ` Gerd Hoffmann
2012-01-05  7:50 ` Gleb Natapov
2012-01-05  7:58 ` Gerd Hoffmann
2012-01-05  8:23   ` Gerhard Wiesinger
2012-01-05  8:32     ` Gleb Natapov
2012-01-05  9:09       ` Gerhard Wiesinger
2012-01-05  9:20         ` Gleb Natapov
2012-01-05  9:27           ` Gerhard Wiesinger
2012-01-05  9:29             ` Gleb Natapov
2012-01-05  9:36               ` Gerhard Wiesinger
2012-01-05  9:42                 ` Gleb Natapov
2012-01-05  9:56                   ` Gerhard Wiesinger
2012-01-05 10:00                     ` Gleb Natapov
2012-01-05  9:42             ` Gerd Hoffmann
2012-01-06  2:05           ` Kevin O'Connor
2012-01-06  5:27             ` Gleb Natapov [this message]
2012-01-07  1:53               ` Kevin O'Connor
2012-01-07  9:00                 ` Gerhard Wiesinger
2012-01-07 16:28                   ` Kevin O'Connor
2012-01-07 22:47                     ` Gerhard Wiesinger
2012-01-07 22:53                       ` Kevin O'Connor
2012-01-08  8:31                     ` Gleb Natapov
2012-01-08 14:58                       ` Kevin O'Connor
2012-01-08 15:17                         ` Gleb Natapov
2012-01-08 15:47                           ` Kevin O'Connor
2012-01-25 20:30                             ` Gerhard Wiesinger
2012-01-09  9:10                 ` Gerd Hoffmann
2012-01-07  8:35             ` Gerhard Wiesinger
2012-01-07 16:20               ` Kevin O'Connor
2012-01-07 22:46                 ` Gerhard Wiesinger
2012-01-07 22:59                   ` Gerhard Wiesinger
2012-01-08 11:59                     ` Gerhard Wiesinger
2012-01-05  8:57     ` Gerd Hoffmann
2012-01-05  9:24       ` Gerhard Wiesinger
2012-01-05  9:53         ` Gerd Hoffmann
2012-01-05 10:08           ` Gerhard Wiesinger
2012-01-05 10:13             ` Gerd Hoffmann
2012-01-05 11:13               ` Gerhard Wiesinger
2012-01-05 11:16                 ` Gleb Natapov

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=20120106052725.GA3105@redhat.com \
    --to=gleb@redhat.com \
    --cc=kevin@koconnor.net \
    --cc=kraxel@redhat.com \
    --cc=lists@wiesinger.com \
    --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).