From owner-freebsd-current@FreeBSD.ORG Mon Jan 23 23:19:06 2006 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 171D216A420; Mon, 23 Jan 2006 23:19:06 +0000 (GMT) (envelope-from julian@elischer.org) Received: from a50.ironport.com (a50.ironport.com [63.251.108.112]) by mx1.FreeBSD.org (Postfix) with ESMTP id C53FC43D48; Mon, 23 Jan 2006 23:19:05 +0000 (GMT) (envelope-from julian@elischer.org) Received: from unknown (HELO [10.251.23.146]) ([10.251.23.146]) by a50.ironport.com with ESMTP; 23 Jan 2006 15:19:04 -0800 X-IronPort-Anti-Spam-Filtered: true Message-ID: <43D56468.1060101@elischer.org> Date: Mon, 23 Jan 2006 15:19:04 -0800 From: Julian Elischer User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.11) Gecko/20050727 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Robert Watson References: <43D05151.5070409@elischer.org> <200601231616.49140.jhb@freebsd.org> <43D55739.80608@elischer.org> <20060123224756.R48094@fledge.watson.org> In-Reply-To: <20060123224756.R48094@fledge.watson.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: freebsd-current@freebsd.org Subject: Re: kernel thread as real threads.. X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Jan 2006 23:19:06 -0000 Robert Watson wrote: > > On Mon, 23 Jan 2006, Julian Elischer wrote: > >> John Baldwin wrote: >> >>> On Thursday 19 January 2006 21:56, Julian Elischer wrote: >>> >>>> some progrsss.. >>>> as the first few lines show, it's not quite perfect yet but it's >>>> most of >>>> the way there.. >>>> (Like proc 1 isn't init) >>> >>> >>> One other note, watch out for the AIO daemons. They have to be >>> kernel procs and not kthreads because they borrow the vmspace of the >>> user process when performing AIO on another process' behalf. >>> >> yeah I found that and the patches account for that. >> >> However I would like to suggest that we change the way that aio works.. >> >> My suggestion is that when a process does AIO, that we "fork a >> ksegroup" and attach it to the process, and assign it a (or some) >> worker thread to do the aio work. The userland process would be >> oblivious of the extra (kernel) threads in that kseg and they would >> be independently schedulable. They would however automatically have >> full access to the correct address space. > > > While I think that, in principle, this is the right thing to do, I'm a > bit worried about doing it in practice. One of the things I like > about the current aio code is the degree to which the the aio daemon > processes are independent of the original requesting process -- we > acquire references to vmspaces, creds, file descriptors, etc, but > don't keep accessing the ones of the process. One might consider this a bug :-) of course the creds would be held until the operation finishes, but there is a linit as to how separate an aio request from a process should be from that process. > This means that if a process changes its uid, changes its threading, > etc, while aio is running, aio is relatively unaffected. I worry that > if we allow tighter integration of the two, we open up the door to > security related race conditions. Also, we introduce concerns about > the run-down when single-threading, exiting, execing, etc. it would be no different from when a threaded process exits now.. With the current setup you have a whole set of worries about what to do about the thread responds when the process has already died. These worries are aqctually simplified if the thread is part of the process. (at least I believe this to be the case). > > Robert N M Watson > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to > "freebsd-current-unsubscribe@freebsd.org"