Paths

Table of Contentst

Differential D36307

arm64 pmap: per-domain pv chunk list
ClosedPublic
Actions

Authored by andrew on Aug 23 2022, 11:07 AM.

Tags

None

Referenced Files

	F102719108: D36307.diff
	Sat, Nov 16, 7:50 AM

	Unknown Object (File)
	Wed, Nov 13, 9:57 AM

	Unknown Object (File)
	Tue, Nov 12, 8:46 AM

	Unknown Object (File)
	Mon, Nov 11, 1:35 AM

	Unknown Object (File)
	Sep 16 2024, 1:18 PM

	Unknown Object (File)
	Sep 15 2024, 12:38 AM

	Unknown Object (File)
	Sep 2 2024, 1:26 AM

	Unknown Object (File)
	Sep 1 2024, 11:49 PM

View All 23 Files

Subscribers

Details

Reviewers

alc
markj
kib
mjg
manu

Group Reviewers

Commits

rG3247bc7cd652: arm64 pmap: per-domain pv chunk list

Summary

As with amd64 use a per-domain pv chunk lock to reduce contention as
chunks get created and removed all the time.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

andrew created this revision.Aug 23 2022, 11:07 AM

Herald added a reviewer: manu. · View Herald TranscriptAug 23 2022, 11:07 AM

Herald added subscribers: emaste, imp. · View Herald Transcript

andrew requested review of this revision.Aug 23 2022, 11:07 AM

Harbormaster completed remote builds in B47071: Diff 109707.Aug 23 2022, 11:07 AM

markj added inline comments.Aug 29 2022, 3:39 PM

sys/arm64/arm64/pmap.c
2631	pv_chunks_mutex is no more, so this comment is stale.
2768	This ignores the thread's NUMA allocation policy. In particular, the thread may have been configured to never allocate pages from a particular domain, but here it may do so. I suspect this function should use one of the domainset iterators like vm_page_alloc_noobj() does.
2779	Extra newline.
5505	Can this simply be `free_pv_chunk_batch(free_chunks);`?

andrew added inline comments.Aug 31 2022, 11:12 AM

sys/arm64/arm64/pmap.c
2768	This looks like it's also a problem on amd64

Fix a comment
Remove an extra newline
Remove an unneeded cast

Harbormaster completed remote builds in B47261: Diff 110210.Sep 6 2022, 12:36 PM

andrew marked 3 inline comments as done.Sep 6 2022, 12:36 PM

This revision was not accepted when it landed; it landed in state Needs Review.Sep 27 2022, 3:15 PM

Closed by commit rG3247bc7cd652: arm64 pmap: per-domain pv chunk list (authored by andrew). · Explain Why

This revision was automatically updated to reflect the committed changes.

andrew added a commit: rG3247bc7cd652: arm64 pmap: per-domain pv chunk list.

Revision Contents
Changeset List

Path

Size

sys/

arm64/

arm64/

180 lines

Diff 111082

sys/arm64/arm64/pmap.c

Loading...