Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 7 Feb 2004 09:12:55 +1100
From:      Tim Robbins <tjr@freebsd.org>
To:        John Baldwin <jhb@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/kern kern_resource.c
Message-ID:  <20040206221255.GA54490@cat.robbins.dropbear.id.au>
In-Reply-To: <200402061930.i16JUCpa011145@repoman.freebsd.org>
References:  <200402061930.i16JUCpa011145@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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



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