Page MenuHomeFreeBSD

arm64: set the correct partition type in make-memstick.sh
ClosedPublic

Authored by mhorne on Oct 28 2020, 2:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 14 2024, 1:15 PM
Unknown Object (File)
Mar 14 2024, 1:15 PM
Unknown Object (File)
Mar 14 2024, 12:50 PM
Unknown Object (File)
Mar 11 2024, 12:49 AM
Unknown Object (File)
Jan 3 2024, 7:34 PM
Unknown Object (File)
Jan 3 2024, 7:34 PM
Unknown Object (File)
Jan 3 2024, 6:45 PM
Unknown Object (File)
Jan 3 2024, 6:30 PM
Subscribers

Details

Summary

We create a UFS root filesystem using makefs(8), and later pass it to mkimg(1) when
creating the final image. The correct partition type is freebsd-ufs; the freebsd
parition type is for partitions containing a BSD disklabel.

Test Plan

With this change:

# gpart show
=>     3  672344  vtbd0  GPT  (328M)
       3   66584      1  efi  (33M)
   66587  605760      2  freebsd-ufs  (296M)

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 34438
Build 31542: arc lint + arc unit

Event Timeline

This is just the partition type reported in the gpt header is it not?
i.e. there was not actually a disklabel before

This is just the partition type reported in the gpt header is it not?
i.e. there was not actually a disklabel before

Ahh, this highlights the gap in my understanding. I was unsure if -p freebsd:= actually created the disklabel or not, but upon another reading it clearly just sets the partition type, with whatever contents you pass to it.

So yes, that is what this change actually does. I will reword the description.

mhorne retitled this revision from Don't use BSD disklabels in arm64 make-memstick.sh to arm64: set the correct partition type in make-memstick.sh.Oct 28 2020, 4:30 PM
mhorne edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Oct 28 2020, 5:03 PM