From owner-freebsd-alpha@FreeBSD.ORG Fri Dec 30 13:36:29 2005 Return-Path: X-Original-To: freebsd-alpha@freebsd.org Delivered-To: freebsd-alpha@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 694B016A41F; Fri, 30 Dec 2005 13:36:29 +0000 (GMT) (envelope-from jhb@freebsd.org) Received: from speedfactory.net (mail6.speedfactory.net [66.23.216.219]) by mx1.FreeBSD.org (Postfix) with ESMTP id 7AA4043D49; Fri, 30 Dec 2005 13:36:28 +0000 (GMT) (envelope-from jhb@freebsd.org) Received: from server.baldwin.cx (unverified [66.23.211.162]) by speedfactory.net (SurgeMail 3.5b3) with ESMTP id 4871340 for multiple; Fri, 30 Dec 2005 08:34:29 -0500 Received: from zion.baldwin.cx (zion.baldwin.cx [192.168.0.7]) (authenticated bits=0) by server.baldwin.cx (8.13.4/8.13.4) with ESMTP id jBUDaPsF048806; Fri, 30 Dec 2005 08:36:27 -0500 (EST) (envelope-from jhb@freebsd.org) From: John Baldwin To: Ruslan Ermilov Date: Fri, 30 Dec 2005 08:36:24 -0500 User-Agent: KMail/1.8.3 References: <20051228163557.GE563@cicely12.cicely.de> <200512291140.19610.jhb@freebsd.org> <20051230100018.GH52756@ip.net.ua> In-Reply-To: <20051230100018.GH52756@ip.net.ua> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Message-Id: <200512300836.25701.jhb@freebsd.org> X-Virus-Scanned: ClamAV 0.87.1/1219/Wed Dec 28 17:57:59 2005 on server.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-1.4 required=4.2 tests=ALL_TRUSTED autolearn=failed version=3.1.0 X-Spam-Checker-Version: SpamAssassin 3.1.0 (2005-09-13) on server.baldwin.cx X-Server: High Performance Mail Server - http://surgemail.com r=1653887525 Cc: Bernd Walter , ticso@cicely.de, freebsd-alpha@freebsd.org Subject: Re: re(4) unaligned panic on -current X-BeenThere: freebsd-alpha@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to the Alpha List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Dec 2005 13:36:29 -0000 On Friday 30 December 2005 05:00 am, Ruslan Ermilov wrote: > On Thu, Dec 29, 2005 at 11:40:17AM -0500, John Baldwin wrote: > > On Wednesday 28 December 2005 11:49 pm, Bernd Walter wrote: > > > On Wed, Dec 28, 2005 at 11:01:47PM -0500, John Baldwin wrote: > > > > On Dec 28, 2005, at 11:35 AM, Bernd Walter wrote: > > > > >The same card works fine on an AS4100 running 5.4-STABLE. > > > > > > > > > >Booting [/boot/kernel/kernel]... > > > > >Entering /boot/kernel/kernel at 0xfffffc000033bf00... > > > > > ... > > > > >re0: port > > > > >0x11000-0x110ff mem 0x80320000-0x803200ff irq 0 at device 11.0 on > > > > > pci0 miibus1: on re0 > > > > >rgephy0: on miibus1 > > > > >rgephy0: 10baseT, 10baseT-FDX, 100baseTX, 100baseTX-FDX, > > > > >1000baseTX, 1000baseTX-FDX, auto > > > > >re0: Ethernet address: 00:40:f4:d0:8d:eb > > > > > > > > > >fatal kernel trap: > > > > > > > > > > trap entry =3D 0x4 (unaligned access fault) > > > > > cpuid =3D 0 > > > > > faulting va =3D 0xfffffc00008a472b > > > > > opcode =3D 0x28 > > > > > register =3D 0x12 > > > > > pc =3D 0xfffffc00003b0608 > > > > > ra =3D 0xfffffc00003b05cc > > > > > sp =3D 0xfffffc00007339d0 > > > > > usp =3D 0x0 > > > > > curthread =3D 0xfffffc000068b008 > > > > > pid =3D 0, comm =3D swapper > > > > > > > > > >[thread pid 0 tid 0 ] > > > > >Stopped at re_init_locked+0xd8: jsr ra, > > > > >(pv),re_init_locked+0xdc > > > > > > > > > >db> bt > > > > >Tracing pid 0 tid 0 td 0xfffffc000068b008 > > > > >re_init_locked() at re_init_locked+0xd8 > > > > >re_diag() at re_diag+0x178 > > > > > > > > My first guess would be Ruslan's IF_LLADDR changes. If so, you can > > > > try doing a bcopy to a char array as a workaround similar to the > > > > recent changes to de(4) and dc(4) to fix similar panics on Alpha. = It > > > > might be something else though. If you could pull up gdb on your > > > > kernel.debug and do 'l *re_init_locked+0xd8' to see what file/line > > > > that corresponds to that would be helpful. > > > > > > Your guess looks right - will try your bcopy suggestion. > > > > > > [54]cicely12# gdb kernel.debug > > > GNU gdb 6.1.1 [FreeBSD] > > > Copyright 2004 Free Software Foundation, Inc. > > > GDB is free software, covered by the GNU General Public License, and > > > you are welcome to change it and/or distribute copies of it under > > > certain conditions. Type "show copying" to see the conditions. > > > There is absolutely no warranty for GDB. Type "show warranty" for > > > details. This GDB was configured as "alpha-marcel-freebsd"... > > > (gdb) l *re_init_locked+0xd8 > > > 0xfffffc00003b0608 is in re_init_locked (../../../dev/re/if_re.c:2127= ). > > > 2122 * Init our MAC address. Even though the chipset > > > 2123 * documentation doesn't mention it, we need to enter > > > "Config 2124 * register write enable" mode to modify the = ID > > > registers. 2125 */ > > > 2126 CSR_WRITE_1(sc, RL_EECMD, RL_EEMODE_WRITECFG); > > > 2127 CSR_WRITE_STREAM_4(sc, RL_IDR0, > > > 2128 *(u_int32_t *)(&IF_LLADDR(sc->rl_ifp)[0])); > > > 2129 CSR_WRITE_STREAM_4(sc, RL_IDR4, > > > 2130 *(u_int32_t *)(&IF_LLADDR(sc->rl_ifp)[4])); > > > 2131 CSR_WRITE_1(sc, RL_EECMD, RL_EEMODE_OFF); > > > (gdb) > > > > Hmm, even worse is that IF_LLADDR() might not be valid yet since from t= he > > trace it looked like re_diag() was being called from re_attach() and th= us > > likely before ether_ifattach(). You'll have to somehow get the copy of > > the MAC address via the softc if this function is called too early like= I > > did for de(4). Try the bcopy first though. > > What do you mean? > > 1236 /* > 1237 * Call MI attach routine. > 1238 */ > 1239 ether_ifattach(ifp, eaddr); > 1240 > 1241 /* Perform hardware diagnostic. */ > 1242 error =3D re_diag(sc); Ah, most drivers call ether_ifattach() last since otherwise you open yourse= lf=20 up to race conditions since, e.g. a user thread could have done an ifconfig= =20 up before this thread returns from ether_ifattach() to call re_diag(). =2D-=20 John Baldwin =A0<>< =A0http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" =A0=3D =A0http://www.FreeBSD.org