From owner-freebsd-ia64@FreeBSD.ORG  Tue Jun 10 01:53:12 2003
Return-Path: <owner-freebsd-ia64@FreeBSD.ORG>
Delivered-To: freebsd-ia64@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 5B0EE37B401
	for <freebsd-ia64@freebsd.org>; Tue, 10 Jun 2003 01:53:12 -0700 (PDT)
Received: from plim.fujitsu-siemens.com (plim.fujitsu-siemens.com
	[217.115.66.8])	by mx1.FreeBSD.org (Postfix) with ESMTP id 790F143FD7
	for <freebsd-ia64@freebsd.org>; Tue, 10 Jun 2003 01:53:10 -0700 (PDT)
	(envelope-from alan.robinson@fujitsu-siemens.com)
Received: from trulli.pdb.fsc.net (this.is.a.RFC1918.address [172.25.96.53]
	(may be forged))h5A8r8F13185;	Tue, 10 Jun 2003 10:53:08 +0200
Received: from athen.mch.fsc.net (backbay.mch.fsc.net [172.25.94.188])
	by trulli.pdb.fsc.net (8.11.6/8.11.6) with ESMTP id h5A8r7g15384;
	Tue, 10 Jun 2003 10:53:07 +0200
Received: from sanpedro.mch.fsc.net (sanpedro [172.25.95.234])
	by athen.mch.fsc.net (8.11.6/8.11.6) with ESMTP id h5A8r6I29267;
	Tue, 10 Jun 2003 10:53:07 +0200 (MDT)
Received: (from robin@localhost)
	by sanpedro.mch.fsc.net (8.9.3p2/8.9.3/Debian 8.9.3-21) id KAA19605;
	Tue, 10 Jun 2003 10:53:06 +0200
From: Alan Robinson <alan.robinson@fujitsu-siemens.com>
Date: Tue, 10 Jun 2003 10:53:06 +0200
To: Marcel Moolenaar <marcel@xcllnt.net>
Message-ID: <20030610105306.A18863@fujitsu-siemens.com>
References: <20030606183130.A19592@fujitsu-siemens.com>
	<20030606182015.GC569@dhcp01.pn.xcllnt.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2.5i
In-Reply-To: <20030606182015.GC569@dhcp01.pn.xcllnt.net>;
	from marcel@xcllnt.net on Fri, Jun 06, 2003 at 11:20:15AM -0700
X-sent-by-me: robin@sanpedro
cc: freebsd-ia64@freebsd.org
Subject: Re: DDB and SMP causes Unaligned Reference
X-BeenThere: freebsd-ia64@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
Reply-To: Alan.Robinson@fujitsu-siemens.com
List-Id: Porting FreeBSD to the IA-64 <freebsd-ia64.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ia64>,
	<mailto:freebsd-ia64-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-ia64>
List-Post: <mailto:freebsd-ia64@freebsd.org>
List-Help: <mailto:freebsd-ia64-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ia64>,
	<mailto:freebsd-ia64-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Jun 2003 08:53:12 -0000

On Fri, Jun 06, 2003 at 11:20:15AM -0700, Marcel Moolenaar wrote:
> On Fri, Jun 06, 2003 at 06:31:30PM +0200, Alan Robinson wrote:
> > 
> > The cr.iip points into swapctx (or savectx) which was called from
> > from interrupt() as follows....
> > 
> >         } else if (vector == ipi_vector[IPI_STOP]) {
> >                 u_int32_t mybit = PCPU_GET(cpumask);
> > 
> >                 CTR1(KTR_SMP, "IPI_STOP, cpuid=%d", PCPU_GET(cpuid));
> >                 savectx(PCPU_GET(pcb));
> 
> We don't set pc_pcb on ia64. The line above should read:
> 	savectx(curthread->td_pcb);
> 
Tried it.
> 
> Try it and let me know if it works. I'll commit it then.
> 
Works.
Works.
(It still causes the strange double entry into DDB)

It would also be a good idea to remove the unneeded pc_pcb line with its
incorrect comment from src/sys/ia64/include/pcpu.h

Alan