Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Aug 2012 15:39:47 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Kevin Oberman <kob6558@gmail.com>
Cc:        current@freebsd.org
Subject:   Re: [PATCH] Add locking to twe(4) so it no longer uses Giant
Message-ID:  <201208031539.47735.jhb@freebsd.org>
In-Reply-To: <CAN6yY1t824WcE3J_9%2BG_VK7QNWr=yuxLdBLMyB_wFZB-OaqrTA@mail.gmail.com>
References:  <201208031418.57941.jhb@freebsd.org> <CAN6yY1t824WcE3J_9%2BG_VK7QNWr=yuxLdBLMyB_wFZB-OaqrTA@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, August 03, 2012 2:56:00 pm Kevin Oberman wrote:
> I'll try to find time to try it later today, but I'm in the middle of
> budget wrangling and I can't make any promises.
> 
> Before I try, will these patches apply to the twe driver in
> 9.1-PRERELEASE? My system with twe was updated to RELENG_9 on July 18.
> I really don't have time to install current right now.

I believe the patches should apply fine on 8 and 9.  If you use it on 8 or 9
please make sure to enable INVARIANTS and INVARIANT_SUPPORT at least for
initial testing.  Thanks!

> -- 
> R. Kevin Oberman, Network Engineer
> E-mail: kob6558@gmail.com
> 
> On Fri, Aug 3, 2012 at 11:18 AM, John Baldwin <jhb@freebsd.org> wrote:
> > We only have a few storage drivers left that use Giant and twe(4) is one of
> > them.  I don't have any hardware to test with, however.  I have verified the
> > patch compiles, but I'd appreciate it if some folks with twe(4) hardware could
> > test it.
> >
> > The patch is at http://www.FreeBSD.org/~jhb/patches/twe_locking.patch
> 

-- 
John Baldwin



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