diff options
| author | attilio <attilio@FreeBSD.org> | 2007-06-09 21:48:44 +0000 |
|---|---|---|
| committer | attilio <attilio@FreeBSD.org> | 2007-06-09 21:48:44 +0000 |
| commit | 12d804e413f2c44d9e10f25ee1b546f2b48c598b (patch) | |
| tree | 7fca182049d1ed6ecc1c3b117e1d77c681c922c3 /sys/kern/kern_acct.c | |
| parent | 82a09af23776643f85a5e710a09bef3be83823d9 (diff) | |
| download | FreeBSD-src-12d804e413f2c44d9e10f25ee1b546f2b48c598b.zip FreeBSD-src-12d804e413f2c44d9e10f25ee1b546f2b48c598b.tar.gz | |
rufetch and calcru sometimes should be called atomically together.
This patch fixes places where they should be called atomically changing
their locking requirements (both assume per-proc spinlock held) and
introducing rufetchcalc which wrappers both calls to be performed in
atomic way.
Reviewed by: jeff
Approved by: jeff (mentor)
Diffstat (limited to 'sys/kern/kern_acct.c')
| -rw-r--r-- | sys/kern/kern_acct.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/kern/kern_acct.c b/sys/kern/kern_acct.c index e7409b3..269b4e4 100644 --- a/sys/kern/kern_acct.c +++ b/sys/kern/kern_acct.c @@ -370,8 +370,7 @@ acct_process(struct thread *td) bcopy(p->p_comm, acct.ac_comm, sizeof acct.ac_comm); /* (2) The amount of user and system time that was used */ - rufetch(p, &ru); - calcru(p, &ut, &st); + rufetchcalc(p, &ru, &ut, &st); acct.ac_utime = encode_timeval(ut); acct.ac_stime = encode_timeval(st); |
