Page MenuHomeFreeBSD
Differential D36305

arm64 pmap: batch chunk removal in pmap_remove_pages
ClosedPublic
Actions

Authored by andrew on Aug 23 2022, 10:15 AM.
Tags
None
Referenced Files
F107657049: D36305.diff
Fri, Jan 17, 6:42 AM
Unknown Object (File)
Thu, Jan 9, 2:34 AM
Unknown Object (File)
Fri, Dec 20, 3:13 AM
Unknown Object (File)
Oct 29 2024, 3:20 AM
Unknown Object (File)
Oct 13 2024, 12:28 PM
Unknown Object (File)
Oct 10 2024, 7:40 AM
Unknown Object (File)
Oct 10 2024, 7:26 AM
Unknown Object (File)
Oct 10 2024, 7:25 AM
View All 53 Files
Subscribers
emaste
imp

Details

Reviewers
alc
kib
markj
mjg
manu
Group Reviewers
arm64
Commits
rG92d73b0b2564: arm64 pmap: batch chunk removal in pmap_remove_pages
Summary

As with amd64 batch chunk removal in pmap_remove_pages to move it out
of the pv list lock. This is one of the main contested locks when
running poudriere on a 160 core Ampere Altra server.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 47069
Build 43958: arc lint + arc unit

Event Timeline

andrew created this revision.Aug 23 2022, 10:15 AM
Herald added a reviewer: manu. · View Herald TranscriptAug 23 2022, 10:15 AM
Herald added subscribers: emaste, imp. · View Herald Transcript
andrew requested review of this revision.Aug 23 2022, 10:15 AM
Harbormaster completed remote builds in B47069: Diff 109705.Aug 23 2022, 10:15 AM
markj added inline comments.Aug 29 2022, 3:05 PM
sys/arm64/arm64/pmap.c
2755

Missing newline.

5418

Do pv chunks need to be freed under the pmap lock?

andrew added inline comments.Aug 31 2022, 11:18 AM
sys/arm64/arm64/pmap.c
5418

It's the same as amd64. If the comments in D21832 don't apply here I can move it.

alc added inline comments.Aug 31 2022, 4:54 PM
sys/arm64/arm64/pmap.c
5418

No, don't move it. The same reason applies here.

andrew updated this revision to Diff 110209.Sep 6 2022, 12:24 PM

Add a missing newline

Harbormaster completed remote builds in B47260: Diff 110209.Sep 6 2022, 12:24 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 27 2022, 3:15 PM
Closed by commit rG92d73b0b2564: arm64 pmap: batch chunk removal in pmap_remove_pages (authored by andrew). · Explain Why
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG92d73b0b2564: arm64 pmap: batch chunk removal in pmap_remove_pages.

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
40 lines

Diff 109705

View Options

sys/arm64/arm64/pmap.c

Loading...