[src/trunk]: src/sys/arch/aarch64/aarch64 No assignment is needed here.

[ryo <ryo%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/7d8d3a3ea2d1
branches:  trunk
changeset: 959428:7d8d3a3ea2d1
user:      ryo <ryo%NetBSD.org@localhost>
date:      Sat Feb 13 18:13:53 2021 +0000

description:
No assignment is needed here.

the loop in pmap_page_remove() always removes the first pv,
and since the list is managed by _pmap_remove_pv(), pp->pp_pv.pv_next always points to the first.

diffstat:

 sys/arch/aarch64/aarch64/pmap.c |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (27 lines):

diff -r 97c2e49da0eb -r 7d8d3a3ea2d1 sys/arch/aarch64/aarch64/pmap.c
--- a/sys/arch/aarch64/aarch64/pmap.c   Sat Feb 13 16:43:12 2021 +0000
+++ b/sys/arch/aarch64/aarch64/pmap.c   Sat Feb 13 18:13:53 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: pmap.c,v 1.101 2021/02/01 18:12:11 ryo Exp $   */
+/*     $NetBSD: pmap.c,v 1.102 2021/02/13 18:13:53 ryo Exp $   */
 
 /*
  * Copyright (c) 2017 Ryo Shimizu <ryo%nerv.org@localhost>
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pmap.c,v 1.101 2021/02/01 18:12:11 ryo Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pmap.c,v 1.102 2021/02/13 18:13:53 ryo Exp $");
 
 #include "opt_arm_debug.h"
 #include "opt_ddb.h"
@@ -2237,7 +2237,7 @@
                        KASSERT(pv == &pp->pp_pv);
                } else {
                        KASSERT(pv == pvtmp);
-                       pp->pp_pv.pv_next = pv->pv_next;
+                       KASSERT(pp->pp_pv.pv_next == pv->pv_next);
                        pv->pv_next = pvtofree;
                        pvtofree = pv;
                }