loader should boot pre-feature flags pools.
ClosedPublic
Actions

Authored by tsoome on Oct 24 2016, 11:51 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, May 11, 6:19 PM

	Unknown Object (File)
	Mon, May 11, 2:57 PM

	Unknown Object (File)
	Sun, May 10, 10:09 AM

	Unknown Object (File)
	Fri, May 8, 12:05 AM

	Unknown Object (File)
	Wed, May 6, 5:32 AM

	Unknown Object (File)
	Sat, May 2, 8:32 PM

	Unknown Object (File)
	Mon, Apr 27, 5:44 PM

	Unknown Object (File)
	Fri, Apr 24, 6:36 AM

View All Files

Subscribers

avg

delphij

Details

Reviewers

avg
allanjude
imp

Commits

rS321680: MFC r307865 (by tsoome): loader should boot pre-feature flags pools.
rS307865: loader should boot pre-feature flags pools.

Summary

Missed corner case with feature flag test - the pre-feature-flags pool
will not have any features.

Test Plan

boot v28 pool.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tsoome updated this revision to Diff 21646.Oct 24 2016, 11:51 AM

tsoome retitled this revision from to loader should boot pre-feature flags pools..

tsoome updated this object.

tsoome edited the test plan for this revision. (Show Details)

tsoome added reviewers: allanjude, imp.

Herald added a subscriber: delphij. · View Herald TranscriptOct 24 2016, 11:51 AM

forgot to return 0...

LGTM

This revision is now accepted and ready to land.Oct 24 2016, 12:07 PM

Approved for commit

Closed by commit rS307865: loader should boot pre-feature flags pools. (authored by tsoome). · Explain WhyOct 24 2016, 4:29 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

boot/

zfs/

zfsimpl.c

9 lines

Diff 21655

View Options

head/sys/boot/zfs/zfsimpl.c

loader should boot pre-feature flags pools.ClosedPublicActions