[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4846D52E.8060101@ct.jp.nec.com>
Date: Wed, 04 Jun 2008 10:47:26 -0700
From: Hiroshi Shimamoto <h-shimamoto@...jp.nec.com>
To: KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>
Cc: Keika Kobayashi <kobayashi.kk@...s.nec.co.jp>,
linux-kernel@...r.kernel.org, akpm@...ux-foundation.org
Subject: Re: [PATCH 4/4] per-task-delay-accounting: /proc export for memory
reclaim delay
KOSAKI Motohiro wrote:
> Hi
>
>> diff --git a/fs/proc/array.c b/fs/proc/array.c
>> index 9e3b8c3..a3e6e86 100644
>> --- a/fs/proc/array.c
>> +++ b/fs/proc/array.c
>> @@ -498,7 +498,7 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns,
>>
>> seq_printf(m, "%d (%s) %c %d %d %d %d %d %u %lu \
>> %lu %lu %lu %lu %lu %ld %ld %ld %ld %d 0 %llu %lu %ld %lu %lu %lu %lu %lu \
>> -%lu %lu %lu %lu %lu %lu %lu %lu %d %d %u %u %llu %lu %ld\n",
>> +%lu %lu %lu %lu %lu %lu %lu %lu %d %d %u %u %llu %llu %lu %ld\n",
>> pid_nr_ns(pid, ns),
>> tcomm,
>> state,
>> @@ -544,6 +544,7 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns,
>> task->rt_priority,
>> task->policy,
>> (unsigned long long)delayacct_blkio_ticks(task),
>> + (unsigned long long)delayacct_freepages_ticks(task),
>> cputime_to_clock_t(gtime),
>> cputime_to_clock_t(cgtime));
>
> userland program oftern below access.
> thus, this patch break userland compatibility.
It's my fault. I suggested him to group delayacct ticks.
I know this breakage, but was not sure what it affects.
I thought that if it'll be a problem we get a claim such you did.
Thanks,
Hiroshi Shimamoto
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists