From owner-freebsd-sparc64@FreeBSD.ORG Tue Mar 22 21:38:58 2011 Return-Path: Delivered-To: freebsd-sparc64@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 228DC106566B; Tue, 22 Mar 2011 21:38:58 +0000 (UTC) (envelope-from cheeky.m@live.com) Received: from bay0-omc1-s28.bay0.hotmail.com (bay0-omc1-s28.bay0.hotmail.com [65.54.190.39]) by mx1.freebsd.org (Postfix) with ESMTP id EFD528FC12; Tue, 22 Mar 2011 21:38:57 +0000 (UTC) Received: from BAY147-W58 ([65.54.190.61]) by bay0-omc1-s28.bay0.hotmail.com with Microsoft SMTPSVC(6.0.3790.4675); Tue, 22 Mar 2011 14:38:58 -0700 Message-ID: X-Originating-IP: [209.6.82.6] From: Roger Hammerstein To: , Date: Tue, 22 Mar 2011 17:38:57 -0400 Importance: Normal In-Reply-To: <20110322211326.GC1882@garage.freebsd.pl> References: <20110307192239.GA31314@alchemy.franken.de> <20110310185423.GA50419@alchemy.franken.de> <20110319152838.GA8594@alchemy.franken.de> <20110321175632.GA19345@darkthrone.kvedulv.de> <20110321175933.GD2086@garage.freebsd.pl> <20110322191117.GH15528@alchemy.franken.de>, <20110322211326.GC1882@garage.freebsd.pl> MIME-Version: 1.0 X-OriginalArrivalTime: 22 Mar 2011 21:38:58.0011 (UTC) FILETIME=[8CED72B0:01CBE8D9] Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.5 Cc: freebsd-sparc64@freebsd.org, mm@freebsd.org Subject: RE: sparc64 hang with zfs v28 X-BeenThere: freebsd-sparc64@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to the Sparc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Mar 2011 21:38:58 -0000 > > Uhm=2C looks like r219089 changed some xcopy{in=2Cout}() into > > ddi_copy{in=2Cout}()=2C i.e. copy{in=2Cout}() into bcopy()=2C which > > is just wrong for copying in data in from/out to userspace. > > However=2C looking at the other uses of ddi_copy{in=2Cout}() it > > generally seems that ddi_copy{in=2Cout}() should be defined to > > copy{in=2Cout}(). With the attached patch at least my simple > > test cases works again. The one remaining xcopyout() in > > zfs_ioctl.c then could be also replaced with a ddi_copyout(). > > Not sure how any of this manages to work on x86 :) >=20 > Yeah=2C I found this as well and waiting for my test machine to be free t= o > test it. Thanks. This patch worked on my ultra 60. I rebuilt the kernel falcon# kldstat Id Refs Address Size Name 1 9 0xc0000000 b1f8c0 kernel 2 1 0xc10a2000 32e000 zfs.ko 3 1 0xc13d0000 104000 opensolaris.ko falcon# falcon# zpool status pool: tank state: ONLINE status: The pool is formatted using an older on-disk format. The pool can still be used=2C but some features are unavailable. action: Upgrade the pool using 'zpool upgrade'. Once this is done=2C the pool will no longer be accessible on older software versions. scan: none requested config: NAME STATE READ WRITE CKSUM tank ONLINE 0 0 0 mirror-0 ONLINE 0 0 0 da3 ONLINE 0 0 0 da6 ONLINE 0 0 0 errors: No known data errors falcon# falcon# I zfs mounted tank and can ls and copy files to it=2C delete files. Looks ok. =