Page MenuHomeFreeBSD
Differential D54022

riscv/pmap: Handle superpages in pmap_extract_and_hold()
ClosedPublic
Actions

Authored by markj on Mon, Dec 1, 9:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 20, 9:27 PM
Unknown Object (File)
Fri, Dec 19, 3:37 PM
Unknown Object (File)
Fri, Dec 19, 1:01 AM
Unknown Object (File)
Thu, Dec 18, 11:30 PM
Unknown Object (File)
Tue, Dec 16, 8:37 AM
Unknown Object (File)
Tue, Dec 16, 5:48 AM
Unknown Object (File)
Mon, Dec 15, 7:10 PM
Unknown Object (File)
Sun, Dec 14, 10:18 PM
View All 25 Files
Subscribers
imp
riscv

Details

Reviewers
kib
alc
Group Reviewers
riscv
Commits
rGf50f8cbb401b: riscv/pmap: Handle superpages in pmap_extract_and_hold()
rG20a0a2f8f36f: riscv/pmap: Handle superpages in pmap_extract_and_hold()
Summary

Without this change, vm_fault_quick_hold_pages() falls back to the slow
fault handler() when it encounters a superpage mapping.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 68974
Build 65857: arc lint + arc unit

Event Timeline

markj created this revision.Mon, Dec 1, 9:53 PM
Herald added subscribers: riscv, imp. · View Herald TranscriptMon, Dec 1, 9:53 PM
markj requested review of this revision.Mon, Dec 1, 9:53 PM
Harbormaster completed remote builds in B68972: Diff 167388.Mon, Dec 1, 9:53 PM
markj updated this revision to Diff 167389.Mon, Dec 1, 9:55 PM

Return failure if the L2 entry is invalid

Harbormaster completed remote builds in B68973: Diff 167389.Mon, Dec 1, 9:55 PM
markj updated this revision to Diff 167390.Mon, Dec 1, 9:56 PM

Fix handling of insufficient permissions.

Harbormaster completed remote builds in B68974: Diff 167390.Mon, Dec 1, 9:56 PM
markj edited the summary of this revision. (Show Details)Mon, Dec 1, 10:02 PM
markj added a reviewer: riscv.
markj edited the summary of this revision. (Show Details)Mon, Dec 1, 10:05 PM
markj added reviewers: kib, alc.Mon, Dec 1, 10:08 PM
kib accepted this revision.Mon, Dec 1, 10:15 PM
This revision is now accepted and ready to land.Mon, Dec 1, 10:15 PM
alc added inline comments.Tue, Dec 2, 5:28 PM
sys/riscv/riscv/pmap.c
1154

The l3p != NULL is now redundant.

1157

This goto is redundant.

markj updated this revision to Diff 167437.Tue, Dec 2, 5:34 PM
markj marked 2 inline comments as done.

Handle Alan's comments.

This revision now requires review to proceed.Tue, Dec 2, 5:34 PM
Harbormaster completed remote builds in B69001: Diff 167437.Tue, Dec 2, 5:34 PM
alc accepted this revision.Tue, Dec 2, 5:36 PM
This revision is now accepted and ready to land.Tue, Dec 2, 5:36 PM
kib accepted this revision.Tue, Dec 2, 10:15 PM
kib added inline comments.
sys/riscv/riscv/pmap.c
1161

Then this part could become just under 'else' and then goto is not needed at all.
But the current version is fine as well.

markj updated this revision to Diff 167445.Tue, Dec 2, 11:27 PM
markj marked an inline comment as done.

Avoid gotos

This revision now requires review to proceed.Tue, Dec 2, 11:27 PM
Harbormaster completed remote builds in B69005: Diff 167445.Tue, Dec 2, 11:27 PM
kib accepted this revision.Tue, Dec 2, 11:45 PM

Thanks.

This revision is now accepted and ready to land.Tue, Dec 2, 11:45 PM
alc accepted this revision.Wed, Dec 3, 8:44 AM
Closed by commit rG20a0a2f8f36f: riscv/pmap: Handle superpages in pmap_extract_and_hold() (authored by markj). · Explain WhyWed, Dec 3, 2:44 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG20a0a2f8f36f: riscv/pmap: Handle superpages in pmap_extract_and_hold().
markj added a commit: rGf50f8cbb401b: riscv/pmap: Handle superpages in pmap_extract_and_hold().Thu, Dec 11, 6:06 PM

Revision Contents
Changeset List

PathSize
sys/
riscv/
riscv/
19 lines

Diff 167390

View Options

sys/riscv/riscv/pmap.c

Loading...