Re: cvs commit: src/sys/kern kern_rwlock.c

2010-08-04 Thread Robert Watson
On Wed, 4 Aug 2010, Attilio Rao wrote: This bug seems also to affect the 6.x backport of rwlocks that Alfred did. Might either you or Alfred consider doing an MFC of this change?  Without it, recursive write locks on rwlocks are broken with INVARIANTS enabled. Thanks for signaling this, I'll

Re: cvs commit: src/sys/kern kern_rwlock.c

2010-08-04 Thread Attilio Rao
2010/8/4 Attilio Rao : > 2010/8/4 Robert Watson : >> On Wed, 14 Nov 2007, Attilio Rao wrote: >> >>> attilio     2007-11-14 21:21:48 UTC >>> >>>  FreeBSD src repository >>> >>>  Modified files: >>>   sys/kern             kern_rwlock.c >>>  Log: >>>  Remove a bogus KASSERT which will prevent rwlock t

Re: cvs commit: src/sys/kern kern_rwlock.c

2010-08-04 Thread Attilio Rao
2010/8/4 Robert Watson : > On Wed, 14 Nov 2007, Attilio Rao wrote: > >> attilio     2007-11-14 21:21:48 UTC >> >>  FreeBSD src repository >> >>  Modified files: >>   sys/kern             kern_rwlock.c >>  Log: >>  Remove a bogus KASSERT which will prevent rwlock to be acquired >>  recursively in ex

Re: cvs commit: src/sys/kern kern_rwlock.c

2010-08-03 Thread Robert Watson
On Wed, 14 Nov 2007, Attilio Rao wrote: attilio 2007-11-14 21:21:48 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Remove a bogus KASSERT which will prevent rwlock to be acquired recursively in exclusive mode with debugging kernels. Submitted b

cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-08-08 Thread Robert Watson
rwatson 2008-08-08 14:36:21 UTC FreeBSD src repository Modified files:(Branch: RELENG_7) sys/kern kern_rwlock.c sys/sys rwlock.h Log: SVN rev 181429 on 2008-08-08 14:36:21Z by rwatson Merge r177843,178147 from head to stable/7: Ad

cvs commit: src/sys/kern kern_rwlock.c

