Source-Changes-HG archive

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

[src/trunk]: src/sys/dev/ofw Remove !cold KASSERT - it does not compile on al...



details:   https://anonhg.NetBSD.org/src/rev/1f8a3230081c
branches:  trunk
changeset: 1011326:1f8a3230081c
user:      martin <martin%NetBSD.org@localhost>
date:      Fri Jun 26 10:14:32 2020 +0000

description:
Remove !cold KASSERT - it does not compile on all kernels, and it is not the
right thing to test for anyway. XXX should we panic instead? Are "compatible"
strings this long happening in real devices?

diffstat:

 sys/dev/ofw/ofw_subr.c |  5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diffs (26 lines):

diff -r 95b92a916a04 -r 1f8a3230081c sys/dev/ofw/ofw_subr.c
--- a/sys/dev/ofw/ofw_subr.c    Fri Jun 26 10:06:57 2020 +0000
+++ b/sys/dev/ofw/ofw_subr.c    Fri Jun 26 10:14:32 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ofw_subr.c,v 1.38 2020/06/25 22:50:56 rin Exp $        */
+/*     $NetBSD: ofw_subr.c,v 1.39 2020/06/26 10:14:32 martin Exp $     */
 
 /*
  * Copyright 1998
@@ -34,7 +34,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ofw_subr.c,v 1.38 2020/06/25 22:50:56 rin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ofw_subr.c,v 1.39 2020/06/26 10:14:32 martin Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -109,7 +109,6 @@
                return (-1);
 
        if (len > sizeof(sbuf)) {
-               KASSERT(!cold);
                buf = malloc(len, M_TEMP, M_WAITOK);
                allocated = 1;
        } else {



Home | Main Index | Thread Index | Old Index