Skip to contentSkip to conversation

Warnings about tasks on CPU 1 during STR

PATCH In review Frans Pop 2 participants 7 messages Jan 25, 2010 – Jan 28, 2010 CPU hotplug

lore
Share & tools
lore

Frans Pop

author
#0 lore raw

With 2.6.33-rc5 during suspend to RAM I'm getting a bunch of warnings I've not seen with earlier kernel versions:

PM: late suspend of devices complete after 28.784 msecs
ACPI: Preparing to enter system sleep state S3
Disabling non-boot CPUs ...
CPU 1 is now offline
SMP alternatives: switching to UP code
Task events/1 (pid = 10) is on cpu 1 (state = 1, flags = 84208040) Task khelper (pid = 11) is on cpu 1 (state = 1, flags = 80208040) Task sync_supers (pid = 77) is on cpu 1 (state = 1, flags = 80208040) Task bdi-default (pid = 79) is on cpu 1 (state = 2, flags = 80a15040) Task kblockd/1 (pid = 81) is on cpu 1 (state = 1, flags = 84208040) [...]
Task pm-suspend (pid = 29782) is on cpu 1 (state = 2, flags = 414000) Extended CMOS year: 2000
Back to C!

All the "Task X is on cpu 1" messages (76 in total) are new. Bug or intentional? As it is a warning, I suspect a bug.

Cheers,
FJP

maintainer
#1 lore raw
Quoted 18 lines · Frans Pop

With 2.6.33-rc5 during suspend to RAM I'm getting a bunch of warnings I've not seen with earlier kernel versions:

PM: late suspend of devices complete after 28.784 msecs
ACPI: Preparing to enter system sleep state S3
Disabling non-boot CPUs ...
CPU 1 is now offline
SMP alternatives: switching to UP code
Task events/1 (pid = 10) is on cpu 1 (state = 1, flags = 84208040) Task khelper (pid = 11) is on cpu 1 (state = 1, flags = 80208040) Task sync_supers (pid = 77) is on cpu 1 (state = 1, flags = 80208040) Task bdi-default (pid = 79) is on cpu 1 (state = 2, flags = 80a15040) Task kblockd/1 (pid = 81) is on cpu 1 (state = 1, flags = 84208040) [...]
Task pm-suspend (pid = 29782) is on cpu 1 (state = 2, flags = 414000) Extended CMOS year: 2000
Back to C!

Quoted 2 lines

All the "Task X is on cpu 1" messages (76 in total) are new. Bug or intentional? As it is a warning, I suspect a bug.

http://lkml.org/lkml/2010/1/21/175

author
#2 lore raw
Quoted 5 lines · Peter Zijlstra
Quoted 2 lines · Frans Pop

All the "Task X is on cpu 1" messages (76 in total) are new. Bug or intentional? As it is a warning, I suspect a bug.

http://lkml.org/lkml/2010/1/21/175

Thanks Peter, that fixes it.

I've a small cleanup patch for the warning that I'll send separately.

Cheers,
FJP

author
#3 lore raw
In reply to Peter Zijlstra
Quoted 5 lines · Peter Zijlstra
Quoted 2 lines · Frans Pop

All the "Task X is on cpu 1" messages (76 in total) are new. Bug or intentional? As it is a warning, I suspect a bug.

http://lkml.org/lkml/2010/1/21/175

Thanks Peter, that fixes it.

I've a small cleanup patch for the warning that I'll send separately.

Cheers,
FJP

author
#4 lore raw

PATCH 1/1 sched: correct whitespace in warning from cpu down task check

In reply to Frans Pop

Due to an incorrect line break the output currently contains tabs. Also remove trailing space.

Signed-off-by Frans Pop <elendilplanet.nl>
Cc Ingo Molnar <[email protected]>
Cc Peter Zijlstra <[email protected]>

---

The actual output that logcheck sent me looked like this:
Task events/1 (pid = 10) is on cpu 1^I^I^I^I(state = 1, flags = 84208040)

After this patch it becomes:
Task events/1 (pid = 10) is on cpu 1 (state = 1, flags = 84208040)

kernel/cpu.c +4 -4
index 08e54e7..677f253 100644
@@ -154,10 +154,10 @@ static inline void check_for_tasks(int cpu)
 		if (task_cpu(p) == cpu && p->state == TASK_RUNNING &&
 		    (!cputime_eq(p->utime, cputime_zero) ||
 		     !cputime_eq(p->stime, cputime_zero)))
