Disable amd64 boot time memory test by default
ClosedPublic
Actions

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

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

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

stable/

10/

sys/

amd64/

machdep.c

14 lines

Diff 5531

View Options

stable/10