From owner-cvs-all@FreeBSD.ORG Mon Mar 15 11:21:42 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 27BDB16A4CE; Mon, 15 Mar 2004 11:21:42 -0800 (PST) Received: from electra.cse.Buffalo.EDU (electra.cse.Buffalo.EDU [128.205.32.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id BCAB643D45; Mon, 15 Mar 2004 11:21:41 -0800 (PST) (envelope-from kensmith@cse.Buffalo.EDU) Received: from electra.cse.Buffalo.EDU (kensmith@localhost [127.0.0.1]) i2FJLd2Z029569; Mon, 15 Mar 2004 14:21:39 -0500 (EST) Received: (from kensmith@localhost) by electra.cse.Buffalo.EDU (8.12.10/8.12.9/Submit) id i2FJLdo3029568; Mon, 15 Mar 2004 14:21:39 -0500 (EST) Date: Mon, 15 Mar 2004 14:21:39 -0500 From: Ken Smith To: Wilko Bulte Message-ID: <20040315192139.GB28070@electra.cse.Buffalo.EDU> References: <200403151848.i2FImSFe047106@repoman.freebsd.org> <20040315190853.GA33607@freebie.xs4all.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040315190853.GA33607@freebie.xs4all.nl> User-Agent: Mutt/1.4.1i cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org cc: John Baldwin Subject: Re: cvs commit: src/sys/kern sys_process.c syscalls.master X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 15 Mar 2004 19:21:42 -0000 On Mon, Mar 15, 2004 at 08:08:53PM +0100, Wilko Bulte wrote: > On Mon, Mar 15, 2004 at 10:48:28AM -0800, John Baldwin wrote: > > jhb 2004/03/15 10:48:28 PST > > > > FreeBSD src repository > > > > Modified files: > > sys/kern sys_process.c syscalls.master > > Log: > > Drop the proc lock around calls to the MD functions ptrace_single_step(), > > ptrace_set_pc(), and cpu_ptrace() so that those functions are free to > > acquire Giant, sleep, etc. We already do a PHOLD/PRELE around them so > > that it is safe to sleep inside of these routines if necessary. This > > allows ptrace() to be marked MP safe again as it no longer triggers lock > > order reversals on Alpha. > > Someone with a SMP alpha box might also want to give this a spin I suppose. > beast (alpha ref machine in the cluster) has dual processors, if something special needs to be done to test this we could arrange something. -- Ken Smith - From there to here, from here to | kensmith@cse.buffalo.edu there, funny things are everywhere. | - Theodore Geisel |