Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 14 Jul 2004 12:10:02 -0700
From:      Alfred Perlstein <alfred@freebsd.org>
To:        Robert Watson <rwatson@freebsd.org>
Cc:        cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/kern kern_event.c src/sys/sys eventvar.h
Message-ID:  <20040714191002.GE95729@elvis.mu.org>
In-Reply-To: <Pine.NEB.3.96L.1040714145151.56002C-100000@fledge.watson.org>
References:  <20040714184445.GC95729@elvis.mu.org> <Pine.NEB.3.96L.1040714145151.56002C-100000@fledge.watson.org>

next in thread | previous in thread | raw e-mail | index | archive | help
* Robert Watson <rwatson@freebsd.org> [040714 11:56] wrote:
> On Wed, 14 Jul 2004, Alfred Perlstein wrote:
> 
> > Do we have this on the plate?  Or are you stalling my work based simply
> > on wishful thinking? :) 
> 
> There are two outstanding patch sets that add locking to kqueue:
> 
> - Brian Feldman's "kqgiant" changes, which I'm in the process of updating
>   in Perforce and adapting for your recent additions of fs kqueue events.
> 
> - John-Mark Gurney's "data locking kqueue" changes, which he's currently
>   working on extracting from his employer.
> 
> In order for network stack locking without Giant to be real for 5.3,
> kqueue locking must be completed.  The pgsigio do represent a serious
> obstacle to getting that locking done, both by virtue of adding an
> immediate coding obstacle to adapt around, but also by introducing an
> architectural construct that will be difficult to deal with cleanly.  My
> goal is for us to have basic locking for kqueues in the sys tree in CVS
> within two weeks, using one or another of the two sets of changes above.

Ok, I'll back it out.  In the future I would appreciate a simple "we
actually have code in progress to do this" rather than what I thought
was wishful thinking.

I know your brain is full right now, but is there any way you see
this actually working?  I know how to avoid the recursion, but, buh..
I think sigio needs an overhaul.  I'll see if that works, for now
I'll just #if 0 the ioctl code to enable it.

-- 
- Alfred Perlstein
- Research Engineering Development Inc.
- email: bright@mu.org cell: 408-480-4684



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