Set bootme GPT flag to mark "active" partition on a GPT-labelled image
Needs RevisionPublic
Actions

Authored by sobomax on Feb 7 2024, 12:51 AM.

Details

Reviewers

Summary

mkimg(1) is a fantastic tool to build all sorts of partitioned disk images, however it lacks the ability to mark GPT boot partition "active".

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

sobomax requested review of this revision.Feb 7 2024, 12:51 AM

sobomax created this revision.

imp accepted this revision.Feb 7 2024, 2:49 AM

This revision is now accepted and ready to land.Feb 7 2024, 2:49 AM

marius requested changes to this revision.Feb 7 2024, 10:43 PM

marius added inline comments.

usr.bin/mkimg/mkimg.1
163	It would be nice to have this list sorted alphabetically. Apart from that, this change looks good to me. Don't forget to bump .Dd when commtting, though.

This revision now requires changes to proceed.Feb 7 2024, 10:43 PM

o sorted alphabetically;
o updated .Dd.

sobomax marked an inline comment as done.Feb 7 2024, 11:55 PM

marius requested changes to this revision.Feb 8 2024, 12:48 AM

marius added inline comments.

usr.bin/mkimg/mkimg.1
160	Uhm, the "and" should go onto a separate line so it's not an argument to the .Ar macro and not rendered specially as expected.

This revision now requires changes to proceed.Feb 8 2024, 12:48 AM

usr.bin/mkimg/gpt.c
211	One final nit... For GPT this is FreeBSD specific. The GPT standard just sets these bits aside for the OS to do what it wants... Linux, windows and MacOS all ignore it. A small comment to this effect would be good.