Page MenuHomeFreeBSD

Add the 4 ChromeOS GPT partition types to gpart
ClosedPublic

Authored by allanjude on Oct 8 2015, 5:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 25 2023, 4:06 PM
Unknown Object (File)
Dec 20 2023, 1:23 AM
Unknown Object (File)
Dec 17 2023, 6:42 PM
Unknown Object (File)
Dec 11 2023, 10:11 AM
Unknown Object (File)
Nov 9 2023, 10:56 PM
Unknown Object (File)
Nov 7 2023, 11:03 PM
Unknown Object (File)
Nov 2 2023, 3:12 PM
Unknown Object (File)
Oct 6 2023, 9:50 PM
Subscribers

Details

Summary

While here, also add OpenBSD, and 2 new Microsoft partition types

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

allanjude retitled this revision from to Add the 4 ChromeOS GPT partition types to gpart While here, also add OpenBSD, and 2 new Microsoft partition types.
allanjude updated this object.
allanjude edited the test plan for this revision. (Show Details)
allanjude added reviewers: marcel, bapt, grehan.

IIRC, aliases must be sorted so that the binary search works. ms-recovery before ms-reserved, please.
I see we disordered ntfs before netbsd-* as well, so we have a pre-existing bug if the binary search still holds.

In D3841#79411, @marcel wrote:

IIRC, aliases must be sorted so that the binary search works. ms-recovery before ms-reserved, please.
I see we disordered ntfs before netbsd-* as well, so we have a pre-existing bug if the binary search still holds.

Is it a problem to re-order the enum?

allanjude retitled this revision from Add the 4 ChromeOS GPT partition types to gpart While here, also add OpenBSD, and 2 new Microsoft partition types to Add the 4 ChromeOS GPT partition types to gpart.
allanjude edited edge metadata.
allanjude updated this object.

Ordered the partition types alphabetically

marcel edited edge metadata.

I double-checked the code and there's actually no binary search logic. It's all linear. Technically speaking, the reordering was not necessary, but I really appreciate the effort. It's a lot easier now to implement a binary search for cases where the linear search gets too time consuming.

Thanks!

This revision is now accepted and ready to land.Nov 3 2015, 7:26 PM
This revision was automatically updated to reflect the committed changes.