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)
	Thu, Aug 13, 2:59 AM

	Unknown Object (File)
	Wed, Aug 12, 7:55 PM

	Unknown Object (File)
	Wed, Aug 12, 11:37 AM

	Unknown Object (File)
	Wed, Aug 12, 11:33 AM

	Unknown Object (File)
	Wed, Aug 12, 8:58 AM

	Unknown Object (File)
	Tue, Aug 11, 8:16 PM

	Unknown Object (File)
	Mon, Aug 10, 2:25 PM

	Unknown Object (File)
	Mon, Aug 10, 10:45 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 Passed

Unit

No Test Coverage

Build Status

Buildable 5699
Build 5993: arc lint + arc unit

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

sys/

boot/

zfs/

zfsimpl.c

7 lines

Diff 21646

View Options

loader should boot pre-feature flags pools.ClosedPublicActions