-			printk(KERN_WARNING "Task %s (pid = %d) is on cpu %d\
-				(state = %ld, flags = %x) \n",
-				 p->comm, task_pid_nr(p), cpu,
-				 p->state, p->flags);
+			printk(KERN_WARNING "Task %s (pid = %d) is on cpu %d "
+				"(state = %ld, flags = %x)\n",
+				p->comm, task_pid_nr(p), cpu,
+				p->state, p->flags);
 	}
 	write_unlock_irq(&tasklist_lock);
 }
#5 lore raw

[tip:sched/urgent] sched: Correct printk whitespace in warning from cpu down task check

Commit-ID:  8048f77492bc428c6783e5039e759de2f960f47d
Gitweb:     http://git.kernel.org/tip/8048f77492bc428c6783e5039e759de2f960f47d
Author:     Frans Pop <[email protected]>
AuthorDate: Mon, 25 Jan 2010 14:56:34 +0100
Committer:  Ingo Molnar <[email protected]>
CommitDate: Wed, 27 Jan 2010 08:34:40 +0100

sched: Correct printk whitespace in warning from cpu down task check

Due to an incorrect line break the output currently contains tabs. Also remove trailing space.

The actual output that logcheck sent me looked like this:
Task events/1 (pid = 10) is on cpu 1^I^I^I^I(state = 1, flags = 84208040)

After this patch it becomes:
Task events/1 (pid = 10) is on cpu 1 (state = 1, flags = 84208040)

Signed-off-by Frans Pop <elendilplanet.nl>

LKML-Reference: <[email protected]>

---
 kernel/cpu.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
kernel/cpu.c +4 -4
index 08e54e7..677f253 100644
@@ -154,10 +154,10 @@ static inline void check_for_tasks(int cpu)
 		if (task_cpu(p) == cpu && p->state == TASK_RUNNING &&
 		    (!cputime_eq(p->utime, cputime_zero) ||
 		     !cputime_eq(p->stime, cputime_zero)))
-			printk(KERN_WARNING "Task %s (pid = %d) is on cpu %d\
-				(state = %ld, flags = %x) \n",
-				 p->comm, task_pid_nr(p), cpu,
-				 p->state, p->flags);
+			printk(KERN_WARNING "Task %s (pid = %d) is on cpu %d "
+				"(state = %ld, flags = %x)\n",
+				p->comm, task_pid_nr(p), cpu,
+				p->state, p->flags);
 	}
 	write_unlock_irq(&tasklist_lock);
 }
#6 lore raw

[tip:sched/urgent] sched: Correct printk whitespace in warning from cpu down task check

In reply to Frans Pop
Commit-ID:  9d3cfc4c1d17c6d3bc1373e3b954c56b92607755
Gitweb:     http://git.kernel.org/tip/9d3cfc4c1d17c6d3bc1373e3b954c56b92607755
Author:     Frans Pop <[email protected]>
AuthorDate: Mon, 25 Jan 2010 14:56:34 +0100
Committer:  Ingo Molnar <[email protected]>
CommitDate: Thu, 28 Jan 2010 06:59:55 +0100

sched: Correct printk whitespace in warning from cpu down task check

Due to an incorrect line break the output currently contains tabs. Also remove trailing space.

The actual output that logcheck sent me looked like this:
Task events/1 (pid = 10) is on cpu 1^I^I^I^I(state = 1, flags = 84208040)

After this patch it becomes:
Task events/1 (pid = 10) is on cpu 1 (state = 1, flags = 84208040)

Signed-off-by Frans Pop <elendilplanet.nl>

LKML-Reference: <[email protected]>

---
 kernel/cpu.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
kernel/cpu.c +4 -4
index 08e54e7..677f253 100644
@@ -154,10 +154,10 @@ static inline void check_for_tasks(int cpu)
 		if (task_cpu(p) == cpu && p->state == TASK_RUNNING &&
 		    (!cputime_eq(p->utime, cputime_zero) ||
 		     !cputime_eq(p->stime, cputime_zero)))
-			printk(KERN_WARNING "Task %s (pid = %d) is on cpu %d\
-				(state = %ld, flags = %x) \n",
-				 p->comm, task_pid_nr(p), cpu,
-				 p->state, p->flags);
+			printk(KERN_WARNING "Task %s (pid = %d) is on cpu %d "
+				"(state = %ld, flags = %x)\n",
+				p->comm, task_pid_nr(p), cpu,
+				p->state, p->flags);
 	}
 	write_unlock_irq(&tasklist_lock);
 }