[src/trunk]: src/sys/kern explain how a printf might happen (since it has bit...

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

details:   https://anonhg.NetBSD.org/src/rev/2c064d63ebe5
branches:  trunk
changeset: 328139:2c064d63ebe5
user:      christos <christos%NetBSD.org@localhost>
date:      Thu Mar 27 21:09:33 2014 +0000

description:
explain how a printf might happen (since it has bitten more than one person)

diffstat:

 sys/kern/kern_sysctl.c |  9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)

diffs (30 lines):

diff -r 013524e4d79c -r 2c064d63ebe5 sys/kern/kern_sysctl.c
--- a/sys/kern/kern_sysctl.c    Thu Mar 27 19:34:39 2014 +0000
+++ b/sys/kern/kern_sysctl.c    Thu Mar 27 21:09:33 2014 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: kern_sysctl.c,v 1.248 2014/03/01 17:27:48 dsl Exp $    */
+/*     $NetBSD: kern_sysctl.c,v 1.249 2014/03/27 21:09:33 christos Exp $       */
 
 /*-
  * Copyright (c) 2003, 2007, 2008 The NetBSD Foundation, Inc.
@@ -68,7 +68,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: kern_sysctl.c,v 1.248 2014/03/01 17:27:48 dsl Exp $");
+__KERNEL_RCSID(0, "$NetBSD: kern_sysctl.c,v 1.249 2014/03/27 21:09:33 christos Exp $");
 
 #include "opt_defcorename.h"
 #include "ksyms.h"
@@ -2105,6 +2105,11 @@
        pnode = root;
        error = sysctl_locate(NULL, &name[0], namelen - 1, &pnode, &ni);
        if (error) {
+               /*
+                * XXX: If you are seeing this printf in early bringup
+                * stages, perhaps your setfault is not functioning and
+                * thus kcopy() is mis-behaving.
+                */
                printf("sysctl_createv: sysctl_locate(%s) returned %d\n",
                       nnode.sysctl_name, error);
                sysctl_unlock();