[haiku-commits] Re: haiku: hrev51171 - in src/system/boot: loader platform/efi platform/atari_m68k platform/u-boot platform/pxe_ia32

  • From: Jessica Hamilton <jessica.l.hamilton@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 17 May 2017 20:07:42 +0000

On Wed, 17 May 2017, 9:49 PM Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:

Am 17/05/2017 um 11:43 schrieb jessica.l.hamilton@xxxxxxxxx:
e888217124fa: loader: fetch all potential boot partitions for boot
device.

  This allows the loader to skip BFS partitions that don't contain
  a bootable system. Useful when you have a BFS data partition that
  comes before the system partition when iterated over.

I don't get the rationale of this change. Why not simply check if the
BFS partition actually contains a bootable system before returning it as
boot partition?


That part is in the generic vfs code. Previously the platform code would
return just one likely partition before giving up and dropping you into the
menu.

If you look at the modified vfs function, you'll see why this change makes
sense.

The logic should already exist to compile the list of boot volumes in
the menu.


Yes, if you want to enter the menu. This allows booting from the first
bootable bfs partition from the disk containing the uefi loader
automatically.

Other related posts: