From owner-freebsd-current@FreeBSD.ORG Thu Aug 9 12:23:04 2012 Return-Path: Delivered-To: current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id A3A121065670; Thu, 9 Aug 2012 12:23:04 +0000 (UTC) (envelope-from mike@sentex.net) Received: from smarthost1.sentex.ca (smarthost1-6.sentex.ca [IPv6:2607:f3e0:0:1::12]) by mx1.freebsd.org (Postfix) with ESMTP id 5C5E48FC0C; Thu, 9 Aug 2012 12:23:04 +0000 (UTC) Received: from [192.168.43.26] (pyroxene.sentex.ca [199.212.134.18]) by smarthost1.sentex.ca (8.14.5/8.14.5) with ESMTP id q79CN316032755; Thu, 9 Aug 2012 08:23:03 -0400 (EDT) (envelope-from mike@sentex.net) Message-ID: <5023AB9D.5070608@sentex.net> Date: Thu, 09 Aug 2012 08:22:53 -0400 From: Mike Tancsa Organization: Sentex Communications User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0) Gecko/20120428 Thunderbird/12.0.1 MIME-Version: 1.0 To: John Baldwin References: <201208031418.57941.jhb@freebsd.org> <201208031726.03652.jhb@freebsd.org> <502121F5.4020705@sentex.net> <201208080727.45595.jhb@freebsd.org> <5022B252.30606@sentex.net> In-Reply-To: <5022B252.30606@sentex.net> X-Enigmail-Version: 1.4.2 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.72 on 64.7.153.18 Cc: Garrett Cooper , current@freebsd.org Subject: Re: [PATCH] Add locking to twe(4) so it no longer uses Giant X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 09 Aug 2012 12:23:04 -0000 On 8/8/2012 2:39 PM, Mike Tancsa wrote: > On 8/8/2012 7:27 AM, John Baldwin wrote: >>> Looks like it breaks 3dm2 and the tw_cli. With the patch, I am not able >>> to see the 8006 controller I added. >> >> Ugh, ok. A few questions: >> >> 1) Does the driver see any attached drives/volumes? > > Yes > >> >> 2) If it does, does basic I/O to the drives work? > > yes > >> >> 3) Can you add some debugging printfs to twe_ioctl() to see what, if anything, >> fails in that routine when tw_cli makes a request? > > Yes, for sure. Let me know what you would like me to add. One more data point, /dev/twe0 does not exist with the patch and I think thats why the utils fail outright. ---Mike -- ------------------- Mike Tancsa, tel +1 519 651 3400 Sentex Communications, mike@sentex.net Providing Internet services since 1994 www.sentex.net Cambridge, Ontario Canada http://www.tancsa.com/