2008-04-04 Thread Jeff Roberson
jeff2008-04-04 10:00:46 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: - Add sysctls at debug.rwlock to control the behavior of the speculative spinning when readers hold a lock. This spinning is speculative because, unlike the

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-03 Thread Jeff Roberson
On Wed, 2 Apr 2008, Max Laier wrote: On Wednesday 02 April 2008 09:15:25 Jeff Roberson wrote: On Wed, 2 Apr 2008, Max Laier wrote: On Wednesday 02 April 2008 03:11:11 Jeff Roberson wrote: On Wed, 2 Apr 2008, Attilio Rao wrote: 2008/4/2, Max Laier <[EMAIL PROTECTED]>: On Wednesday 02 April 2

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-03 Thread Jeff Roberson
On Wed, 2 Apr 2008, David Schultz wrote: On Tue, Apr 01, 2008, Jeff Roberson wrote: On Wed, 2 Apr 2008, Max Laier wrote: On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: attilio 2008-04-01 20:31:55 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-02 Thread David Schultz
On Tue, Apr 01, 2008, Jeff Roberson wrote: > On Wed, 2 Apr 2008, Max Laier wrote: > > >On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: > >>attilio 2008-04-01 20:31:55 UTC > >> > >> FreeBSD src repository > >> > >> Modified files: > >>sys/kern kern_rwlock.c > >>sys/sy

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-02 Thread Max Laier
On Wednesday 02 April 2008 09:15:25 Jeff Roberson wrote: > On Wed, 2 Apr 2008, Max Laier wrote: > > On Wednesday 02 April 2008 03:11:11 Jeff Roberson wrote: > >> On Wed, 2 Apr 2008, Attilio Rao wrote: > >>> 2008/4/2, Max Laier <[EMAIL PROTECTED]>: > On Wednesday 02 April 2008 00:52:45 Jeff Rob

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-02 Thread Attilio Rao
2008/4/2, Jeff Roberson <[EMAIL PROTECTED]>: > > On Wed, 2 Apr 2008, Attilio Rao wrote: > > > > 2008/4/2, Max Laier <[EMAIL PROTECTED]>: > > > > > On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: > > > > On Wed, 2 Apr 2008, Max Laier wrote: > > > >> On Tuesday 01 April 2008 22:31:55 Attili

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-02 Thread Jeff Roberson
On Tue, 1 Apr 2008, Jeff Roberson wrote: On Wed, 2 Apr 2008, Max Laier wrote: On Wednesday 02 April 2008 03:11:11 Jeff Roberson wrote: On Wed, 2 Apr 2008, Attilio Rao wrote: 2008/4/2, Max Laier <[EMAIL PROTECTED]>: On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: On Wed, 2 Apr 2008

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-02 Thread Robert Watson
On Tue, 1 Apr 2008, Jeff Roberson wrote: I don't think this is a good thing either, but I also think that there are some cases where there just are different access orders. I'd rather want a clean way out of this than a lot of difficult per-instance hacks. This does not mean that these can'

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-02 Thread Jeff Roberson
On Wed, 2 Apr 2008, Max Laier wrote: On Wednesday 02 April 2008 03:11:11 Jeff Roberson wrote: On Wed, 2 Apr 2008, Attilio Rao wrote: 2008/4/2, Max Laier <[EMAIL PROTECTED]>: On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: On Wed, 2 Apr 2008, Max Laier wrote: On Tuesday 01 April 200

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Max Laier
On Wednesday 02 April 2008 03:11:11 Jeff Roberson wrote: > On Wed, 2 Apr 2008, Attilio Rao wrote: > > 2008/4/2, Max Laier <[EMAIL PROTECTED]>: > >> On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: > >> > On Wed, 2 Apr 2008, Max Laier wrote: > >> >> On Tuesday 01 April 2008 22:31:55 Attilio

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Jeff Roberson
On Wed, 2 Apr 2008, Attilio Rao wrote: 2008/4/2, Max Laier <[EMAIL PROTECTED]>: On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: > On Wed, 2 Apr 2008, Max Laier wrote: >> On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: >>> attilio 2008-04-01 20:31:55 UTC >>> >>> FreeBSD src

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Attilio Rao
2008/4/2, Max Laier <[EMAIL PROTECTED]>: > On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: > > On Wed, 2 Apr 2008, Max Laier wrote: > > > On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: > > >> attilio 2008-04-01 20:31:55 UTC > > >> > > >> FreeBSD src repository > > >> > >

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Max Laier
On Wednesday 02 April 2008 00:52:45 Jeff Roberson wrote: > On Wed, 2 Apr 2008, Max Laier wrote: > > On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: > >> attilio 2008-04-01 20:31:55 UTC > >> > >> FreeBSD src repository > >> > >> Modified files: > >> sys/kern kern_rwlock.

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Kris Kennaway
Max Laier wrote: On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: attilio 2008-04-01 20:31:55 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c sys/sys rwlock.h Log: Add rw_try_rlock() and rw_try_wlock() to rwlocks. These funct

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Jeff Roberson
On Wed, 2 Apr 2008, Max Laier wrote: On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: attilio 2008-04-01 20:31:55 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c sys/sys rwlock.h Log: Add rw_try_rlock() and rw_try_wlock() to r

Re: cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Max Laier
On Tuesday 01 April 2008 22:31:55 Attilio Rao wrote: > attilio 2008-04-01 20:31:55 UTC > > FreeBSD src repository > > Modified files: > sys/kern kern_rwlock.c > sys/sys rwlock.h > Log: > Add rw_try_rlock() and rw_try_wlock() to rwlocks. > These functio

cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2008-04-01 Thread Attilio Rao
attilio 2008-04-01 20:31:55 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c sys/sys rwlock.h Log: Add rw_try_rlock() and rw_try_wlock() to rwlocks. These functions try the specified operation (rlocking and wlocking) and true is

cvs commit: src/sys/kern kern_rwlock.c

