PR/56381 CVS commit: src/sys/arch/sh3/sh3

["Rin Okuyama" <rin%netbsd.org@localhost>]

The following reply was made to PR port-sh3/56381; it has been noted by GNATS.

From: "Rin Okuyama" <rin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/56381 CVS commit: src/sys/arch/sh3/sh3
Date: Thu, 2 Sep 2021 07:55:56 +0000

 Module Name:	src
 Committed By:	rin
 Date:		Thu Sep  2 07:55:56 UTC 2021
 
 Modified Files:
 	src/sys/arch/sh3/sh3: pmap.c
 
 Log Message:
 PR port-sh3/56381
 
 pmap_enter() returns ENOMEM if __pmap_pte_alloc() fails and PMAP_CANFAIL
 flag is specified. In this case, remove pv via __pmap_pv_remove() if it is
 added to p-v map list via __pmap_pv_enter().
 
 Otherwise, pmap becomes an inconsistent state, which results in an infinite
 loop in pmap_page_protect(), as reported in the PR.
 
 Also, KASSERT's are added for sure, in order to detect the infinite loops.
 
 Great thanks to chs@ for finding out this bug!!
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.85 -r1.86 src/sys/arch/sh3/sh3/pmap.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.