Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 19 Oct 2012 15:32:59 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Andre Oppermann <andre@freebsd.org>
Cc:        Davide Italiano <davide@freebsd.org>, mlaier@freebsd.org, svn-src-projects@freebsd.org, attilio@freebsd.org, Konstantin Belousov <kostikbel@gmail.com>, src-committers@freebsd.org, Stephan Uphoff <ups@freebsd.org>
Subject:   Re: svn commit: r238907 - projects/calloutng/sys/kern
Message-ID:  <201210191532.59244.jhb@freebsd.org>
In-Reply-To: <508119DE.6020106@freebsd.org>
References:  <201207301350.q6UDobCI099069@svn.freebsd.org> <201209171113.06861.jhb@freebsd.org> <508119DE.6020106@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, October 19, 2012 5:14:06 am Andre Oppermann wrote:
> On 17.09.2012 17:13, John Baldwin wrote:
> > On Monday, September 17, 2012 3:25:44 am Andre Oppermann wrote:
> >> Hello Attilio,
> >>
> >> could you integrate and test this patch from Isilon as well:
> >>
> >>     Add INVARIANT and WITNESS support to rm_lock locks and optimize the
> >>     synchronization path by replacing a LIST of active readers with a
> >>     TAILQ.
> >>
> >>     Obtained from:	Isilon
> >>     Submitted by:	mlaier
> >>
> >>     http://svn.freebsd.org/changeset/base/234648
> >>
> >> You're far deeper into locking than I am.
> >
> > Some of this is already in the tree, and their WITNESS changes don't quite
> > work correctly.  I have several fixes (including some of these already) that I
> > am just waiting to develop a good test for.  Having an _rm_assert() to pull
> > from will be good though.
> 
> Any news/progress on this?

I need to get some time to test them.  www.freebsd.org/~jhb/patches/rmlock.patch is
the current set of patches I have.

-- 
John Baldwin



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