Differential D47652

geom: Allow BSD type '!0' partitions
ClosedPublic
Actions

Authored by jlduran on Mon, Nov 18, 12:29 AM.

Tags

None

Referenced Files

	F103096671: D47652.diff
	Wed, Nov 20, 10:34 PM

	Unknown Object (File)
	Tue, Nov 19, 8:45 PM

	Unknown Object (File)
	Tue, Nov 19, 1:07 PM

	Unknown Object (File)
	Tue, Nov 19, 2:06 AM

	Unknown Object (File)
	Mon, Nov 18, 9:52 PM

	Unknown Object (File)
	Mon, Nov 18, 9:50 PM

	Unknown Object (File)
	Mon, Nov 18, 8:52 PM

Subscribers

dnelson_1901_yahoo.com

Details

Reviewers

emaste
imp

Commits

rGaccf71534c61: geom: Allow BSD type '!0' partitions

Summary

Allow the creation of partition type '!0', in order to replicate the
behavior of bsdlabel -w.

Fix it by not considering "0" an invalid partition type.

Test Plan

md0=$(mdconfig -s 100m)
md1=$(mdconfig -s 100m)

bsdlabel -w "$md0"

gpart create -s bsd "$md1"
gpart add -t '!0' -b 16 "$md1"

gpart show
=>     0  204800  md0  BSD  (100M)
       0      16       - free -  (8.0K)
      16  204784    1  !0  (100M)

=>     0  204800  md1  BSD  (100M)
       0      16       - free -  (8.0K)
      16  204784    1  !0  (100M)

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jlduran requested review of this revision.Mon, Nov 18, 12:29 AM

jlduran created this revision.

Harbormaster completed remote builds in B60651: Diff 146628.Mon, Nov 18, 12:29 AM

jlduran added a child revision: D47653: nanobsd: Remove dependency on bsdlabel.Mon, Nov 18, 12:29 AM

dnelson_1901_yahoo.com added a subscriber: dnelson_1901_yahoo.com.Mon, Nov 18, 12:54 AM

I think this is reasonable -- 0 is unused/reserved, but no reason we shouldn't be able to specify it. It's not out of range etc.

This revision is now accepted and ready to land.Mon, Nov 18, 1:35 PM

emaste mentioned this in D47653: nanobsd: Remove dependency on bsdlabel.Mon, Nov 18, 4:19 PM

Closed by commit rGaccf71534c61: geom: Allow BSD type '!0' partitions (authored by jlduran). · Explain WhyWed, Nov 20, 10:34 PM

This revision was automatically updated to reflect the committed changes.

jlduran added a commit: rGaccf71534c61: geom: Allow BSD type '!0' partitions.

Revision Contents
Changeset List

Path

Size

sys/

geom/

part/

2 lines

Diff 146777

sys/geom/part/g_part_bsd.c

Loading...