From owner-freebsd-sparc64@FreeBSD.ORG  Thu Feb 16 17:43:43 2006
Return-Path: <owner-freebsd-sparc64@FreeBSD.ORG>
X-Original-To: freebsd-sparc64@freebsd.org
Delivered-To: freebsd-sparc64@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 7B83616A420;
	Thu, 16 Feb 2006 17:43:43 +0000 (GMT)
	(envelope-from antoine@peanut.dreadbsd.org)
Received: from barton.dreadbsd.org (peanut.dreadbsd.org [82.67.196.50])
	by mx1.FreeBSD.org (Postfix) with ESMTP id AB1A443D72;
	Thu, 16 Feb 2006 17:43:38 +0000 (GMT)
	(envelope-from antoine@peanut.dreadbsd.org)
Received: from barton.dreadbsd.org (localhost [127.0.0.1])
	by barton.dreadbsd.org (8.13.4/8.13.4) with ESMTP id k1GHhTGB005814;
	Thu, 16 Feb 2006 18:43:29 +0100 (CET)
	(envelope-from antoine@peanut.dreadbsd.org)
Received: (from antoine@localhost)
	by barton.dreadbsd.org (8.13.4/8.13.1/Submit) id k1GHhSPf005813;
	Thu, 16 Feb 2006 18:43:28 +0100 (CET) (envelope-from antoine)
Date: Thu, 16 Feb 2006 18:43:28 +0100
From: Antoine Brodin <antoine.brodin@laposte.net>
To: Marius Strobl <marius@alchemy.franken.de>
Message-Id: <20060216184328.749c4454.antoine.brodin@laposte.net>
In-Reply-To: <20060216134823.S53619@newtrinity.zeist.de>
References: <200602131150.k1DBo6S1074438@freefall.freebsd.org>
	<200602131223.51561.jhb@freebsd.org>
	<20060213193613.547d1b8f.antoine.brodin@laposte.net>
	<200602131430.11228.jhb@freebsd.org>
	<20060213213719.7767921e.antoine.brodin@laposte.net>
	<20060214094744.A81690@newtrinity.zeist.de>
	<20060214205432.38121641.antoine.brodin@laposte.net>
	<20060216134823.S53619@newtrinity.zeist.de>
X-Mailer: Sylpheed version 2.0.4 (GTK+ 2.8.12; i386-portbld-freebsd7.0)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Cc: freebsd-sparc64@freebsd.org
Subject: Re: sparc64/93226: DEBUG_LOCKS (really stack_save()) causes panics
 on sparc64
X-BeenThere: freebsd-sparc64@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Porting FreeBSD to the Sparc <freebsd-sparc64.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-sparc64>, 
	<mailto:freebsd-sparc64-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-sparc64>
List-Post: <mailto:freebsd-sparc64@freebsd.org>
List-Help: <mailto:freebsd-sparc64-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-sparc64>,
	<mailto:freebsd-sparc64-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Feb 2006 17:43:43 -0000

Marius Strobl <marius@alchemy.franken.de> wrote:
> Ok, how about the attached patch? It uses two pairs of dummy symbols
> in exception.S to determine in stack_save() whether it was one of the
> tl0_*() or tl1_*() asm functions; one pair for those in the .trap
> section that is "magically" placed at the beginning of the .text
> section via the linker script and the other pair for those in the
> regular .text section. That way we don't rely on the location of
> these functions in the kernel and don't have the performance penalty
> of *search_symbol()/*symbol_values(). For consistency db_backtrace()
> is changed to also use the new markers instead of bcmp()'ing with
> the symbol names.

If this fixes the panic, that's excellent

Cheers,

Antoine