From owner-cvs-src@FreeBSD.ORG Mon Jul 7 01:25:17 2003 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 4938137B401; Mon, 7 Jul 2003 01:25:17 -0700 (PDT) Received: from HAL9000.homeunix.com (ip114.bella-vista.sfo.interquest.net [66.199.86.114]) by mx1.FreeBSD.org (Postfix) with ESMTP id 81CC643FA3; Mon, 7 Jul 2003 01:25:15 -0700 (PDT) (envelope-from das@FreeBSD.org) Received: from HAL9000.homeunix.com (localhost [127.0.0.1]) by HAL9000.homeunix.com (8.12.9/8.12.9) with ESMTP id h678P9K5090689; Mon, 7 Jul 2003 01:25:09 -0700 (PDT) (envelope-from das@FreeBSD.org) Received: (from das@localhost) by HAL9000.homeunix.com (8.12.9/8.12.9/Submit) id h678P6ih090688; Mon, 7 Jul 2003 01:25:06 -0700 (PDT) (envelope-from das@FreeBSD.org) Date: Mon, 7 Jul 2003 01:25:06 -0700 From: David Schultz To: David Xu Message-ID: <20030707082506.GA90638@HAL9000.homeunix.com> Mail-Followup-To: David Xu , src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org References: <200307070428.h674SNfq092785@repoman.freebsd.org> <20030707060318.GA28281@HAL9000.homeunix.com> <001c01c3444f$7c904280$f001a8c0@davidw2k> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <001c01c3444f$7c904280$f001a8c0@davidw2k> cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libpthread/thread thr_attr_get_np.c thr_cancel.c thr_getschedparam.c thr_join.c thr_mutex_prioceiling.c thr_sigaction.c thr_sigmask.c thr_sigpending.c thr_sigsuspend.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 07 Jul 2003 08:25:17 -0000 On Mon, Jul 07, 2003, David Xu wrote: > > > davidxu 2003/07/06 21:28:23 PDT > > > > > > FreeBSD src repository > > > > > > Modified files: > > > lib/libpthread/thread thr_attr_get_np.c thr_cancel.c > > > thr_getschedparam.c thr_join.c > > > thr_mutex_prioceiling.c thr_sigaction.c > > > thr_sigmask.c thr_sigpending.c > > > thr_sigsuspend.c > > > Log: > > > Avoid accessing user provided parameters in critical region. > > > > Cool. What happens if a page fault is taken in a critical region? > > Does this merely make the KSE unusable by other threads until the > > page is faulted in, or does it deadlock the UTS? (If the latter > > is true, you'd need to wire a stack page or two to ensure > > correctness, or do soemthing differently.) > > > > The change is nothing to do with page fault, also page fault > shouldn't cause deadlock. It is true the kse can not be used > by other threads if page fault occurs while in critical region. > For normal page fault not in critical region, kernel KSE codes > still does not call thread_user_enter(), but I want to insert > such call in trap.c, so when a page fault occurs, and thread > is blocked in paging, other threads still can run in userland, > if a user thread is in kernel mode, it is not affected. That's what I was talking about. If you enter the UTS upon taking a page fault, you have to be sure that the page that faulted didn't belong to the UTS itself, or you at least have to have some way of breaking the loop. But since the UTS is unaware of page faults presently, I guess this isn't a problem yet.