From a7a8bac81f171b93c4770f51c694b15c84a2f9f8 Mon Sep 17 00:00:00 2001 From: jeff Date: Fri, 1 Jun 2007 01:12:45 +0000 Subject: - Move rusage from being per-process in struct pstats to per-thread in td_ru. This removes the requirement for per-process synchronization in statclock() and mi_switch(). This was previously supported by sched_lock which is going away. All modifications to rusage are now done in the context of the owning thread. reads proceed without locks. - Aggregate exiting threads rusage in thread_exit() such that the exiting thread's rusage is not lost. - Provide a new routine, rufetch() to fetch an aggregate of all rusage structures from all threads in a process. This routine must be used in any place requiring a rusage from a process prior to it's exit. The exited process's rusage is still available via p_ru. - Aggregate tick statistics only on demand via rufetch() or when a thread exits. Tick statistics are kept in the thread and protected by sched_lock until it exits. Initial patch by: attilio Reviewed by: attilio, bde (some objections), arch (mostly silent) --- sys/vm/vm_fault.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'sys/vm/vm_fault.c') diff --git a/sys/vm/vm_fault.c b/sys/vm/vm_fault.c index 5855430..614956a 100644 --- a/sys/vm/vm_fault.c +++ b/sys/vm/vm_fault.c @@ -918,15 +918,10 @@ readrest: * Unlock everything, and return */ unlock_and_deallocate(&fs); - PROC_LOCK(curproc); - if ((curproc->p_sflag & PS_INMEM) && curproc->p_stats) { - if (hardfault) { - curproc->p_stats->p_ru.ru_majflt++; - } else { - curproc->p_stats->p_ru.ru_minflt++; - } - } - PROC_UNLOCK(curproc); + if (hardfault) + curthread->td_ru.ru_majflt++; + else + curthread->td_ru.ru_minflt++; return (KERN_SUCCESS); } -- cgit v1.1