Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, > PS: You can fetch the image from the known location to test Looks good. Viewing xorriso arguments in /mnt/iso/.disk/mkisofs ... looks good too. The ISO is now added to my collection of bootable ISOs for regression test. My test loop checks whether the ISOs which are repacked from existing

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Mark Cave-Ayland
On 27/04/2019 17:21, John Paul Adrian Glaubitz wrote: > On 4/27/19 5:35 PM, Mark Cave-Ayland wrote: >>> I set $CDDIR to the source directory which contains core.img? How does the >>> initial >>> bootloader know that it has to boot core.img? >> >> This is the part which Thomas explained: the

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 7:11 PM, Thomas Schmitt wrote: > John Paul Adrian Glaubitz wrote: >> -J -joliet-long -cache-inodes -l >> -G $CDDIR/../CD1/boot/grub/sparc64-ieee1275/cdboot.img -B ... >> >> Would it be enough to just switch to xorriso here? > > I think you forgot -o result.o and -V Volume_Id. Those

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 7:16 PM, John Paul Adrian Glaubitz wrote: > xorriso -as mkisofs -r -checksum_algorithm_iso md5,sha1 -V 'Debian 10.0 > sparc64 n' -o /home/glaubitz/debian-cd-test/debian-10.0-sparc64-NETINST-1.iso > -J -joliet-long -cache-inodes -G >

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Mark Cave-Ayland
On 27/04/2019 19:26, Thomas Schmitt wrote: > Hi, > > xorriso-wise, the debian-10.0-sparc64-grub-NETINST-1.iso looks ok: > > $ xorriso -indev debian-10.0-sparc64-grub-NETINST-1.iso \ > -report_system_area plain > ... > Volume id: 'Debian 10.0 sparc64 n' > System area

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 6:47 PM, Thomas Schmitt wrote: > Mark Cave-Ayland wrote: >> In that case I think the change for Adrian's second patch should in fact be >> this: >> add_mkisofs_opt $CDDIR/../$N.mkisofs_opts "-G cdboot.img -B ... >> --grub2-sparc-core /boot/grub/core.img" >> >> Does that look better? >

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Mark Cave-Ayland
On 27/04/2019 18:12, John Paul Adrian Glaubitz wrote: > On 4/27/19 6:47 PM, Thomas Schmitt wrote: >> Mark Cave-Ayland wrote: >>> In that case I think the change for Adrian's second patch should in fact be >>> this: >>> add_mkisofs_opt $CDDIR/../$N.mkisofs_opts "-G cdboot.img -B ... >>>

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, > xorriso -as mkisofs -r -checksum_algorithm_iso md5,sha1 > -V 'Debian 10.0 sparc64 n' > -o /home/glaubitz/debian-cd-test/debian-10.0-sparc64-NETINST-1.iso > -J -joliet-long -cache-inodes > -G /home/glaubitz/tmp/sid/CD1/../CD1/boot/grub/sparc64-ieee1275/cdboot.img > -B ... >

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, xorriso-wise, the debian-10.0-sparc64-grub-NETINST-1.iso looks ok: $ xorriso -indev debian-10.0-sparc64-grub-NETINST-1.iso \ -report_system_area plain ... Volume id: 'Debian 10.0 sparc64 n' System area options: 0x000c System area summary: SUN-SPARC-Disk-Label

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, there is a copy of the -G image file in the ISO: /boot/grub/sparc64-ieee1275/cdboot.img It has only 512 bytes. Very non-zero. But block 0 is for the SUN disk label (partition table). libisofs overwrites it completely at ISO production time. See

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 9:08 PM, Thomas Schmitt wrote: > Maybe you should prepend 512 0-bytes to your cdboot.img. Yes, indeed that did the trick. The uploaded image now works correctly and boots with GRUB on CD. We can finally get rid of SILO \o/. Thanks everyone for the help! PS: You can fetch the image

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 11:12 PM, Thomas Schmitt wrote: > The ISO is now added to my collection of bootable ISOs for regression > test. My test loop checks whether the ISOs which are repacked from > existing ISOs still show the same boot equipment and file tree content. > (Happens when i change boot related

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 5:40 PM, Thomas Schmitt wrote: >> Once this is done and the ISO image is generated, [...] >> - Next check the offset and size of /boot/grub/core.img embedded in >> sector 1 using dd and hexdump. As per Thomas' message you should find >> the offset in bytes of the start of

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 7:25 PM, Mark Cave-Ayland wrote: > I think it's trying to find /boot/grub/core.img in the ISO image built from > your CD1 > directory and failing - did you also update grub-mkimage in your first patch > so that > the core image is output as core.img and not sparc64.elf? Yes, of

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, Mark Cave-Ayland wrote: > In that case I think the change for Adrian's second patch should in fact be > this: > add_mkisofs_opt $CDDIR/../$N.mkisofs_opts "-G cdboot.img -B ... > --grub2-sparc-core /boot/grub/core.img" > > Does that look better? If /boot/grub/core.img is the path of the next

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, John Paul Adrian Glaubitz wrote: > -J -joliet-long -cache-inodes -l > -G $CDDIR/../CD1/boot/grub/sparc64-ieee1275/cdboot.img -B ... > > Would it be enough to just switch to xorriso here? I think you forgot -o result.o and -V Volume_Id. They are all in the list of emulated options (xorriso

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 7:11 PM, Thomas Schmitt wrote: > $ xorriso -as mkisofs \ > -o test.iso \ > -V 'FAKE_SPARC_64_ISO' \ > -J -joliet-long -cache-inodes -l \ > -G ./dummy_for_G \ > -B '...' \ > --grub2-sparc-core /boot/grub/core.img \ > -graft-points \ >

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, John Paul Adrian Glaubitz: > xorriso -as mkisofs -r -checksum_algorithm_iso md5,sha1 > -V 'Debian 10.0 sparc64 n' > -o /home/glaubitz/debian-cd-test/debian-10.0-sparc64-NETINST-1.iso > -J -joliet-long -cache-inodes > -G /home/glaubitz/tmp/sid/CD1/../CD1/boot/grub/sparc64-ieee1275/cdboot.img >

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Thomas Schmitt
Hi, i wrote: > grub-mkrescue copies stuff to bytes 512 to 767 of its -G image. It's 512 to 1023, of course. Have a nice day :) Thomas

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 9:30 PM, John Paul Adrian Glaubitz wrote: > On 4/27/19 9:08 PM, Thomas Schmitt wrote: >> Maybe you should prepend 512 0-bytes to your cdboot.img. > > Yes, indeed that did the trick. The uploaded image now works > correctly and boots with GRUB on CD. We can finally get rid > of SILO

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
On 4/27/19 5:35 PM, Mark Cave-Ayland wrote: >> I set $CDDIR to the source directory which contains core.img? How does the >> initial >> bootloader know that it has to boot core.img? > > This is the part which Thomas explained: the offset of core.img *as embedded > within > the ISO filesystem

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread John Paul Adrian Glaubitz
Hi! On 4/27/19 5:02 PM, Mark Cave-Ayland wrote: > Adrian: based upon this I think what you need to do is: > > - For consistency rename /boot/grub/sparc64.elf to /boot/grub/core.img to > match > the descriptions of the grub components in the documentation Ok. But still generate it with

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Mark Cave-Ayland
On 26/04/2019 18:04, Thomas Schmitt wrote: > Hi, > >> - the first 32K of an ISO9660 is undefined > > That's such a negative word. :)) > It is "reserved for system use" and "not specified" by ISO 9660 / ECMA-119. > > >> - boot.S and diskboot.S get built into cdboot.img > > grub-mkrescue opens

Re: Bug#927892: grub-ieee1275-bin: Please add bootinfo.txt for sparc and sparc64

2019-04-27 Thread Mark Cave-Ayland
On 27/04/2019 16:40, Thomas Schmitt wrote: > Hi, > > Mark Cave-Ayland wrote: >> - In your second patch re-enable the -G/-B options but with -G set to >> cdboot.img: >> add_mkisofs_opt $CDDIR/../$N.mkisofs_opts "-G cdboot.img -B ..." > > Does this refer to > From: John Paul Adrian Glaubitz