First of all, let me repeat, I am not going to discuss whether we need these changes or not, I do not know even if I understand your motivation. My only concern is correctness, but On 11/11, Michael Holzheu wrote: > > * Add cumulative dead thread CPU times to taskstats > * Introduce parallel accounting process hierarchy (based on discussions > with Oleg Nesterov and Roland McGrath) Michael, I really think this patch does too many different things. I forgot the details of our previous discussion, and I got lost trying to understand the new version. I already asked you to split these changes, perhaps you can do this? Say, bacct_add_tsk() looks overcomplicated, the change in copy_process() shouldn't introduce the new CLONE_THREAD check, not sure I understand why release_task() was chosen for reparenting, other nits... But it is not easy to discuss these completely different things looking at the single patch. Imho, it would be much better if you make a separate patch which adds acct_parent/etc and implements the parallel hierarchy. This also makes sense because it touches the core kernel. Personally I think that even "struct cdata" + __account_ctime() helper needs a separate patch, and perhaps this change makes sense by itself as cleanup. And this way the "trivial" changes (like the changes in k_getrusage) won't distract from the functional changes. The final change should introduce cdata_acct and actually implement the complete cumulative accounting. At least, please distill parallel accounting process hierarchy. We do not want bugs in this area ;) What do you think? Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
