riscv: Rename struct pmap's pm_l1 field to pm_top
ClosedPublic
Actions

Authored by markj on Feb 14 2022, 10:29 PM.

Details

Reviewers

jhb
alc

Commits

rG82f4e0d0f064: riscv: Rename struct pmap's pm_l1 field to pm_top

Summary

In SV48 mode, the top-level page will be an L0 page rather than an L1
page. Rename the field accordingly. No functional change intended.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 44499
Build 41387: arc lint + arc unit

Event Timeline

markj created this revision.Feb 14 2022, 10:29 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 14 2022, 10:29 PM

markj requested review of this revision.Feb 14 2022, 10:29 PM

Harbormaster completed remote builds in B44439: Diff 102756.Feb 14 2022, 10:29 PM

markj added a child revision: D34271: riscv: Use generic CSR macros for writing SATP.Feb 14 2022, 10:29 PM

Is it L-1 for SV57? :-P

This revision is now accepted and ready to land.Feb 14 2022, 10:56 PM

In D34270#775783, @jhb wrote:

Is it L-1 for SV57? :-P

I think at that point it'll be time to factor some of the page table management code out into an MI layer. Then I'd use L0 instead of L3 to refer to leaf PTPs, L1 instead of L2, etc..

alc accepted this revision.Feb 16 2022, 5:12 PM

Fix a mismerge.

This revision now requires review to proceed.Feb 18 2022, 2:27 PM

Harbormaster completed remote builds in B44499: Diff 102934.Feb 18 2022, 2:27 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2022, 2:40 PM

Closed by commit rG82f4e0d0f064: riscv: Rename struct pmap's pm_l1 field to pm_top (authored by markj). · Explain Why

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG82f4e0d0f064: riscv: Rename struct pmap's pm_l1 field to pm_top.

Revision Contents
Changeset List

Path

Size

sys/

riscv/

include/

pmap.h

2 lines

riscv/

pmap.c

37 lines

Diff 102934

View Options

sys/riscv/include/pmap.h