Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 09 Aug 2012 08:22:53 -0400
From:      Mike Tancsa <mike@sentex.net>
To:        John Baldwin <jhb@freebsd.org>
Cc:        Garrett Cooper <yanegomi@gmail.com>, current@freebsd.org
Subject:   Re: [PATCH] Add locking to twe(4) so it no longer uses Giant
Message-ID:  <5023AB9D.5070608@sentex.net>
In-Reply-To: <5022B252.30606@sentex.net>
References:  <201208031418.57941.jhb@freebsd.org> <201208031726.03652.jhb@freebsd.org> <502121F5.4020705@sentex.net> <201208080727.45595.jhb@freebsd.org> <5022B252.30606@sentex.net>

next in thread | previous in thread | raw e-mail | index | archive | help
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/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5023AB9D.5070608>