Jan Klemkow <j.klem...@wemelug.de> writes:

> Hi,
>
> This fix [1] in seabios breaks our "boot device cdrom" feature.
>
> # vmctl start -Lc -d disk.img -r cd70.iso -B cdrom vm
> ...
> No bootable device.  Retrying in 60 seconds.
>
> # vmctl start -Lc -d disk.img -r cd70.iso vm
> doas vmctl start -c -r cd70.iso vm
> ...
> CD-ROM: E0
> Loading /7.0/AMD64/CDBOOT
> probing: pc0 com0 mem[638K 510M a20=on]
> disk: cd0
>>> OpenBSD/amd64 CDBOOT 3.53
> boot>
>
> The diff below, fixes the lun number of the bootorder string for cdrom.
>
> OK?

This change definitely fixes -B cdrom, but -B disk seems broken as well.

ok dv to fix the -B cdrom issue, but do you also have an idea how to fix
the -B disk option?

Thanks for looking at this.

>
> bye,
> Jan
>
> [1]: 
> https://github.com/coreboot/seabios/commit/f3ca59c6f3da0f30110ca216b072f8b602313734#diff-fb210bc834fdee64b4c337a57a96d0593afd9c86e956f5ef12a437810ebe61d1
>
> Index: fw_cfg.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/vmd/fw_cfg.c,v
> retrieving revision 1.3
> diff -u -p -r1.3 fw_cfg.c
> --- fw_cfg.c  16 Jun 2021 16:55:02 -0000      1.3
> +++ fw_cfg.c  4 Nov 2021 13:33:37 -0000
> @@ -80,7 +80,7 @@ fw_cfg_init(struct vmop_create_params *v
>               bootorder = "/pci@i0cf8/*@2\nHALT";
>               break;
>       case VMBOOTDEV_CDROM:
> -             bootorder = "/pci@i0cf8/*@4/*@0/*@0,100\nHALT";
> +             bootorder = "/pci@i0cf8/*@4/*@0/*@0,40000100\nHALT";
>               break;
>       case VMBOOTDEV_NET:
>               /* XXX not yet */

Reply via email to