2008-02-12 Thread John Baldwin
jhb 2008-02-12 19:16:53 UTC FreeBSD src repository Modified files:(Branch: RELENG_7) sys/kern kern_rwlock.c Log: MFC: Remove a conditional that is always true. Revision ChangesPath 1.28.4.3 +1 -1 src/sys/kern/kern_rwlock.c ___

cvs commit: src/sys/kern kern_rwlock.c

2008-02-06 Thread Jeff Roberson
jeff2008-02-07 06:16:54 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: - In rw_wunlock_hard prefer to wakeup writers if there are both readers and writers available. Doing otherwise can cause deadlocks as no read locks can proce

cvs commit: src/sys/kern kern_rwlock.c

2008-01-17 Thread John Baldwin
jhb 2008-01-17 20:15:15 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Remove a conditional that is always true. MFC after: 2 weeks Revision ChangesPath 1.34 +1 -1 src/sys/kern/kern_rwlock.c _

cvs commit: src/sys/kern kern_rwlock.c

2007-11-14 Thread Attilio Rao
attilio 2007-11-14 23:33:41 UTC FreeBSD src repository Modified files:(Branch: RELENG_7) sys/kern kern_rwlock.c Log: MFC: revision 1.30 Approved by: re Revision ChangesPath 1.28.4.1 +0 -3 src/sys/kern/kern_rwlock.c __

cvs commit: src/sys/kern kern_rwlock.c

2007-11-14 Thread Attilio Rao
attilio 2007-11-14 21:21:48 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Remove a bogus KASSERT which will prevent rwlock to be acquired recursively in exclusive mode with debugging kernels. Submitted by: kmacy Approved by: jeff

cvs commit: src/sys/kern kern_rwlock.c

2007-09-27 Thread John Baldwin
jhb 2007-09-27 18:06:49 UTC FreeBSD src repository Modified files:(Branch: RELENG_6) sys/kern kern_rwlock.c Log: A few tweaks to reduce diffs with HEAD including using lock_init() and lock_destroy() and adding a missing #include so that NO_ADAPTIVE_RWLOC

cvs commit: src/sys/kern kern_rwlock.c

2007-09-05 Thread Alfred Perlstein
alfred 2007-09-05 22:18:39 UTC FreeBSD src repository Modified files:(Branch: RELENG_6) sys/kern kern_rwlock.c Log: Remove/refactor calls to turnstile_release() as it turns out that turnstile_unpend_queue() drops the lock that turnstile_release() does. T

cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2007-07-20 Thread Attilio Rao
attilio 2007-07-20 08:43:42 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c sys/sys rwlock.h Log: Fix some problems with lock profiling in rw locks: - Adjust lock_profiling stubs semantic in the hard functions in order to be m

cvs commit: src/sys/kern kern_rwlock.c src/sys/sys _rwlock.h rwlock.h

2007-06-26 Thread Attilio Rao
attilio 2007-06-26 21:31:56 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c sys/sys _rwlock.h rwlock.h Log: Introduce a new rwlocks initialization function: rw_init_flags. This is very similar to sx_init_flags: it initializes the

cvs commit: src/sys/kern kern_rwlock.c kern_sx.c src/sys/sys rwlock.h sx.h

2007-05-08 Thread John Baldwin
jhb 2007-05-08 21:51:37 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c kern_sx.c sys/sys rwlock.h sx.h Log: Add destroyed cookie values for sx locks and rwlocks as well as extra KASSERTs so that any lock operations on a destr

cvs commit: src/sys/kern kern_rwlock.c

2007-03-30 Thread John Baldwin
jhb 2007-03-30 18:08:55 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: - Drop memory barriers in rw_try_upgrade(). We don't need an 'acq' memory barrier here as the earlier rw_rlock() already contained one. - Comment fix. Revisi

cvs commit: src/sys/kern kern_rwlock.c

2007-03-13 Thread John Baldwin
jhb 2007-03-13 16:51:27 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Print readers count as unsigned in ddb 'show lock'. Submitted by: attilio Revision ChangesPath 1.20 +2 -2 src/sys/kern/kern_rwlock.c ___

cvs commit: src/sys/kern kern_rwlock.c

