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)
	Fri, Nov 21, 1:04 AM

	Unknown Object (File)
	Fri, Nov 21, 12:54 AM

	Unknown Object (File)
	Fri, Nov 21, 12:51 AM

	Unknown Object (File)
	Fri, Nov 21, 12:47 AM

	Unknown Object (File)
	Fri, Nov 21, 12:44 AM

	Unknown Object (File)
	Wed, Nov 12, 3:50 AM

	Unknown Object (File)
	Thu, Oct 30, 12:25 PM

	Unknown Object (File)
	Fri, Oct 24, 8: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