Skip to content

Conversation

@gozora
Copy link
Member

@gozora gozora commented May 12, 2016

Hi guys,

I've created first draft as discussed in #831. Currently works only with elilo and manual intervention in EFI shell is needed to boot rear rescue (this will of course change in the future).
Please comment about possible changes and improvements.

@jsmeix jsmeix added the enhancement Adaptions and new features label May 13, 2016
@jsmeix jsmeix added this to the Rear v1.19 milestone May 13, 2016
@jsmeix jsmeix self-assigned this May 13, 2016
@jsmeix
Copy link
Member

jsmeix commented May 13, 2016

From plain looking at your code
(without really understanding UEFI stuff)
I see nothing what could cause a regression.

Therefore I simply accept it as a good starting point
towards USB UEFI boot support #831

In particular I like it because that all the actual work
for USB UEFI boot support happens in one single
separated script output/USB/Linux-i386/10_create_efiboot.sh

In that script regarding

# Should we crash if umount fails?
umount ${EFI_MPT}

I checked what the rear *.sh scripts do in general
when umounting via

$ find . -name '*.sh' | xargs grep -A3 'umount ' | less

and in almost all cases the exit code of
a plain "umount" command is ignored
except in lib/global-functions.sh

Interestingly the umount_mountpoint function
in lib/global-functions.sh is nowhere used ;-)

@jsmeix jsmeix merged commit 70c11f4 into rear:master May 13, 2016
@jsmeix jsmeix mentioned this pull request May 13, 2016
@gozora
Copy link
Member Author

gozora commented May 13, 2016

Thanks for the feedback, It was important for me to know that the starting point is right. Now I can continue to work on it.

jsmeix added a commit that referenced this pull request May 17, 2016
USB EFI boot support for elilo and grub
see #831

As follow-up of #834
this #837 complements
so that it should now work regardless of the bootloader.

UEFI booting from USB disk/stick is impemented in two parts:

Part 1 is #603 that
implements support for "rear format --efi" to format
an USB disk/stik with an additional UEFI (FAT) partition.

Part 2 is #831 that
implements filling the UEFI (FAT) partition on USB disk/stick
with bootloader and creating an appropriate bootloader config.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Adaptions and new features fixed / solved / done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants