Commit-ID: 1ca4fa3ab604734e38e2a3000c9abf788512ffa7
Gitweb: https://git.kernel.org/tip/1ca4fa3ab604734e38e2a3000c9abf788512ffa7
Author: Hidetoshi Seto
AuthorDate: Tue, 29 Jan 2019 10:12:45 -0500
Committer: Ingo Molnar
CommitDate: Mon, 4 Feb 2019 09:13:21 +0100
sched/debug: Initialize
By changes in vtime* codes by previous patches, now account_idle_time()
become a function to be called only from tick-accounting codes.
Introduce __account_idle_ticks() to do iowait accounting in ticks
properly. For this purpose record jiffies at end of iowait.
Not-Tested-by: Hidetoshi Seto
Get iowait's timestamp for accounting w/ VIRT_CPU_ACCOUNTING_GEN.
(currently arm is only user of this?)
At last of this series of changes, introduce common function
vtime_iowait_exit to replace all arch_record_iowait_exit.
Not-tested-by: Hidetoshi Seto
---
arch/ia64/include/asm/cputime.h
Like s390 and ia64, ppc also has VIRT_CPU_ACCOUNTING.
Check "timestamp at end of iowait" for idle/iowait accounting.
Not-Tested-by: Hidetoshi Seto
---
arch/powerpc/include/asm/cputime.h |3 +++
arch/powerpc/kernel/time.c | 21 +
2 files changed, 20
Using VIRT_CPU_ACCOUNTING, ia64 utilize "timestamp at end of iowait"
like s390.
Not-Tested-by: Hidetoshi Seto
---
arch/ia64/include/asm/cputime.h |2 +
arch/ia64/kernel/time.c | 43 ++-
2 files changed, 44 insertions(+), 1 deletion
s390_get_idle_time give us the duration from idle entry to now.
But it does not tell us how to divide it to idle and iowait.
Modify this function to return 2 values. To realize this, s390's
cputime accounting also requires timestamp at end of iowait.
Not-Tested-by: Hidetoshi Seto
---
arch/s390
The current account_idle_time() cannot process mixed cputime which
contain both of idle cputime and iowait cputime.
So introduce new account_idle_and_iowait() to do paranoid work.
Following patches will add users of this new function.
Not-Tested-by: Hidetoshi Seto
---
kernel/sched/cputime.c
Now observer cpu can refer both of idle entry time and iowait exit
time of observed sleeping cpu, so observer can get idle/iowait time
of sleeping cpu by calculating cputimes not accounted yet.
Not-Tested-by: Hidetoshi Seto
---
include/linux/sched.h|1 +
kernel/sched/core.c | 27
Suggested-by: Peter Zijlstra
Not-Tested-by: Hidetoshi Seto
---
kernel/sched/core.c| 40
kernel/sched/cputime.c |2 +-
kernel/sched/sched.h |4 +++-
3 files changed, 32 insertions(+), 14 deletions(-)
diff --git a/kernel/sched/core.c b/kerne
/pub/linux/kernel/people/wli/vm/iowait/iowait-2.5.45-6
Thanks,
H.Seto
---
Hidetoshi Seto (8):
cputime, sched: record last_iowait
cputime, nohz: handle last_iowait for nohz
cputime: introduce account_idle_and_iowait
cputime, s390: introduce s390_get_idle_and_iowait
cpu
.
update patch description, separate from following changes.
(patch body does not changed from 1/2 of v4)
v1-4: https://lkml.org/lkml/2014/4/17/120
Signed-off-by: Hidetoshi Seto
Reported-by: Fernando Luis Vazquez Cao
Reported-by: Tetsuo Handa
Cc: Frederic Weisbecker
Cc: Thomas Gleixner
Cc
.
update patch description, separate from following changes.
(patch body does not changed from 1/2 of v4)
v1-4: https://lkml.org/lkml/2014/4/17/120
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reported-by: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Reported-by: Tetsuo
/kernel/people/wli/vm/iowait/iowait-2.5.45-6
Thanks,
H.Seto
---
Hidetoshi Seto (8):
cputime, sched: record last_iowait
cputime, nohz: handle last_iowait for nohz
cputime: introduce account_idle_and_iowait
cputime, s390: introduce s390_get_idle_and_iowait
cputime, ia64
Zijlstra pet...@infradead.org
Not-Tested-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
kernel/sched/core.c| 40
kernel/sched/cputime.c |2 +-
kernel/sched/sched.h |4 +++-
3 files changed, 32 insertions(+), 14 deletions(-)
diff --git
Now observer cpu can refer both of idle entry time and iowait exit
time of observed sleeping cpu, so observer can get idle/iowait time
of sleeping cpu by calculating cputimes not accounted yet.
Not-Tested-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
include/linux/sched.h|1
The current account_idle_time() cannot process mixed cputime which
contain both of idle cputime and iowait cputime.
So introduce new account_idle_and_iowait() to do paranoid work.
Following patches will add users of this new function.
Not-Tested-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
s390_get_idle_time give us the duration from idle entry to now.
But it does not tell us how to divide it to idle and iowait.
Modify this function to return 2 values. To realize this, s390's
cputime accounting also requires timestamp at end of iowait.
Not-Tested-by: Hidetoshi Seto seto.hideto
Using VIRT_CPU_ACCOUNTING, ia64 utilize timestamp at end of iowait
like s390.
Not-Tested-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
arch/ia64/include/asm/cputime.h |2 +
arch/ia64/kernel/time.c | 43 ++-
2 files changed, 44 insertions
Like s390 and ia64, ppc also has VIRT_CPU_ACCOUNTING.
Check timestamp at end of iowait for idle/iowait accounting.
Not-Tested-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
arch/powerpc/include/asm/cputime.h |3 +++
arch/powerpc/kernel/time.c | 21 +
2
Get iowait's timestamp for accounting w/ VIRT_CPU_ACCOUNTING_GEN.
(currently arm is only user of this?)
At last of this series of changes, introduce common function
vtime_iowait_exit to replace all arch_record_iowait_exit.
Not-tested-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
arch
By changes in vtime* codes by previous patches, now account_idle_time()
become a function to be called only from tick-accounting codes.
Introduce __account_idle_ticks() to do iowait accounting in ticks
properly. For this purpose record jiffies at end of iowait.
Not-Tested-by: Hidetoshi Seto
(2014/05/21 12:19), Chen Yucong wrote:
> On Wed, 2014-05-21 at 11:43 +0900, Hidetoshi Seto wrote:
>> (2014/05/21 11:03), Chen Yucong wrote:
>>> On Wed, 2014-05-21 at 10:40 +0900, Hidetoshi Seto wrote:
>>>> (2014/05/20 11:11), Chen Yucong wrote:
>>>>>
(2014/05/21 11:03), Chen Yucong wrote:
> On Wed, 2014-05-21 at 10:40 +0900, Hidetoshi Seto wrote:
>> (2014/05/20 11:11), Chen Yucong wrote:
>>> mces_seen is a Per-CPU variable which should only be accessed by Per-CPU as
>>> possible. So the
>>> clear operati
(2014/05/20 11:11), Chen Yucong wrote:
> mces_seen is a Per-CPU variable which should only be accessed by Per-CPU as
> possible. So the
> clear operation of mces_seen should also be lcoal to Per-CPU rather than
> monarch CPU.
I don't think it should be local.
Originally what we want to have
(2014/05/20 11:11), Chen Yucong wrote:
mces_seen is a Per-CPU variable which should only be accessed by Per-CPU as
possible. So the
clear operation of mces_seen should also be lcoal to Per-CPU rather than
monarch CPU.
I don't think it should be local.
Originally what we want to have here is
(2014/05/21 11:03), Chen Yucong wrote:
On Wed, 2014-05-21 at 10:40 +0900, Hidetoshi Seto wrote:
(2014/05/20 11:11), Chen Yucong wrote:
mces_seen is a Per-CPU variable which should only be accessed by Per-CPU as
possible. So the
clear operation of mces_seen should also be lcoal to Per-CPU
(2014/05/21 12:19), Chen Yucong wrote:
On Wed, 2014-05-21 at 11:43 +0900, Hidetoshi Seto wrote:
(2014/05/21 11:03), Chen Yucong wrote:
On Wed, 2014-05-21 at 10:40 +0900, Hidetoshi Seto wrote:
(2014/05/20 11:11), Chen Yucong wrote:
mces_seen is a Per-CPU variable which should only be accessed
(iowait_exittime - idle_entrytime)
> gets accounted as iowait, and the remaining (now - iowait_exittime)
> as "true" idle.
>
> Run-tested: /proc/stats no longer go backwards.
>
> Signed-off-by: Denys Vlasenko
> Cc: Frederic Weisbecker
> Cc: Hidetoshi Seto
> Cc
accounted as iowait, and the remaining (now - iowait_exittime)
as true idle.
Run-tested: /proc/stats no longer go backwards.
Signed-off-by: Denys Vlasenko dvlas...@redhat.com
Cc: Frederic Weisbecker fweis...@gmail.com
Cc: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Cc: Fernando Luis Vazquez Cao
(2014/04/23 18:41), Peter Zijlstra wrote:
> On Wed, Apr 23, 2014 at 04:40:18PM +0900, Hidetoshi Seto wrote:
>> (2014/04/23 4:45), Peter Zijlstra wrote:
>>> On Thu, Apr 17, 2014 at 06:41:41PM +0900, Hidetoshi Seto wrote:
>>>> [TARGET OF THIS PATCH]:
>>>>
&g
(2014/04/23 4:45), Peter Zijlstra wrote:
> On Thu, Apr 17, 2014 at 06:41:41PM +0900, Hidetoshi Seto wrote:
>> [TARGET OF THIS PATCH]:
>>
>> Complete rework for iowait accounting implies that some user
>> interfaces might be replaced completely. It will intro
(2014/04/23 18:41), Peter Zijlstra wrote:
On Wed, Apr 23, 2014 at 04:40:18PM +0900, Hidetoshi Seto wrote:
(2014/04/23 4:45), Peter Zijlstra wrote:
On Thu, Apr 17, 2014 at 06:41:41PM +0900, Hidetoshi Seto wrote:
[TARGET OF THIS PATCH]:
Complete rework for iowait accounting implies that some
(2014/04/23 4:45), Peter Zijlstra wrote:
On Thu, Apr 17, 2014 at 06:41:41PM +0900, Hidetoshi Seto wrote:
[TARGET OF THIS PATCH]:
Complete rework for iowait accounting implies that some user
interfaces might be replaced completely. It will introduce new
counter or so, and kill per-cpu
Ping?
(I'll have a week holidays from next week.
So thank you if you could give me your comments soon!)
Thanks,
H.Seto
(2014/04/17 18:35), Hidetoshi Seto wrote:
> Hi all,
>
> This patch set (rebased on v3.15-rc1) is my 4th try to fix an issue
> that idle/iowait of /proc/stat can
Ping?
(I'll have a week holidays from next week.
So thank you if you could give me your comments soon!)
Thanks,
H.Seto
(2014/04/17 18:35), Hidetoshi Seto wrote:
Hi all,
This patch set (rebased on v3.15-rc1) is my 4th try to fix an issue
that idle/iowait of /proc/stat can go backward
(2014/04/17 19:05), Peter Zijlstra wrote:
> Anyway, if you want to preserve the same broken ass crap we had pre
> NOHZ, something like the below should do that.
>
> I'm not really thrilled with iowait_{start,stop}() but I think they
> should have the same general cost as the atomic ops we already
qcount instead of seqlock
(achieved by inserting cleanup as former patch)
plus introduce delayed iowait accounting
v2: update comments and description about problem 2.
include fix for minor typo
Signed-off-by: Hidetoshi Seto
Reported-by: Fernando Luis Vazquez Cao
Reported-by: Tetsuo Handa
Cc
update_ts_time_stats(), fold
this static routine into tick_nohz_stop_idle().
(Still there is problem 2. Continue to following patch 2/2.)
Signed-off-by: Hidetoshi Seto
Reported-by: Fernando Luis Vazquez Cao
Reported-by: Tetsuo Handa
Cc: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc
so).
Of course still reviews are welcome.
Thanks,
H.Seto
Hidetoshi Seto (2):
nohz: make updating sleep stats local
nohz: delayed iowait accounting for nohz idle time stats
include/linux/tick.h |6 +-
kernel/time/tick-sched.c | 179
so).
Of course still reviews are welcome.
Thanks,
H.Seto
Hidetoshi Seto (2):
nohz: make updating sleep stats local
nohz: delayed iowait accounting for nohz idle time stats
include/linux/tick.h |6 +-
kernel/time/tick-sched.c | 179
update_ts_time_stats(), fold
this static routine into tick_nohz_stop_idle().
(Still there is problem 2. Continue to following patch 2/2.)
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reported-by: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Reported-by: Tetsuo Handa penguin-ker
by inserting cleanup as former patch)
plus introduce delayed iowait accounting
v2: update comments and description about problem 2.
include fix for minor typo
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reported-by: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Reported
(2014/04/17 19:05), Peter Zijlstra wrote:
Anyway, if you want to preserve the same broken ass crap we had pre
NOHZ, something like the below should do that.
I'm not really thrilled with iowait_{start,stop}() but I think they
should have the same general cost as the atomic ops we already had.
(2014/04/16 18:36), Peter Zijlstra wrote:
> On Wed, Apr 16, 2014 at 03:33:06PM +0900, Hidetoshi Seto wrote:
>> So we need 2 operations:
>> a) remove regression
>
> What regression; there's never been talk about a regression, just a bug
> found. AFAICT this 'regr
(2014/04/15 19:19), Peter Zijlstra wrote:
> On Thu, Apr 10, 2014 at 06:13:54PM +0900, Hidetoshi Seto wrote:
>> [WHAT THIS PATCH PROPOSED]:
>>
>> To fix problem 1, this patch adds seqcount for NO_HZ idle
>> accounting to avoid possible races between reader/writer.
>&
(2014/04/15 19:04), Peter Zijlstra wrote:
> On Thu, Apr 10, 2014 at 06:13:54PM +0900, Hidetoshi Seto wrote:
>> This patch is v3 of patch set to fix an issue that idle/iowait
>> of /proc/stat can go backward. Originally reported by Tetsuo and
>> Fernando at last year, Mar 201
(2014/04/15 19:04), Peter Zijlstra wrote:
On Thu, Apr 10, 2014 at 06:13:54PM +0900, Hidetoshi Seto wrote:
This patch is v3 of patch set to fix an issue that idle/iowait
of /proc/stat can go backward. Originally reported by Tetsuo and
Fernando at last year, Mar 2013.
[BACKGROUNDS]: idle
(2014/04/15 19:19), Peter Zijlstra wrote:
On Thu, Apr 10, 2014 at 06:13:54PM +0900, Hidetoshi Seto wrote:
[WHAT THIS PATCH PROPOSED]:
To fix problem 1, this patch adds seqcount for NO_HZ idle
accounting to avoid possible races between reader/writer.
And to cope with problem 2, I introduced
(2014/04/16 18:36), Peter Zijlstra wrote:
On Wed, Apr 16, 2014 at 03:33:06PM +0900, Hidetoshi Seto wrote:
So we need 2 operations:
a) remove regression
What regression; there's never been talk about a regression, just a bug
found. AFAICT this 'regression' is ever since we introduced NOHZ
Ping?
(2014/04/10 18:07), Hidetoshi Seto wrote:
> Hi all,
>
> This patch set (rebased on v3.14) is my 3rd try to fix an issue
> that idle/iowait of /proc/stat can go backward. Originally reported
> by Tetsuo and Fernando at last year, Mar 2013.
>
> This v3 takes new approa
Ping?
(2014/04/10 18:07), Hidetoshi Seto wrote:
Hi all,
This patch set (rebased on v3.14) is my 3rd try to fix an issue
that idle/iowait of /proc/stat can go backward. Originally reported
by Tetsuo and Fernando at last year, Mar 2013.
This v3 takes new approach to fix the problem
scription about problem 2.
include fix for minor typo
Signed-off-by: Hidetoshi Seto
Reported-by: Fernando Luis Vazquez Cao
Reported-by: Tetsuo Handa
Cc: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Andrew Morton
Cc: Arjan van de Ven
Cc: Oleg Nesterov
-off-by: Hidetoshi Seto
Cc: Fernando Luis Vazquez Cao
Cc: Tetsuo Handa
Cc: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: Andrew Morton
Cc: Arjan van de Ven
Cc: Oleg Nesterov
Cc: Preeti U Murthy
Cc: Denys Vlasenko
Cc:
---
include/linux/tick.h |4
still reviews are welcome!
Thanks,
H.Seto
Hidetoshi Seto (2):
nohz: stop updating sleep stats from get_cpu_{idle,iowait}_time_us()
nohz: use delayed iowait accounting to avoid race on idle time stats
include/linux/tick.h |6 ++-
kernel/time/tick-sched.c | 116
still reviews are welcome!
Thanks,
H.Seto
Hidetoshi Seto (2):
nohz: stop updating sleep stats from get_cpu_{idle,iowait}_time_us()
nohz: use delayed iowait accounting to avoid race on idle time stats
include/linux/tick.h |6 ++-
kernel/time/tick-sched.c | 116
-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Cc: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Cc: Tetsuo Handa penguin-ker...@i-love.sakura.ne.jp
Cc: Frederic Weisbecker fweis...@gmail.com
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@kernel.org
Cc: Peter Zijlstra pet
.
include fix for minor typo
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reported-by: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Reported-by: Tetsuo Handa penguin-ker...@i-love.sakura.ne.jp
Cc: Frederic Weisbecker fweis...@gmail.com
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo
(2014/04/03 18:51), Denys Vlasenko wrote:
> On Thu, Apr 3, 2014 at 9:02 AM, Hidetoshi Seto
> wrote:
>>>> [PROBLEM 2]: broken iowait accounting.
>>>>
>>>> As historical nature, cpu's idle time was accounted as either
>>>> idle or iowait
(2014/04/03 4:35), Denys Vlasenko wrote:
> On Mon, Mar 31, 2014 at 4:08 AM, Hidetoshi Seto
> wrote:
>> There are 2 problems:
>>
>> [PROBLEM 1]: there is no exclusive control.
>>
>> It is easy to understand that there are 2 different cpu - an
>> observing
(2014/04/03 18:51), Denys Vlasenko wrote:
On Thu, Apr 3, 2014 at 9:02 AM, Hidetoshi Seto
seto.hideto...@jp.fujitsu.com wrote:
[PROBLEM 2]: broken iowait accounting.
As historical nature, cpu's idle time was accounted as either
idle or iowait depending on the presence of tasks blocked by
I/O
(2014/04/03 4:35), Denys Vlasenko wrote:
On Mon, Mar 31, 2014 at 4:08 AM, Hidetoshi Seto
seto.hideto...@jp.fujitsu.com wrote:
There are 2 problems:
[PROBLEM 1]: there is no exclusive control.
It is easy to understand that there are 2 different cpu - an
observing cpu where running a program
reproducer and stressor for a day. The rate of reproduce
is different for different system, but in my case, running
"git gc" on kernel source repository aside of checker works fine.
Signed-off-by: Hidetoshi Seto
Reviewed-by: Preeti U Murthy
Reported-by: Fernando Luis Vazquez Cao
Reported-
9/86
v2: update comments and description about problem 2.
include fix for minor typo
Signed-off-by: Hidetoshi Seto
Reviewed-by: Preeti U Murthy
Reported-by: Fernando Luis Vazquez Cao
Reported-by: Tetsuo Handa
Cc: Frederic Weisbecker
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: Peter Zijlstra
Cc: And
Hi all,
This patch set (rebased on v3.14-rc8) will fix an issue that
idle/iowait of /proc/stat can go backward. Originally reported
by Tetsuo and Fernando at last year, Mar 2013.
v2 have Preeti's Reviewed-by (Thanks!).
Of course still reviews are welcome.
Thanks,
H.Seto
Hidetoshi Seto (2
Hi all,
This patch set (rebased on v3.14-rc8) will fix an issue that
idle/iowait of /proc/stat can go backward. Originally reported
by Tetsuo and Fernando at last year, Mar 2013.
v2 have Preeti's Reviewed-by (Thanks!).
Of course still reviews are welcome.
Thanks,
H.Seto
Hidetoshi Seto (2
about problem 2.
include fix for minor typo
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reviewed-by: Preeti U Murthy pre...@linux.vnet.ibm.com
Reported-by: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Reported-by: Tetsuo Handa penguin-ker...@i-love.sakura.ne.jp
Cc
for a day. The rate of reproduce
is different for different system, but in my case, running
git gc on kernel source repository aside of checker works fine.
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reviewed-by: Preeti U Murthy pre...@linux.vnet.ibm.com
Reported-by: Fernando Luis
(2014/03/24 16:45), Preeti Murthy wrote:
> Hi Hidetoshi,
>
> The patch looks good to me except the comments around the monotonicity
> of the return value of the idle stats observer. I am unable to relate them
> to the dependency on nr_iowait_cpu.
>
> I see that when the reader queries for the
(2014/03/24 16:45), Preeti Murthy wrote:
Hi Hidetoshi,
The patch looks good to me except the comments around the monotonicity
of the return value of the idle stats observer. I am unable to relate them
to the dependency on nr_iowait_cpu.
I see that when the reader queries for the idle
l/2013/8/8/638
[PATCH RESEND 0/4] nohz: Fix racy sleeptime stats
https://lkml.org/lkml/2013/8/16/274
2nd patchset from Frederic:
[RFC PATCH 0/5] nohz: Fix racy sleeptime stats v2
https://lkml.org/lkml/2013/10/19/86
Signed-off-by: Hidetoshi Seto
Reported-by: Fernando Luis Vazquez
reproducer and stressor for a day. The rate of reproduce
is different for different system, but in my case, running
"git gc" on kernel source repository aside of checker works fine.
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto
Reported-by: Fernando Luis Vazquez Cao
Reported-by: Tetsuo Handa
Cc:
Hi all,
This patch set (based on v3.14-rc7) will fix an issue that
idle/iowait of /proc/stat can go backward. Originally reported
by Tetsuo and Fernando at last year, Mar 2013.
Reviews are welcome.
Thanks,
H.Seto
Hidetoshi Seto (2):
nohz: use seqlock to avoid race on idle time stats
Hi all,
This patch set (based on v3.14-rc7) will fix an issue that
idle/iowait of /proc/stat can go backward. Originally reported
by Tetsuo and Fernando at last year, Mar 2013.
Reviews are welcome.
Thanks,
H.Seto
Hidetoshi Seto (2):
nohz: use seqlock to avoid race on idle time stats
0/4] nohz: Fix racy sleeptime stats
https://lkml.org/lkml/2013/8/16/274
2nd patchset from Frederic:
[RFC PATCH 0/5] nohz: Fix racy sleeptime stats v2
https://lkml.org/lkml/2013/10/19/86
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reported-by: Fernando Luis Vazquez Cao
and stressor for a day. The rate of reproduce
is different for different system, but in my case, running
git gc on kernel source repository aside of checker works fine.
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
Reported-by: Fernando Luis Vazquez Cao fernando...@lab.ntt.co.jp
Fix corporate name for copyright.
Signed-off-by: Hidetoshi Seto
---
include/linux/srcu.h |2 +-
kernel/rcu/srcu.c|2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/srcu.h b/include/linux/srcu.h
index 9b058ee..04f5abb 100644
--- a/include/linux/srcu.h
Fix corporate name for copyright.
Signed-off-by: Hidetoshi Seto
---
fs/btrfs/delayed-inode.c |2 +-
fs/btrfs/delayed-inode.h |2 +-
fs/btrfs/math.h |2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/fs/btrfs/delayed-inode.c b/fs/btrfs/delayed-inode.c
Fix corporate name for copyright.
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
include/linux/srcu.h |2 +-
kernel/rcu/srcu.c|2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/srcu.h b/include/linux/srcu.h
index 9b058ee..04f5abb
Fix corporate name for copyright.
Signed-off-by: Hidetoshi Seto seto.hideto...@jp.fujitsu.com
---
fs/btrfs/delayed-inode.c |2 +-
fs/btrfs/delayed-inode.h |2 +-
fs/btrfs/math.h |2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/fs/btrfs/delayed-inode.c b
Bob Picco wrote:
I will be pushing Peter Keilty's clocksource ia64 patches within the next
week or so. At that time I'll ask for inclusion into -mm. Please see:
http://marc.info/?t=11788158551=1=2
You'll notice that the time interpolator is removed.
I see. Your patch will replace the time
Bob Picco wrote:
I will be pushing Peter Keilty's clocksource ia64 patches within the next
week or so. At that time I'll ask for inclusion into -mm. Please see:
http://marc.info/?t=11788158551r=1w=2
You'll notice that the time interpolator is removed.
I see. Your patch will replace the
atch
- x3 process:
1.59us / 1 gettimeofday() call
1.11us / 1 gettimeofday() call with patch
- x4 process:
2.34us / 1 gettimeofday() call
1.29us / 1 gettimeofday() call with patch
I know that this patch could not help quite huge system since
such system like having 1024CPUs shoul
such system like having 1024CPUs should have better clocksource
instead of doing cmpxchg. Even though this patch will work good
on middle-sized box (4~8way, possibly 16~64way?).
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto [EMAIL PROTECTED]
-
arch/ia64/kernel/fsys.S | 10 +++---
kernel
Arjan van de Ven wrote:
>> It'd be nice if we could just teach the userspace balancer to not try to
>> move perpcu IRQs?
>>
>> otoh, the patch is super-cheap. Arjan?
>
> I can fix irqbalance no problem, however I like the kernel approach as
> well, since it's not just irqbalance that moves
Arjan van de Ven wrote:
It'd be nice if we could just teach the userspace balancer to not try to
move perpcu IRQs?
otoh, the patch is super-cheap. Arjan?
I can fix irqbalance no problem, however I like the kernel approach as
well, since it's not just irqbalance that moves irqs, sysadmins
t stuff of CPEI need to be fixed, but I think that
returning error to attempting move PER_CPU irq is useful for all
applications since it will never work.
Following small patch takes b) style.
It works, the warning disappeared and irqbalance still runs well.
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto <
well.
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto [EMAIL PROTECTED]
---
kernel/irq/proc.c |3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Index: linux-2.6.19/kernel/irq/proc.c
===
--- linux-2.6.19.orig/kernel/irq/proc.c
Matthew Wilcox wrote:
On Thu, Sep 01, 2005 at 05:45:54PM -0500, Brent Casavant wrote:
I am extremely concerned about the performance implications of this
implementation. These changes have several deleterious effects on I/O
performance.
I agree. I think the iochk patches should be
Oh my,
Hidetoshi Seto wrote:
I'd like to merge this part into 2.6.13-rc1 even if the latter half isn't
This is typo, should be 2.6.14-rc1. :-p
Thanks,
H.Seto
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTE
Oh my,
Hidetoshi Seto wrote:
I'd like to merge this part into 2.6.13-rc1 even if the latter half isn't
This is typo, should be 2.6.14-rc1. :-p
Thanks,
H.Seto
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo
Matthew Wilcox wrote:
On Thu, Sep 01, 2005 at 05:45:54PM -0500, Brent Casavant wrote:
I am extremely concerned about the performance implications of this
implementation. These changes have several deleterious effects on I/O
performance.
I agree. I think the iochk patches should be
to merge this part into 2.6.13-rc1 even if the latter half isn't
accepted. This former half functions without the latter, and helps
realize of effective recovery from MCA.
Tony, could you apply this part to your tree?
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto <[EMAIL PROTECTED]>
---
arc
on the situation.
Comments, to this paranoia part, are welcomed.
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto <[EMAIL PROTECTED]>
---
arch/ia64/kernel/mca.c | 21 +
arch/ia64/lib/iomap_check.c | 28 +++-
include/asm-ia64/io.h
Thank you for your comment, Brent.
Brent Casavant wrote:
On Thu, 1 Sep 2005, Hidetoshi Seto wrote:
static inline unsigned int
___ia64_inb (unsigned long port)
{
volatile unsigned char *addr = __ia64_mk_io_addr(port);
unsigned char ret;
+ unsigned long flags
Thank you for your comment, Brent.
Brent Casavant wrote:
On Thu, 1 Sep 2005, Hidetoshi Seto wrote:
static inline unsigned int
___ia64_inb (unsigned long port)
{
volatile unsigned char *addr = __ia64_mk_io_addr(port);
unsigned char ret;
+ unsigned long flags
to merge this part into 2.6.13-rc1 even if the latter half isn't
accepted. This former half functions without the latter, and helps
realize of effective recovery from MCA.
Tony, could you apply this part to your tree?
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto [EMAIL PROTECTED]
---
arch/ia64
This patch implements IOCHK interfaces that enable PCI drivers to
detect error and make their error handling easier.
Please refer archives if you need, e.g. http://lwn.net/Articles/139240/
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto <[EMAIL PROTECTED]>
---
drivers/pci
This patch implements ia64-specific IOCHK interfaces that enable
PCI drivers to detect error and make their error handling easier.
Please refer archives if you need, e.g. http://lwn.net/Articles/139240/
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto <[EMAIL PROTECTED]>
---
arch/ia64/K
This patch implements IOCHK interfaces that enable PCI drivers to
detect error and make their error handling easier.
Please refer archives if you need, e.g. http://lwn.net/Articles/139240/
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto [EMAIL PROTECTED]
---
drivers/pci/pci.c |2
This patch implements ia64-specific IOCHK interfaces that enable
PCI drivers to detect error and make their error handling easier.
Please refer archives if you need, e.g. http://lwn.net/Articles/139240/
Thanks,
H.Seto
Signed-off-by: Hidetoshi Seto [EMAIL PROTECTED]
---
arch/ia64/Kconfig
1 - 100 of 148 matches
Mail list logo