Source-Changes-HG archive

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

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



details:   https://anonhg.NetBSD.org/src/rev/12a4415d13ba
branches:  trunk
changeset: 980652:12a4415d13ba
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 a687fedd2de1 -r 12a4415d13ba 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;
                }



Home | Main Index | Thread Index | Old Index