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)
Fri, Nov 7, 11:39 AM
Unknown Object (File)
Sun, Nov 2, 6:31 AM
Unknown Object (File)
Thu, Oct 30, 3:59 AM
Unknown Object (File)
Thu, Oct 30, 12:36 AM
Unknown Object (File)
Tue, Oct 28, 1:31 AM
Unknown Object (File)
Fri, Oct 24, 3:37 PM
Unknown Object (File)
Wed, Oct 22, 9:10 PM
Unknown Object (File)
Tue, Oct 21, 6:42 AM
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.