[haiku-commits] Re: haiku: hrev50832 - in src/system/boot: platform/efi loader

  • From: Jessica Hamilton <jessica.l.hamilton@xxxxxxxxx>
  • To: "haiku-commits@xxxxxxxxxxxxx" <haiku-commits@xxxxxxxxxxxxx>
  • Date: Sun, 8 Jan 2017 15:38:01 +1300

On 8 January 2017 at 13:00, Fredrik Holmqvist
<fredrik.holmqvist@xxxxxxxxx> wrote:

On Sun, 8 Jan 2017 4:24 am Fredrik Holmqvist <fredrik.holmqvist@xxxxxxxxx>
wrote:
It shouldn't actually do any reading of the other devices though. At most
it does a media present check. I'm not sure whether that is quick though.
Perhaps the media present check can move into
device_contains_partition if that is the case
?


It adds cd-devices by default, which means Haiku will need to scan for
partitions on it before you even pick boot partition. I rather see we follow
the logic of other boot-platforms and don't do any device path or device
magic unless one of the other cases requires it. I just want a minimal
boot-loader.

No it doesn't; scanning all Nodes for partitions doesn't happen that
early. You should read
https://github.com/haiku/haiku/blob/master/src/system/boot/loader/vfs.cpp#L648
closer.

It stops on the first device it finds. And we insert the CD devices at
the end of the list. So unless the first Node that it encounters (from
add_boot_device_for_image), decides isn't bootable
(device_contains_partition), it won't continue to the Node(s)
representing CD devices.

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 01-2017