All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kevin O'Connor <kevin@koconnor.net>
To: Gleb Natapov <gleb@redhat.com>
Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org, seabios@seabios.org
Subject: Re: [PATCHv6 00/16] boot order specification
Date: Wed, 1 Dec 2010 21:25:40 -0500	[thread overview]
Message-ID: <20101202022540.GA23364@morn.localdomain> (raw)
In-Reply-To: <20101201122740.GL2187@redhat.com>

On Wed, Dec 01, 2010 at 02:27:40PM +0200, Gleb Natapov wrote:
> On Tue, Nov 30, 2010 at 09:53:32PM -0500, Kevin O'Connor wrote:
> > BTW, what's the plan for handling SCSI adapters?  Lets say a user has
> > a scsi card with three drives (lun 1, lun 3, lun 5) that show up as 3
> > bcvs (lun1, lun3, lun5 in that order) and the user wishes to boot from
> > lun3.  I understand this use case may not be important for qemu, but
> > I'd like to use the same code on coreboot.  Being able to boot from
> > any drive is important - it doesn't have to autodetect, but it should
> > be possible.
> > 
> We can't. Option rom does not give us back enough information to do so.
> Form device path we know exactly what id:lun boot should be done from,
> but pnp_data does not have information to map BCV back to id:lun. I do
> not see how coreboot can provide better information to Seabios then
> qemu here.

You're thinking in terms of which device to boot, which does make this
difficult.  However, it's equally valid to think in terms of which
boot method to invoke, which makes this easy.

We could tell the coreboot user to edit the "bootorder" file and add
"/pci@i0cf8/rom1@4" (second rom on 4th pci device - the exact syntax
of the name is not important).

>BTW to create proper EDD entry for SCSI boot device BIOS also
> needs too map BCV to id:lun. How it can be done? 

It's the responsibility of the rom to build the EDD info.  I don't
know if all roms do this - I don't believe it's possible to get at the
EDD info until after the drive has been mapped (ie, too late to use it
for boot ordering).

> SCSI _is_ important for qemu. Not HBA  we have now, but something supported
> by recent operation systems out of the box. When such HBA will be emulated
> in qemu we will add native Seabios support for it.

I understand.  However, we'll still need to support arbitrary rom
based BEVs and BCVs, so the use case is still important.

-Kevin

WARNING: multiple messages have this Message-ID (diff)
From: Kevin O'Connor <kevin@koconnor.net>
To: Gleb Natapov <gleb@redhat.com>
Cc: seabios@seabios.org, qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: [Qemu-devel] Re: [PATCHv6 00/16] boot order specification
Date: Wed, 1 Dec 2010 21:25:40 -0500	[thread overview]
Message-ID: <20101202022540.GA23364@morn.localdomain> (raw)
In-Reply-To: <20101201122740.GL2187@redhat.com>

On Wed, Dec 01, 2010 at 02:27:40PM +0200, Gleb Natapov wrote:
> On Tue, Nov 30, 2010 at 09:53:32PM -0500, Kevin O'Connor wrote:
> > BTW, what's the plan for handling SCSI adapters?  Lets say a user has
> > a scsi card with three drives (lun 1, lun 3, lun 5) that show up as 3
> > bcvs (lun1, lun3, lun5 in that order) and the user wishes to boot from
> > lun3.  I understand this use case may not be important for qemu, but
> > I'd like to use the same code on coreboot.  Being able to boot from
> > any drive is important - it doesn't have to autodetect, but it should
> > be possible.
> > 
> We can't. Option rom does not give us back enough information to do so.
> Form device path we know exactly what id:lun boot should be done from,
> but pnp_data does not have information to map BCV back to id:lun. I do
> not see how coreboot can provide better information to Seabios then
> qemu here.

You're thinking in terms of which device to boot, which does make this
difficult.  However, it's equally valid to think in terms of which
boot method to invoke, which makes this easy.

We could tell the coreboot user to edit the "bootorder" file and add
"/pci@i0cf8/rom1@4" (second rom on 4th pci device - the exact syntax
of the name is not important).

>BTW to create proper EDD entry for SCSI boot device BIOS also
> needs too map BCV to id:lun. How it can be done? 

It's the responsibility of the rom to build the EDD info.  I don't
know if all roms do this - I don't believe it's possible to get at the
EDD info until after the drive has been mapped (ie, too late to use it
for boot ordering).

> SCSI _is_ important for qemu. Not HBA  we have now, but something supported
> by recent operation systems out of the box. When such HBA will be emulated
> in qemu we will add native Seabios support for it.

