NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: kern/60522: evbppc/RB800 hits MI pmap KASSERT failure



The following reply was made to PR kern/60522; it has been noted by GNATS.

From: Nick Hudson <nick.hudson%gmx.co.uk@localhost>
To: Rin Okuyama <rokuyama.rk%gmail.com@localhost>
Cc: gnats-bugs%netbsd.org@localhost,
 kern-bug-people%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost,
 netbsd-bugs%netbsd.org@localhost
Subject: Re: kern/60522: evbppc/RB800 hits MI pmap KASSERT failure
Date: Sat, 1 Aug 2026 15:48:33 +0100

 > On 1 Aug 2026, at 15:44, Rin Okuyama <rokuyama.rk%gmail.com@localhost> 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
 



Home | Main Index | Thread Index | Old Index