From owner-cvs-all@FreeBSD.ORG Fri Feb 6 14:13:14 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 9400116A4CE; Fri, 6 Feb 2004 14:13:14 -0800 (PST) Received: from smtp01.syd.iprimus.net.au (smtp01.syd.iprimus.net.au [210.50.30.52]) by mx1.FreeBSD.org (Postfix) with ESMTP id B1BD143D53; Fri, 6 Feb 2004 14:13:09 -0800 (PST) (envelope-from tim@robbins.dropbear.id.au) Received: from robbins.dropbear.id.au (210.50.112.210) by smtp01.syd.iprimus.net.au (7.0.024) id 400C4DF4007565D7; Sat, 7 Feb 2004 09:14:35 +1100 Received: by robbins.dropbear.id.au (Postfix, from userid 1000) id 3952F41AD; Sat, 7 Feb 2004 09:12:55 +1100 (EST) Date: Sat, 7 Feb 2004 09:12:55 +1100 From: Tim Robbins To: John Baldwin Message-ID: <20040206221255.GA54490@cat.robbins.dropbear.id.au> References: <200402061930.i16JUCpa011145@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200402061930.i16JUCpa011145@repoman.freebsd.org> User-Agent: Mutt/1.4.1i cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_resource.c 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: Fri, 06 Feb 2004 22:13:14 -0000 On Fri, Feb 06, 2004 at 11:30:12AM -0800, John Baldwin wrote: > jhb 2004/02/06 11:30:12 PST > > FreeBSD src repository > > Modified files: > sys/kern kern_resource.c > Log: > - Correct the translation of old rlimit values to properly handle the old > RLIM_INFINITY case for ogetrlimit(). > - Use %jd and intmax_t to output negative time in usec in calcru(). > - Rework getrusage() to make a copy of the rusage struct into a local > variable while holding Giant and then do the copyout from the local > variable to avoid having to have the original process rusage struct > locked while doing the copyout (which would not be safe). This also > includes a few style fixes from Bruce to getrusage(). Thanks (from the one who added the XXX comment). Can't we use the proc lock here though? Tim