On Thu, Nov 04, 2021 at 02:44:18PM +0100, Jan Klemkow wrote: > 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? > > 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 */ >
I noticed this was broken too after a recent discussion on twitter. ok brynet@
