Page MenuHomeFreeBSD
Differential D34277

riscv: Maintain the allpmaps list only in SV39 mode
ClosedPublic
Actions

Authored by markj on Feb 14 2022, 10:30 PM.
Tags
None
Referenced Files
F166377109: D34277.diff
Thu, Aug 13, 6:12 AM
Unknown Object (File)
Wed, Aug 12, 1:32 PM
Unknown Object (File)
Sat, Aug 8, 8:33 AM
Unknown Object (File)
Sat, Aug 8, 12:05 AM
Unknown Object (File)
Tue, Aug 4, 7:13 AM
Unknown Object (File)
Tue, Aug 4, 6:46 AM
Unknown Object (File)
Tue, Aug 4, 6:10 AM
Unknown Object (File)
Tue, Aug 4, 4:07 AM
View All Files
Subscribers
imp
jhb

Details

Reviewers
jhb
Commits
rGceed61483ca9: riscv: Maintain the allpmaps list only in SV39 mode
Summary

When four-level page tables are used, there is no need to distribute
updates to the top-level page to all pmaps.

Diff Detail

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

Event Timeline

markj created this revision.Feb 14 2022, 10:30 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 14 2022, 10:30 PM
markj requested review of this revision.Feb 14 2022, 10:30 PM
Harbormaster completed remote builds in B44446: Diff 102763.Feb 14 2022, 10:30 PM
markj added a parent revision: D34276: riscv: Add pmap helper functions required by four-level page tables.Feb 14 2022, 10:31 PM
markj added a child revision: D34278: riscv: Handle four-level page tables in various pmap traversal routines.
jhb added a subscriber: jhb.Feb 15 2022, 12:13 AM
jhb added inline comments.
sys/riscv/riscv/pmap.c
1289

Maybe just leave this outside of the if-else?

markj added inline comments.Feb 15 2022, 2:29 PM
sys/riscv/riscv/pmap.c
1289

It looks slightly odd since I haven't rebased on D34267. Once that's done the redundancy will be gone.

markj updated this revision to Diff 102798.Feb 15 2022, 3:10 PM

Rebase

Harbormaster completed remote builds in B44464: Diff 102798.Feb 15 2022, 3:10 PM
jhb accepted this revision.Feb 15 2022, 5:40 PM
This revision is now accepted and ready to land.Feb 15 2022, 5:40 PM
markj updated this revision to Diff 102933.Feb 18 2022, 2:26 PM

Copy only a single L0 entry in pmap_pinit().

This revision now requires review to proceed.Feb 18 2022, 2:26 PM
Harbormaster completed remote builds in B44498: Diff 102933.Feb 18 2022, 2:26 PM
markj updated this revision to Diff 102935.Feb 18 2022, 2:28 PM

Rebase.

Harbormaster completed remote builds in B44500: Diff 102935.Feb 18 2022, 2:28 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2022, 2:40 PM
Closed by commit rGceed61483ca9: riscv: Maintain the allpmaps list only in SV39 mode (authored by markj). · Explain Why
This revision was automatically updated to reflect the committed changes.
markj added a commit: rGceed61483ca9: riscv: Maintain the allpmaps list only in SV39 mode.

Revision Contents
Changeset List

PathSize
sys/
riscv/
riscv/
52 lines

Diff 102933

View Options

sys/riscv/riscv/pmap.c

Loading...