From owner-p4-projects@FreeBSD.ORG Mon Feb 28 17:36:56 2011 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id BD6B51065674; Mon, 28 Feb 2011 17:36:56 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 801521065670 for ; Mon, 28 Feb 2011 17:36:56 +0000 (UTC) (envelope-from mjacob@freebsd.org) Received: from skunkworks.freebsd.org (skunkworks.freebsd.org [IPv6:2001:4f8:fff6::2d]) by mx1.freebsd.org (Postfix) with ESMTP id 534B98FC0A for ; Mon, 28 Feb 2011 17:36:56 +0000 (UTC) Received: from skunkworks.freebsd.org (localhost [127.0.0.1]) by skunkworks.freebsd.org (8.14.4/8.14.4) with ESMTP id p1SHauxW014370 for ; Mon, 28 Feb 2011 17:36:56 GMT (envelope-from mjacob@freebsd.org) Received: (from perforce@localhost) by skunkworks.freebsd.org (8.14.4/8.14.4/Submit) id p1SHauVI014367 for perforce@freebsd.org; Mon, 28 Feb 2011 17:36:56 GMT (envelope-from mjacob@freebsd.org) Date: Mon, 28 Feb 2011 17:36:56 GMT Message-Id: <201102281736.p1SHauVI014367@skunkworks.freebsd.org> X-Authentication-Warning: skunkworks.freebsd.org: perforce set sender to mjacob@freebsd.org using -f From: Matt Jacob To: Perforce Change Reviews Precedence: bulk Cc: Subject: PERFORCE change 189287 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 28 Feb 2011 17:36:57 -0000 http://p4web.freebsd.org/@@189287?ac=10 Change 189287 by mjacob@mjacob-sandbox on 2011/02/28 17:36:50 Remove my local sio_suppressed hack- it gets in the way of running a clean 'make universe' and it isn't worth fixing for all architecture. Affected files ... .. //depot/projects/mjacob-dev/sys/dev/uart/uart_tty.c#4 edit .. //depot/projects/mjacob-dev/sys/kern/kern_shutdown.c#3 edit .. //depot/projects/mjacob-dev/sys/kern/subr_kdb.c#4 edit Differences ... ==== //depot/projects/mjacob-dev/sys/dev/uart/uart_tty.c#4 (text+ko) ==== @@ -40,7 +40,6 @@ #include #include #include -#include #include #include @@ -108,15 +107,10 @@ uart_term(cp->cn_arg); } -int sio_suppressed = 0; -SYSCTL_INT(_machdep, OID_AUTO, sio_suppressed, CTLFLAG_RW, &sio_suppressed, 0, ""); - static void uart_cnputc(struct consdev *cp, int c) { - if (sio_suppressed) { - return; - } + uart_putc(cp->cn_arg, c); } ==== //depot/projects/mjacob-dev/sys/kern/kern_shutdown.c#3 (text+ko) ==== @@ -512,8 +512,6 @@ /* NOTREACHED */ /* assuming reset worked */ } -extern int sio_suppressed; - /* * Panic is called on unresolvable fatal errors. It prints "panic: mesg", * and then reboots. If we are called twice, then we avoid trying to sync @@ -530,7 +528,6 @@ va_list ap; static char buf[256]; - sio_suppressed = 0; critical_enter(); #ifdef SMP /* ==== //depot/projects/mjacob-dev/sys/kern/subr_kdb.c#4 (text+ko) ==== @@ -271,7 +271,6 @@ return (brk); } -extern int sio_suppressed; /* * Print a backtrace of the calling thread. The backtrace is generated by * the selected debugger, provided it supports backtraces. If no debugger @@ -282,8 +281,6 @@ void kdb_backtrace(void) { - int osup = sio_suppressed; - sio_suppressed = 0; if (kdb_dbbe != NULL && kdb_dbbe->dbbe_trace != NULL) { printf("KDB: stack backtrace:\n"); @@ -298,7 +295,6 @@ stack_print_ddb(&st); } #endif - sio_suppressed = osup; } /* @@ -331,8 +327,6 @@ void kdb_enter(const char *why, const char *msg) { - int osup = sio_suppressed; - sio_suppressed = 0; if (kdb_dbbe != NULL && kdb_active == 0) { if (msg != NULL) @@ -341,7 +335,6 @@ breakpoint(); kdb_why = KDB_WHY_UNSET; } - sio_suppressed = osup; } /* @@ -510,8 +503,6 @@ int did_stop_cpus; #endif int handled; - int osup = sio_suppressed; - sio_suppressed = 0; be = kdb_dbbe; if (be == NULL || be->dbbe_trap == NULL) @@ -557,6 +548,5 @@ intr_restore(intr); - sio_suppressed = osup; return (handled); }