The patch titled
make taskstats sending completely independent of delay accounting on/off status
has been added to the -mm tree. Its filename is
make-taskstats-sending-completely-independent-of-delay.patch
See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this
------------------------------------------------------
Subject: make taskstats sending completely independent of delay accounting on/off status
From: Shailabh Nagar <nagar@xxxxxxxxxxxxxx>
Complete the separation of delay accounting and taskstats by ignoring the
return value of delay accounting functions that fill in parts of taskstats
before it is sent out (either in response to a command or as part of a task
exit).
Also make delayacct_add_tsk return silently when delay accounting is turned
off rather than treat it as an error.
Signed-off-by: Shailabh Nagar <nagar@xxxxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
include/linux/delayacct.h | 4 +---
kernel/taskstats.c | 8 +++-----
2 files changed, 4 insertions(+), 8 deletions(-)
diff -puN include/linux/delayacct.h~make-taskstats-sending-completely-independent-of-delay include/linux/delayacct.h
--- a/include/linux/delayacct.h~make-taskstats-sending-completely-independent-of-delay
+++ a/include/linux/delayacct.h
@@ -80,9 +80,7 @@ static inline void delayacct_blkio_end(v
static inline int delayacct_add_tsk(struct taskstats *d,
struct task_struct *tsk)
{
- if (likely(!delayacct_on))
- return -EINVAL;
- if (!tsk->delays)
+ if (likely(!delayacct_on) || !tsk->delays)
return 0;
return __delayacct_add_tsk(d, tsk);
}
diff -puN kernel/taskstats.c~make-taskstats-sending-completely-independent-of-delay kernel/taskstats.c
--- a/kernel/taskstats.c~make-taskstats-sending-completely-independent-of-delay
+++ a/kernel/taskstats.c
@@ -177,7 +177,7 @@ static int send_cpu_listeners(struct sk_
static int fill_pid(pid_t pid, struct task_struct *pidtsk,
struct taskstats *stats)
{
- int rc;
+ int rc = 0;
struct task_struct *tsk = pidtsk;
if (!pidtsk) {
@@ -196,12 +196,10 @@ static int fill_pid(pid_t pid, struct ta
* Each accounting subsystem adds calls to its functions to
* fill in relevant parts of struct taskstsats as follows
*
- * rc = per-task-foo(stats, tsk);
- * if (rc)
- * goto err;
+ * per-task-foo(stats, tsk);
*/
- rc = delayacct_add_tsk(stats, tsk);
+ delayacct_add_tsk(stats, tsk);
stats->version = TASKSTATS_VERSION;
/* Define err: label here if needed */
_
Patches currently in -mm which might be from nagar@xxxxxxxxxxxxxx are
make-taskstats-sending-completely-independent-of-delay.patch
taskstats-free-skb-avoid-returns-in.patch
delay-accounting-temporarily-enable-by-default.patch
netlink-improve-string-attribute-validation.patch
task-watchers-register-per-task-delay-accounting.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
