Subject: Re: [PATCH] Performance Stats: Kernel patch

From: Dave Jones <[email-masked]>
Date: Tue, 22 May 2007 14:48:59 -0400

On Tue, May 22, 2007 at 05:19:52PM +0000, Maxim Uvarov wrote:

 > diff --git a/include/asm-i386/thread_info.h b/include/asm-i386/thread_info.h
 > index 4b187bb..bccfd6a 100644
 > --- a/include/asm-i386/thread_info.h
 > +++ b/include/asm-i386/thread_info.h
 > @@ -33,6 +33,7 @@ struct thread_info {
 >  	int			preempt_count;	/* 0 => preemptable, <0 => BUG */
 >  
 >  
 > +	unsigned long		syscall_count;	/* Syscall counter */
 >  	mm_segment_t		addr_limit;	/* thread address space:
 >  					 	   0-0xBFFFFFFF for user-thead
 >  						   0-0xFFFFFFFF for kernel-thread

It seems a bit unkind to bloat up the thread_info for every process
of every user when the common case will be people that don't care about
this feature at all.

Especially when the same information could be got from ptrace.

	Dave

-- 
http://www.codemonkey.org.uk