Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 17 Sep 2012 11:13:06 -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:  <201209171113.06861.jhb@freebsd.org>
In-Reply-To: <5056D078.3020904@freebsd.org>
References:  <201207301350.q6UDobCI099069@svn.freebsd.org> <CAJ-FndA8Yende_=-hgOMjfUkQVhaSdSjAb0W8xthqN1ThwT=Vg@mail.gmail.com> <5056D078.3020904@freebsd.org>

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

-- 
John Baldwin



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