2007-03-12 Thread John Baldwin
jhb 2007-03-12 20:10:29 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Fix a typo. Revision ChangesPath 1.19 +1 -1 src/sys/kern/kern_rwlock.c ___ cvs-all@freebsd.org mailing

cvs commit: src/sys/kern kern_rwlock.c

2007-03-07 Thread John Baldwin
jhb 2007-03-07 20:48:48 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Fix some nits in lock profiling for rwlocks: - Properly note when a read lock is released. - Always note when we contest on a read lock. - Only note success of obt

cvs commit: src/sys/kern kern_rwlock.c src/sys/sys rwlock.h

2006-04-19 Thread John Baldwin
jhb 2006-04-19 21:06:52 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c sys/sys rwlock.h Log: Implement rw_try_upgrade() and rw_downgrade(). rw_try_upgrade() makes a single attempt at upgrading a read lock to a write lock, an

cvs commit: src/sys/kern kern_rwlock.c

2006-04-18 Thread Wojciech A. Koszek
wkoszek 2006-04-18 20:32:42 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: 'owner' is not used without SMP. Fix kernel build for such kernel configurations. Approved by:jhb Revision ChangesPath 1.8 +4 -0 src/sy

cvs commit: src/sys/kern kern_rwlock.c

2006-04-18 Thread John Baldwin
jhb 2006-04-18 18:27:54 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Adaptively spin before blocking on the turnstile if an rwlock is write locked. In general the adaptive spinning is similar to the same code for mutexes with some ex

cvs commit: src/sys/kern kern_rwlock.c

2006-04-17 Thread John Baldwin
jhb 2006-04-17 21:11:01 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: - Add a rw_wowner() macro that just returns the owner of a write lock and use it in places that only care about the write owner instead of rw_owner() as a baby s

Re: cvs commit: src/sys/kern kern_rwlock.c

2006-02-04 Thread Doug Barton
Scott Long wrote: > scottl 2006-02-01 04:18:08 UTC > > FreeBSD src repository > > Modified files: > sys/kern kern_rwlock.c > Log: > Fix another compile problem. If I find any more, this file is going in the > Attic until it is properly fixed. Without supper I pr

cvs commit: src/sys/kern kern_rwlock.c

2006-01-31 Thread Scott Long
scottl 2006-02-01 04:18:08 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Fix another compile problem. If I find any more, this file is going in the Attic until it is properly fixed. Revision ChangesPath 1.5 +1 -1 src

cvs commit: src/sys/kern kern_rwlock.c

2006-01-30 Thread Scott Long
scottl 2006-01-30 19:25:52 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Regroup order of operations to better reflect what was probably intended. Submitted by: Peter Jeremy Revision ChangesPath 1.4 +3 -3 src/sys/k

Re: cvs commit: src/sys/kern kern_rwlock.c

2006-01-30 Thread Scott Long
Peter Jeremy wrote: On Sun, 2006-Jan-29 20:48:25 +, Scott Long wrote: gcc can't figure out the order of operations at line 519, and neither can I, but this is my best guess. Also correct a number of typos and syntax errors. Revision ChangesPath 1.3 +4 -4 src/sys/kern/kern_

Re: cvs commit: src/sys/kern kern_rwlock.c

2006-01-30 Thread Peter Jeremy
On Sun, 2006-Jan-29 20:48:25 +, Scott Long wrote: > gcc can't > figure out the order of operations at line 519, and neither can I, but this > is my best guess. Also correct a number of typos and syntax errors. > > Revision ChangesPath > 1.3 +4 -4 src/sys/kern/kern_rwlock

cvs commit: src/sys/kern kern_rwlock.c

2006-01-29 Thread Scott Long
scottl 2006-01-29 20:48:25 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Take a stab at making this compile when WITNESS is not defined. gcc can't figure out the order of operations at line 519, and neither can I, but this is my best gue

cvs commit: src/sys/kern kern_rwlock.c

2006-01-28 Thread Max Laier
mlaier 2006-01-29 02:35:22 UTC FreeBSD src repository Modified files: sys/kern kern_rwlock.c Log: Unbreak on archs where %d doesn't print uintptr_t arithmetic. Revision ChangesPath 1.2 +2 -1 src/sys/kern/kern_rwlock.c __