Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 11 Dec 2025 18:05:51 +0000
From:      Mark Johnston <markj@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: f50f8cbb401b - stable/15 - riscv/pmap: Handle superpages in pmap_extract_and_hold()
Message-ID:  <693b07ff.21aaf.7fe3f4f8@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help

The branch stable/15 has been updated by markj:

URL: https://cgit.FreeBSD.org/src/commit/?id=f50f8cbb401b35d0fad17ae1f84c56040bf12d2c

commit f50f8cbb401b35d0fad17ae1f84c56040bf12d2c
Author:     Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2025-12-03 13:41:30 +0000
Commit:     Mark Johnston <markj@FreeBSD.org>
CommitDate: 2025-12-11 17:59:09 +0000

    riscv/pmap: Handle superpages in pmap_extract_and_hold()
    
    Without this change, vm_fault_quick_hold_pages() falls back to the slow
    fault handler when it encounters a superpage mapping.
    
    Reviewed by:    alc, kib
    Reported and tested by: br
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D54022
    
    (cherry picked from commit 20a0a2f8f36f5db704b674106fa61d8d95b6b071)
---
 sys/riscv/riscv/pmap.c | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/sys/riscv/riscv/pmap.c b/sys/riscv/riscv/pmap.c
index 26efaecc64d1..0deb8b93a6dc 100644
--- a/sys/riscv/riscv/pmap.c
+++ b/sys/riscv/riscv/pmap.c
@@ -1134,19 +1134,29 @@ pmap_extract(pmap_t pmap, vm_offset_t va)
 vm_page_t
 pmap_extract_and_hold(pmap_t pmap, vm_offset_t va, vm_prot_t prot)
 {
+	pd_entry_t *l2p, l2;
 	pt_entry_t *l3p, l3;
 	vm_page_t m;
 
 	m = NULL;
 	PMAP_LOCK(pmap);
-	l3p = pmap_l3(pmap, va);
-	if (l3p != NULL && (l3 = pmap_load(l3p)) != 0) {
-		if ((l3 & PTE_W) != 0 || (prot & VM_PROT_WRITE) == 0) {
-			m = PTE_TO_VM_PAGE(l3);
-			if (!vm_page_wire_mapped(m))
-				m = NULL;
+	l2p = pmap_l2(pmap, va);
+	if (l2p == NULL || ((l2 = pmap_load(l2p)) & PTE_V) == 0) {
+		;
+	} else if ((l2 & PTE_RWX) != 0) {
+		if ((l2 & PTE_W) != 0 || (prot & VM_PROT_WRITE) == 0) {
+			m = PHYS_TO_VM_PAGE(L2PTE_TO_PHYS(l2) +
+			    (va & L2_OFFSET));
+		}
+	} else {
+		l3p = pmap_l2_to_l3(l2p, va);
+		if ((l3 = pmap_load(l3p)) != 0) {
+			if ((l3 & PTE_W) != 0 || (prot & VM_PROT_WRITE) == 0)
+				m = PTE_TO_VM_PAGE(l3);
 		}
 	}
+	if (m != NULL && !vm_page_wire_mapped(m))
+		m = NULL;
 	PMAP_UNLOCK(pmap);
 	return (m);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?693b07ff.21aaf.7fe3f4f8>