Disable amd64 boot time memory test by default
ClosedPublic
Actions

Authored by emaste on Jan 16 2015, 9:38 PM.

Tags

None

Referenced Files

	F111426214: D1544.diff
	Mon, Mar 3, 3:06 PM

	Unknown Object (File)
	Jan 22 2025, 8:26 AM

	Unknown Object (File)
	Nov 30 2024, 8:23 PM

	Unknown Object (File)
	Nov 25 2024, 12:34 AM

	Unknown Object (File)
	Nov 19 2024, 12:46 PM

	Unknown Object (File)
	Nov 18 2024, 3:21 AM

	Unknown Object (File)
	Nov 18 2024, 2:42 AM

	Unknown Object (File)
	Nov 18 2024, 12:34 AM

View All 72 Files

Subscribers

None

Details

Reviewers

kib
jhb

Commits

rS283262: MFC r258431: Disable amd64 boot time memory test by default

Summary

The page presence memory test takes a long time on large memory systems and has little value on contemporary amd64 hardware.

MFC of r258431

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste updated this revision to Diff 3214.Jan 16 2015, 9:38 PM

emaste retitled this revision from to Disable amd64 boot time memory test by default .

emaste updated this object.

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

emaste added reviewers: jhb, kib.

Hmm, I haven't seen a review request for an MFC before. Presumably changes should get reviewed before going into HEAD, so an MFC doesn't need the same level of review. I certainly think it is fine to MFC this change though.

I think this is a request for opinion about possibility of MFC, i.e. whether it is suitable to change the behaviour on the stable branch.

I do not see why not. The memory 'test' is useless, IMO.

This revision is now accepted and ready to land.Jan 26 2015, 9:03 AM

Closed by commit rS283262: MFC r258431: Disable amd64 boot time memory test by default (authored by emaste). · Explain WhyMay 21 2015, 7:40 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

machdep.c

14 lines

Diff 3214

View Options

Disable amd64 boot time memory test by default ClosedPublicActions