Re: kern/60522: evbppc/RB800 hits MI pmap KASSERT failure
Nick Hudson <[email protected]> Sat, 1 Aug 2026 15:48:33 +0100
| Newsgroups | gmane.os.netbsd.bugs |
|---|---|
| Message-ID | <[email protected]> |
> On 1 Aug 2026, at 15:44, Rin Okuyama <[email protected]> wrote: >=20 > Hi, >=20 > On 2026/08/01 21:42, Nick Hudson wrote: >> Is this a regression? If so can you bisect please >=20 > Sure. The first bad commit was: >=20 > = https://github.com/NetBSD/src/commit/6162b83d2b80429d87007f1f1c451e538e577= 581 thanks. >=20 > KASSERT failure disappears if part of this commit is reverted: >=20 > ``` > diff --git a/sys/uvm/pmap/pmap.c b/sys/uvm/pmap/pmap.c > index cb0f6b3b065..eba0cbba4c7 100644 > --- a/sys/uvm/pmap/pmap.c > +++ b/sys/uvm/pmap/pmap.c > @@ -1326,7 +1326,7 @@ pmap_pte_protect(pmap_t pmap, vaddr_t sva, = vaddr_t eva, pt_entry_t *ptep, > struct vm_page_md * const mdpg =3D VM_PAGE_TO_MD(pg); >=20 > pmap_page_set_attributes(mdpg, VM_PAGEMD_MODIFIED); > - if (VM_PAGEMD_EXECPAGE_P(mdpg) || (prot & VM_PROT_EXECUTE)) { > + if (VM_PAGEMD_EXECPAGE_P(mdpg)) { > KASSERT(!VM_PAGEMD_PVLIST_EMPTY_P(mdpg)); > #ifdef PMAP_VIRTUAL_CACHE_ALIASES > if (VM_PAGEMD_CACHED_P(mdpg)) { > ``` >=20 This doesn=E2=80=99t make a lot of sense to me at first glance. Could you get the UVMHIST dumps with it in place right after the panic, = please? >=20 >> Otherwise, please enable UVMHIST and dump pmaphist and pmapexechist = after the panic. >=20