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, Sep 7, 11:00 PM

	Unknown Object (File)
	Sun, Sep 6, 8:37 AM

	Unknown Object (File)
	Thu, Aug 27, 6:49 PM

	Unknown Object (File)
	Thu, Aug 27, 6:32 PM

	Unknown Object (File)
	Wed, Aug 26, 5:38 PM

	Unknown Object (File)
	Wed, Aug 26, 5:10 PM

	Unknown Object (File)
	Wed, Aug 26, 11:40 AM

	Unknown Object (File)
	Tue, Aug 25, 4:29 PM

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