Differential D23046

Fix zone and keg alignment as well as boot pages calculations on numa.
ClosedPublic
Actions

Authored by jeff on Jan 5 2020, 11:53 PM.

Tags

None

Referenced Files

	F143320644: D23046.diff
	Wed, Jan 28, 8:52 PM

	F143318484: D23046.diff
	Wed, Jan 28, 8:06 PM

	Unknown Object (File)
	Sun, Jan 25, 5:46 PM

	Unknown Object (File)
	Fri, Jan 23, 2:41 AM

	Unknown Object (File)
	Mon, Jan 19, 5:29 PM

	Unknown Object (File)
	Sun, Jan 11, 10:18 PM

	Unknown Object (File)
	Mon, Jan 5, 8:32 PM

	Unknown Object (File)
	Dec 16 2025, 6:42 PM

Subscribers

Details

Reviewers

rlibby
markj

Commits

rS356393: Fix uma boot pages calculations on NUMA machines that also don't have

Summary

While making NUMA amd64 w/o MD_SMALL_ALLOC boot I noticed a number of numa bugs but also weaker than required alignment. In practice the alignment of zones was actually dictated by the alignment of the per-cpu cache structure. This just clarifies existing behavior while simplifying code.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 28494
Build 26551: arc lint + arc unit

Event Timeline

jeff created this revision.Jan 5 2020, 11:53 PM

Harbormaster completed remote builds in B28494: Diff 66394.Jan 5 2020, 11:54 PM

jeff edited the summary of this revision. (Show Details)Jan 5 2020, 11:55 PM

jeff added reviewers: rlibby, markj.

jeff set the repository for this revision to rS FreeBSD src repository - subversion.

Herald added a subscriber: imp. · View Herald TranscriptJan 5 2020, 11:55 PM

jeff added inline comments.Jan 5 2020, 11:56 PM

sys/amd64/include/vmparam.h
79	obviously not committing this.

rlibby accepted this revision.Jan 6 2020, 12:10 AM

This revision is now accepted and ready to land.Jan 6 2020, 12:10 AM

markj accepted this revision.Jan 6 2020, 1:03 AM

markj added inline comments.

sys/vm/uma_int.h
169	Typo: terminology. I think we used to define CACHE_LINE_SIZE to 128 on amd64 for this reason. I'm not sure when that changed.

Closed by commit rS356393: Fix uma boot pages calculations on NUMA machines that also don't have (authored by jeff). · Explain WhyJan 6 2020, 2:51 AM

This revision was automatically updated to reflect the committed changes.

jeff added a commit: rS356393: Fix uma boot pages calculations on NUMA machines that also don't have.

Revision Contents
Changeset List

Path

Size

sys/

amd64/

include/

2 lines

vm/

78 lines

11 lines

13 lines

Diff 66394

sys/amd64/include/vmparam.h

Loading...

sys/vm/uma_core.c

Loading...

sys/vm/uma_int.h

Loading...

sys/vm/vm_page.c

Loading...