Page MenuHomeFreeBSD
Differential D32339

Only demote when needed in the arm64 pmap_change_props_locked
ClosedPublic
Actions

Authored by andrew on Oct 6 2021, 4:46 PM.
Tags
None
Referenced Files
F83577598: D32339.id96352.diff
Sun, May 12, 3:29 AM
F83520589: D32339.id96405.diff
Sat, May 11, 4:22 PM
Unknown Object (File)
Fri, May 10, 12:18 AM
Unknown Object (File)
Wed, May 1, 1:15 AM
Unknown Object (File)
Sun, Apr 28, 5:30 PM
Unknown Object (File)
Mar 7 2024, 7:41 PM
Unknown Object (File)
Dec 12 2023, 4:45 PM
Unknown Object (File)
Dec 3 2023, 10:58 AM
View All 27 Files
Subscribers
emaste
imp

Details

Reviewers
alc
kib
markj
manu
Group Reviewers
arm64
Commits
rG39dbb494404f: Only demote when needed in the arm64 pmap_change_props_locked
rG806a88e74200: Only demote when needed in the arm64 pmap_change_props_locked
Summary

When changing page table properties there is no need to demote a
level 1 or level 2 block if we are changing the entire memory range the
block is mapping. In this case just change the block directly.

Diff Detail

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

Event Timeline

andrew created this revision.Oct 6 2021, 4:46 PM
Herald added a reviewer: manu. · View Herald TranscriptOct 6 2021, 4:46 PM
Herald added subscribers: emaste, imp. · View Herald Transcript
andrew requested review of this revision.Oct 6 2021, 4:46 PM
Harbormaster completed remote builds in B41986: Diff 96352.Oct 6 2021, 4:47 PM
alc added inline comments.Oct 7 2021, 7:47 AM
sys/arm64/arm64/pmap.c
6113

L3_SIZE -> pte_size?

andrew updated this revision to Diff 96405.Oct 7 2021, 10:20 AM

Fix the size in cpu_dcache_wbinv_range

Harbormaster completed remote builds in B42000: Diff 96405.Oct 7 2021, 10:20 AM
markj accepted this revision.Oct 7 2021, 4:09 PM
markj added inline comments.
sys/arm64/arm64/pmap.c
6072

We should have an explicit /* FALLTHROUGH */ annotation here and below.

This revision is now accepted and ready to land.Oct 7 2021, 4:09 PM
alc accepted this revision.Oct 8 2021, 6:51 AM
kib accepted this revision.Oct 8 2021, 10:31 AM
Closed by commit rG806a88e74200: Only demote when needed in the arm64 pmap_change_props_locked (authored by andrew). · Explain WhyOct 11 2021, 10:44 AM
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rG806a88e74200: Only demote when needed in the arm64 pmap_change_props_locked.
andrew added a commit: rG39dbb494404f: Only demote when needed in the arm64 pmap_change_props_locked.Oct 25 2021, 1:55 PM

Revision Contents
Changeset List

PathSize
sys/
arm64/
arm64/
74 lines

Diff 96352

View Options

sys/arm64/arm64/pmap.c

Loading...