The patch titled Fix the timespec_sub() interface has been added to the -mm tree. Its filename is per-task-delay-accounting-setup-fix-1.patch See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this From: Balbir Singh <balbir@xxxxxxxxxx> 1. Change the interface of timespec_sub() to return by value and rename the arguments. Use lhs,rhs instead of end,start. 2. Pass the arguments by value Signed-off-by: Balbir Singh <balbir@xxxxxxxxxx> Cc: Shailabh Nagar <nagar@xxxxxxxxxxxxxx> Cc: Jes Sorensen <jes@xxxxxxx> Cc: Peter Chubb <peterc@xxxxxxxxxxxxxxxxxx> Cc: Erich Focht <efocht@xxxxxxxxxx> Cc: Levent Serinol <lserinol@xxxxxxxxx> Cc: Jay Lan <jlan@xxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxx> --- include/linux/time.h | 12 +++++++----- kernel/delayacct.c | 2 +- 2 files changed, 8 insertions(+), 6 deletions(-) diff -puN include/linux/time.h~per-task-delay-accounting-setup-fix-1 include/linux/time.h --- devel/include/linux/time.h~per-task-delay-accounting-setup-fix-1 2006-05-11 09:23:43.000000000 -0700 +++ devel-akpm/include/linux/time.h 2006-05-11 09:23:43.000000000 -0700 @@ -68,13 +68,15 @@ extern unsigned long mktime(const unsign extern void set_normalized_timespec(struct timespec *ts, time_t sec, long nsec); /* - * sub = end - start, in normalized form + * sub = lhs - rhs, in normalized form */ -static inline void timespec_sub(struct timespec *start, struct timespec *end, - struct timespec *sub) +static inline struct timespec timespec_sub(struct timespec lhs, + struct timespec rhs) { - set_normalized_timespec(sub, end->tv_sec - start->tv_sec, - end->tv_nsec - start->tv_nsec); + struct timespec ts_delta; + set_normalized_timespec(&ts_delta, lhs.tv_sec - rhs.tv_sec, + lhs.tv_nsec - rhs.tv_nsec); + return ts_delta; } /* diff -puN kernel/delayacct.c~per-task-delay-accounting-setup-fix-1 kernel/delayacct.c --- devel/kernel/delayacct.c~per-task-delay-accounting-setup-fix-1 2006-05-11 09:23:43.000000000 -0700 +++ devel-akpm/kernel/delayacct.c 2006-05-11 09:23:43.000000000 -0700 @@ -74,7 +74,7 @@ static inline void delayacct_end(struct s64 ns; do_posix_clock_monotonic_gettime(end); - timespec_sub(&ts, start, end); + ts = timespec_sub(*end, *start); ns = timespec_to_ns(&ts); if (ns < 0) return; _ Patches currently in -mm which might be from balbir@xxxxxxxxxx are fix-dcache-race-during-umount.patch fix-dcache-race-during-umount-fix.patch prune_one_dentry-tweaks.patch per-task-delay-accounting-setup.patch per-task-delay-accounting-setup-fix-1.patch per-task-delay-accounting-setup-fix-2.patch per-task-delay-accounting-sync-block-i-o-and-swapin-delay-collection.patch per-task-delay-accounting-sync-block-i-o-and-swapin-delay-collection-fix-1.patch per-task-delay-accounting-cpu-delay-collection-via-schedstats.patch per-task-delay-accounting-cpu-delay-collection-via-schedstats-fix-1.patch per-task-delay-accounting-utilities-for-genetlink-usage.patch per-task-delay-accounting-taskstats-interface.patch per-task-delay-accounting-taskstats-interface-fix-1.patch per-task-delay-accounting-delay-accounting-usage-of-taskstats-interface.patch per-task-delay-accounting-documentation.patch per-task-delay-accounting-proc-export-of-aggregated-block-i-o-delays.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