d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

riscv: Handle invalid L2 entries in pmap_extract()

Browse Source 
While here, eliminate a single-use local variable.

PR:		266103
Reviewed by:	mhorne
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D36395
This commit is contained in:
Mark Johnston 2022-09-29 13:07:26 -04:00
parent 76aebeab7c
commit ec21f85ab5
1 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
sys/riscv/riscv/pmap.c
View File

@ -903,27 +903,25 @@ vm_paddr_t
pmap_extract(pmap_t pmap, vm_offset_t va)
{
pd_entry_t *l2p, l2;
pt_entry_t *l3p, l3;
pt_entry_t *l3p;
vm_paddr_t pa;
pa = 0;
PMAP_LOCK(pmap);
/*
* Start with the l2 tabel. We are unable to allocate
* pages in the l1 table.
* Start with an L2 lookup, L1 superpages are currently not implemented.
*/
PMAP_LOCK(pmap);
l2p = pmap_l2(pmap, va);
if (l2p != NULL) {
l2 = pmap_load(l2p);
if ((l2 & PTE_RX) == 0) {
if (l2p != NULL && ((l2 = pmap_load(l2p)) & PTE_V) != 0) {
if ((l2 & PTE_RWX) == 0) {
l3p = pmap_l2_to_l3(l2p, va);
if (l3p != NULL) {
l3 = pmap_load(l3p);
pa = PTE_TO_PHYS(l3);
pa = PTE_TO_PHYS(pmap_load(l3p));
pa |= (va & L3_OFFSET);
}
} else {
/* L2 is superpages */
/* L2 is a superpage mapping. */
pa = L2PTE_TO_PHYS(l2);
pa |= (va & L2_OFFSET);
}