From owner-cvs-all@FreeBSD.ORG  Sun Feb 22 03:13:15 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id E1D2716A4DB; Sun, 22 Feb 2004 03:13:15 -0800 (PST)
Received: from white.imgsrc.co.jp (ns.imgsrc.co.jp [210.226.20.2])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id B426643D1D; Sun, 22 Feb 2004 03:13:15 -0800 (PST)
	(envelope-from kuriyama@imgsrc.co.jp)
Received: from localhost (localhost [127.0.0.1])
	by white.imgsrc.co.jp (Postfix) with ESMTP
	id 07C542840D8; Sun, 22 Feb 2004 19:12:21 +0900 (JST)
Received: from black.imgsrc.co.jp (black.imgsrc.co.jp
	[IPv6:2001:218:422:2::130])
	by white.imgsrc.co.jp (Postfix) with ESMTP
	id CFAE02840CE; Sun, 22 Feb 2004 19:12:19 +0900 (JST)
Received: from black.imgsrc.co.jp (black.imgsrc.co.jp
	[IPv6:2001:218:422:2::130])
	by black.imgsrc.co.jp (Postfix) with ESMTP
	id 9BC071E49FB; Sun, 22 Feb 2004 19:12:19 +0900 (JST)
Date: Sun, 22 Feb 2004 19:12:19 +0900
Message-ID: <7mptc7jud8.wl@black.imgsrc.co.jp>
From: Jun Kuriyama <kuriyama@imgsrc.co.jp>
To: Bruce Evans <bde@zeta.org.au>
In-Reply-To: <7m3c99vp6i.wl@black.imgsrc.co.jp>
References: <200401192127.i0JLRBL3041817@repoman.freebsd.org>
	<20040120175334.W3279@gamplex.bde.org>
	<20040213035557.M24031@gamplex.bde.org>
	<7mfzdexjzs.wl@black.imgsrc.co.jp>
	<20040215094444.F6303@gamplex.bde.org>
	<7m3c99vp6i.wl@black.imgsrc.co.jp>
User-Agent: Wanderlust/2.10.1 (Watching The Wheels) SEMI/1.14.5 (Awara-Onsen)
	FLIM/1.14.5 (Demachiyanagi) APEL/10.6 Emacs/21.3 (i386--freebsd) MULE/5.0
	(SAKAKI)
MIME-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen")
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
X-Virus-Scanned: by AMaViS snapshot-20020531
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: Poul-Henning Kamp <phk@FreeBSD.org>
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/alpha/alpha support.s src/sys/i386/i386
	swtch.s src/sys/kern kern_shutdown.c src/sys/sys systm.h
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 22 Feb 2004 11:13:16 -0000

At Tue, 17 Feb 2004 21:57:41 +0900,
kuriyama wrote:
> But I personally feel this should be back out after re-reading and we
> should tackle traceback() improvements.

I checked something around backtrace().  A db_print_backtrace() is
depending on DDB, but as Bruce said, DDB is dangerous for default
feature for -stable release.

=46rom my (non kernel hacker) understanding, I'm not sure whether this
is possible and preferred approach or not.  How about this approach?

o Add DDB_BACKTRACE_ONLY option.
o Compile minimal code for db_print_backtrace() into the kernel (some
  #ifdef storm?) when this option is enabled without DDB.
o Call backtrace() if this option is enabled like as DDB_TRACE.
o Enable this option in GENERIC by default.


--=20
Jun Kuriyama <kuriyama@imgsrc.co.jp> // IMG SRC, Inc.
             <kuriyama@FreeBSD.org> // FreeBSD Project