I understand.  However, we'll still need to support arbitrary rom
based BEVs and BCVs, so the use case is still important.

-Kevin

  reply	other threads:[~2010-12-02  2:25 UTC|newest]

Thread overview: 136+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-17 16:43 [PATCHv6 00/16] boot order specification Gleb Natapov
2010-11-17 16:43 ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 01/16] Introduce fw_name field to DeviceInfo structure Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 02/16] Introduce new BusInfo callback get_fw_dev_path Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 03/16] Keep track of ISA ports ISA device is using in qdev Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 04/16] Add get_fw_dev_path callback to ISA bus " Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 05/16] Store IDE bus id in IDEBus structure for easy access Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 06/16] Add get_fw_dev_path callback to IDE bus Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 07/16] Add get_dev_path callback for system bus Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 08/16] Add get_fw_dev_path callback for pci bus Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 09/16] Record which USBDevice USBPort belongs too Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 10/16] Add get_dev_path callback for usb bus Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 11/16] Add get_dev_path callback to scsi bus Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:43 ` [PATCHv6 12/16] Add bootindex parameter to net/block/fd device Gleb Natapov
2010-11-17 16:43   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:44 ` [PATCHv6 13/16] Change fw_cfg_add_file() to get full file path as a parameter Gleb Natapov
2010-11-17 16:44   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:44 ` [PATCHv6 14/16] Add bootindex for option roms Gleb Natapov
2010-11-17 16:44   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:44 ` [PATCHv6 15/16] Add notifier that will be called when machine is fully created Gleb Natapov
2010-11-17 16:44   ` [Qemu-devel] " Gleb Natapov
2010-11-17 16:44 ` [PATCHv6 16/16] Pass boot device list to firmware Gleb Natapov
2010-11-17 16:44   ` [Qemu-devel] " Gleb Natapov
2010-11-23 15:31 ` [PATCHv6 00/16] boot order specification Gleb Natapov
2010-11-23 15:31   ` [Qemu-devel] " Gleb Natapov
2010-11-23 16:12   ` Anthony Liguori
2010-11-23 16:12     ` [Qemu-devel] " Anthony Liguori
2010-11-23 19:30     ` Blue Swirl
2010-11-23 19:30       ` [Qemu-devel] " Blue Swirl
2010-11-27 20:56     ` Avi Kivity
2010-11-27 20:56       ` [Qemu-devel] " Avi Kivity
2010-11-28  7:54       ` Gleb Natapov
2010-11-28  7:54         ` [Qemu-devel] " Gleb Natapov
2010-11-28  9:38         ` Avi Kivity
2010-11-28  9:38           ` [Qemu-devel] " Avi Kivity
2010-11-28  9:47           ` Gleb Natapov
2010-11-28  9:47             ` [Qemu-devel] " Gleb Natapov
2010-11-28 12:39         ` Blue Swirl
2010-11-28 12:39           ` [Qemu-devel] " Blue Swirl
2010-11-28 13:03           ` Gleb Natapov
2010-11-28 13:03             ` [Qemu-devel] " Gleb Natapov
2010-11-28 13:13         ` Michael S. Tsirkin
2010-11-28 13:13           ` [Qemu-devel] " Michael S. Tsirkin
2010-11-28 13:19           ` Gleb Natapov
2010-11-28 13:19             ` [Qemu-devel] " Gleb Natapov
2010-11-28 13:22             ` Blue Swirl
2010-11-28 13:22               ` [Qemu-devel] " Blue Swirl
2010-11-28 17:02               ` Michael S. Tsirkin
2010-11-28 17:02                 ` [Qemu-devel] " Michael S. Tsirkin
2010-11-28 17:23             ` Michael S. Tsirkin
2010-11-28 17:23               ` [Qemu-devel] " Michael S. Tsirkin
2010-11-28 18:54               ` Gleb Natapov
2010-11-28 18:54                 ` [Qemu-devel] " Gleb Natapov
2010-11-28 19:09                 ` Michael S. Tsirkin
2010-11-28 19:09                   ` [Qemu-devel] " Michael S. Tsirkin
2010-11-28 19:20                   ` Gleb Natapov
2010-11-28 19:20                     ` [Qemu-devel] " Gleb Natapov
2010-11-28 13:25           ` Gleb Natapov
2010-11-28 13:25             ` [Qemu-devel] " Gleb Natapov
2010-11-24  1:19   ` Kevin O'Connor
2010-11-24  1:19     ` [Qemu-devel] " Kevin O'Connor
2010-11-24 10:03     ` Gleb Natapov
2010-11-24 10:03       ` [Qemu-devel] " Gleb Natapov
2010-11-27 15:41       ` Kevin O'Connor
2010-11-27 15:41         ` [Qemu-devel] " Kevin O'Connor
2010-11-27 16:22         ` Gleb Natapov
2010-11-27 16:22           ` [Qemu-devel] " Gleb Natapov
2010-11-27 16:49           ` Kevin O'Connor
2010-11-27 16:49             ` [Qemu-devel] " Kevin O'Connor
2010-11-27 17:06             ` Gleb Natapov
2010-11-27 17:06               ` [Qemu-devel] " Gleb Natapov
2010-11-27 17:47               ` Kevin O'Connor
2010-11-27 17:47                 ` [Qemu-devel] " Kevin O'Connor
2010-11-27 18:15                 ` Gleb Natapov
2010-11-27 18:15                   ` [Qemu-devel] " Gleb Natapov
2010-11-27 18:40                   ` Kevin O'Connor
2010-11-27 18:40                     ` [Qemu-devel] " Kevin O'Connor
2010-11-27 19:04                     ` Gleb Natapov
2010-11-27 19:04                       ` [Qemu-devel] " Gleb Natapov
2010-11-27 21:07                       ` Kevin O'Connor
2010-11-27 21:07                         ` [Qemu-devel] " Kevin O'Connor
2010-11-28  7:45                         ` Gleb Natapov
2010-11-28  7:45                           ` [Qemu-devel] " Gleb Natapov
2010-11-28 17:15                           ` Kevin O'Connor
2010-11-28 17:15                             ` [Qemu-devel] " Kevin O'Connor
2010-11-28 18:47                             ` Gleb Natapov
2010-11-28 18:47                               ` [Qemu-devel] " Gleb Natapov
2010-11-30  1:34                               ` Kevin O'Connor
2010-11-30 14:01                                 ` Gleb Natapov
2010-11-30 14:01                                   ` [Qemu-devel] " Gleb Natapov
2010-12-01  2:53                                   ` Kevin O'Connor
2010-12-01  2:53                                     ` [Qemu-devel] " Kevin O'Connor
2010-12-01 12:27                                     ` Gleb Natapov
2010-12-01 12:27                                       ` [Qemu-devel] " Gleb Natapov
2010-12-02  2:25                                       ` Kevin O'Connor [this message]
2010-12-02  2:25                                         ` Kevin O'Connor
2010-12-02 12:30                                         ` Gleb Natapov
2010-12-02 12:30                                           ` [Qemu-devel] " Gleb Natapov
2010-12-02 15:07                                           ` [SeaBIOS] " Peter Stuge
2010-12-02 15:07                                             ` [Qemu-devel] " Peter Stuge
2010-12-02 17:13                                             ` Gleb Natapov
2010-12-02 17:13                                               ` [Qemu-devel] " Gleb Natapov
2010-12-02 21:22                                           ` Sebastian Herbszt
2010-12-02 21:22                                             ` [Qemu-devel] " Sebastian Herbszt
2010-12-03  2:01                                           ` Kevin O'Connor
2010-12-03  2:01                                             ` [Qemu-devel] " Kevin O'Connor
2010-12-03  5:55                                             ` Gleb Natapov
2010-12-03  5:55                                               ` [Qemu-devel] " Gleb Natapov
2010-11-29 10:50                             ` Gerd Hoffmann
2010-11-29 10:50                               ` [Qemu-devel] " Gerd Hoffmann
2010-11-30  1:55                               ` Kevin O'Connor
2010-11-30 14:59                                 ` Gleb Natapov
2010-11-30 14:59                                   ` [Qemu-devel] " Gleb Natapov
2010-11-28 19:00                           ` [SeaBIOS] " Peter Stuge
2010-11-28 19:00                             ` [Qemu-devel] " Peter Stuge
2010-11-28 19:11                             ` Peter Stuge
2010-11-28 19:11                               ` [Qemu-devel] " Peter Stuge
2010-11-28 19:52                               ` Gleb Natapov
2010-11-28 19:52                                 ` [Qemu-devel] " Gleb Natapov
2010-11-28 19:16                             ` Gleb Natapov
2010-11-28 19:16                               ` [Qemu-devel] " Gleb Natapov
2010-11-29 10:19                     ` Gerd Hoffmann
2010-11-29 10:19                       ` [Qemu-devel] " Gerd Hoffmann
2010-11-29 12:07                       ` Gleb Natapov
2010-11-29 12:07                         ` [Qemu-devel] " 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=20101202022540.GA23364@morn.localdomain \
    --to=kevin@koconnor.net \
    --cc=gleb@redhat.com \
    --cc=kvm@vger.kernel.org \
    --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.