efi_loader: Fix disk dp's for pre-DM/legacy devices
authorRob Clark <robdclark@gmail.com>
Sun, 8 Oct 2017 15:33:08 +0000 (11:33 -0400)
committerAlexander Graf <agraf@suse.de>
Mon, 9 Oct 2017 05:00:38 +0000 (07:00 +0200)
This fixes an issue with OpenBSD's bootloader, and I think should also
fix a similar issue with grub2 on legacy devices.  In the legacy case
we were creating disk objects for the partitions, but not also the
parent device.

Reported-by: Jonathan Gray <jsg@jsg.id.au>
Signed-off-by: Rob Clark <robdclark@gmail.com>
Signed-off-by: Alexander Graf <agraf@suse.de>
lib/efi_loader/efi_disk.c

index eb9ce772d1d6fe00830e9bbc2732917e8136b8af..47b487aa30b29f854332d393e685400df133ed30 100644 (file)
@@ -340,6 +340,8 @@ int efi_disk_register(void)
                for (i = 0; i < 4; i++) {
                        struct blk_desc *desc;
                        char devname[32] = { 0 }; /* dp->str is u16[32] long */
+                       disk_partition_t info;
+                       int part = 1;
 
                        desc = blk_get_devnum_by_type(if_type, i);
                        if (!desc)
@@ -349,6 +351,15 @@ int efi_disk_register(void)
 
                        snprintf(devname, sizeof(devname), "%s%d",
                                 if_typename, i);
+
+                       /* add devices for each partition: */
+                       while (!part_get_info(desc, part, &info)) {
+                               efi_disk_add_dev(devname, if_typename, desc,
+                                                i, 0, part);
+                               part++;
+                       }
+
+                       /* ... and add block device: */
                        efi_disk_add_dev(devname, if_typename, desc, i, 0, 0);
                        disks++;