From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 25 Nov 2010 17:59:39 +0530 From: Balbir Singh Subject: Re: [patch 1/4] taskstats: Introduce "struct cdata" Message-ID: <20101125122938.GC3298@balbir.in.ibm.com> Reply-To: balbir@linux.vnet.ibm.com References: <20101119201108.269346583@linux.vnet.ibm.com> <20101119201143.787050299@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <20101119201143.787050299@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: Michael Holzheu Cc: Oleg Nesterov , Shailabh Nagar , Andrew Morton , Peter Zijlstra , John stultz , Thomas Gleixner , Martin Schwidefsky , Heiko Carstens , Roland McGrath , linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org List-ID: * Michael Holzheu [2010-11-19 21:11:09]: > From: Michael Holzheu > > This patch introduces a new structure "struct cdata" that is used to > store cumulative resource counters for dead child processes and threads. > > Note that there is one asymmetry: > For "struct task_io_accounting" (ioc) there is no extra accounting field for > dead threads. One field is used for both, dead processes and threads. > > This patch introduces no functional change. > > Signed-off-by: Michael Holzheu > --- > fs/binfmt_elf.c | 4 +- > fs/exec.c | 2 - > fs/proc/array.c | 16 ++++---- > include/linux/sched.h | 22 +++++++---- > kernel/exit.c | 86 ++++++++++++++++++++++++---------------------- > kernel/posix-cpu-timers.c | 12 +++--- > kernel/sys.c | 44 ++++++++++++----------- > 7 files changed, 100 insertions(+), 86 deletions(-) Looks good to me Acked-by: Balbir Singh -- Three Cheers, Balbir