lkml.org 
[lkml]   [2013]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [sched/get_online_cpus] INFO: task swapper/0:1 blocked for more than 120 seconds.
Hi, Fengguang

On 11/10/2013 06:16 PM, Fengguang Wu wrote:
> Greetings,
>
> I got the below dmesg and the first bad commit is

I guess this will disappear when '!CONFIG_RCU_BOOST'...

AFAIK, if the rsp was in boost mode, we count on smpboot-thread
'rcu_cpu_thread_spec' to finish the callback, which will be
parked before do sync-rcu inside _cpu_down(), if that was true,
then the sync will never finish...

May be some brainless fix like this?



diff --git a/kernel/cpu.c b/kernel/cpu.c
index 63aa50d..aa24338 100644
--- a/kernel/cpu.c
+++ b/kernel/cpu.c
@@ -306,7 +306,6 @@ static int __ref _cpu_down(unsigned int cpu, int tasks_frozen)
__func__, cpu);
goto out_release;
}
- smpboot_park_threads(cpu);

/*
* By now we've cleared cpu_active_mask, wait for all preempt-disabled
@@ -321,6 +320,8 @@ static int __ref _cpu_down(unsigned int cpu, int tasks_frozen)
#endif
synchronize_rcu();

+ smpboot_park_threads(cpu);
+
/*
* So now all preempt/rcu users must observe !cpu_active().
*/
Regards,
Michael Wang

>
> commit 6acce3ef84520537f8a09a12c9ddbe814a584dd2
> Author: Peter Zijlstra <peterz@infradead.org>
> Date: Fri Oct 11 14:38:20 2013 +0200
>
> sched: Remove get_online_cpus() usage
>
> Remove get_online_cpus() usage from the scheduler; there's 4 sites that
> use it:
>
> - sched_init_smp(); where its completely superfluous since we're in
> 'early' boot and there simply cannot be any hotplugging.
>
> - sched_getaffinity(); we already take a raw spinlock to protect the
> task cpus_allowed mask, this disables preemption and therefore
> also stabilizes cpu_online_mask as that's modified using
> stop_machine. However switch to active mask for symmetry with
> sched_setaffinity()/set_cpus_allowed_ptr(). We guarantee active
> mask stability by inserting sync_rcu/sched() into _cpu_down.
>
> - sched_setaffinity(); we don't appear to need get_online_cpus()
> either, there's two sites where hotplug appears relevant:
> * cpuset_cpus_allowed(); for the !cpuset case we use possible_mask,
> for the cpuset case we hold task_lock, which is a spinlock and
> thus for mainline disables preemption (might cause pain on RT).
> * set_cpus_allowed_ptr(); Holds all scheduler locks and thus has
> preemption properly disabled; also it already deals with hotplug
> races explicitly where it releases them.
>
> - migrate_swap(); we can make stop_two_cpus() do the heavy lifting for
> us with a little trickery. By adding a sync_sched/rcu() after the
> CPU_DOWN_PREPARE notifier we can provide preempt/rcu guarantees for
> cpu_active_mask. Use these to validate that both our cpus are active
> when queueing the stop work before we queue the stop_machine works
> for take_cpu_down().
>
> Signed-off-by: Peter Zijlstra <peterz@infradead.org>
> Cc: "Srivatsa S. Bhat" <srivatsa.bhat@linux.vnet.ibm.com>
> Cc: Paul McKenney <paulmck@linux.vnet.ibm.com>
> Cc: Mel Gorman <mgorman@suse.de>
> Cc: Rik van Riel <riel@redhat.com>
> Cc: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
> Cc: Andrea Arcangeli <aarcange@redhat.com>
> Cc: Johannes Weiner <hannes@cmpxchg.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Steven Rostedt <rostedt@goodmis.org>
> Cc: Oleg Nesterov <oleg@redhat.com>
> Link: http://lkml.kernel.org/r/20131011123820.GV3081@twins.programming.kicks-ass.net
> Signed-off-by: Ingo Molnar <mingo@kernel.org>
>
> ===================================================
> PARENT COMMIT NOT CLEAN. LOOK OUT FOR WRONG BISECT!
> ===================================================
>
> +------------------------------------------------------------------------------------+--------------+--------------+
> | | 746023159c40 | 1918664d0b2b |
> +------------------------------------------------------------------------------------+--------------+--------------+
> | good_boots | 594 | 7 |
> | has_kernel_error_warning | 6 | 12 |
> | BUG:kernel_early_hang_without_any_printk_output | 6 | |
> | INFO:task_blocked_for_more_than_seconds | 0 | 12 |
> | INFO:NMI_handler(arch_trigger_all_cpu_backtrace_handler)took_too_long_to_run:msecs | 0 | 12 |
> | Kernel_panic-not_syncing:hung_task:blocked_tasks | 0 | 12 |
> +------------------------------------------------------------------------------------+--------------+--------------+
>
> [ 10.410377] EDD information not available.
> [ 10.412866] Unregister pv shared memory for cpu 0
> [ 10.499606] input: ImExPS/2 Generic Explorer Mouse as /devices/platform/i8042/serio1/input/input3
> [ 241.206735] INFO: task swapper/0:1 blocked for more than 120 seconds.
> [ 241.209244] Not tainted 3.12.0-02098-g1918664 #98
> [ 241.211290] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
> [ 241.214286] swapper/0 D ffff88000ec9e040 0 1 0 0x00000000
> [ 241.217142] ffff88000eca1c10 0000000000000046 ffffffff826aca60 ffff88000ec9e040
> [ 241.220200] ffff88000eca1fd8 0000000000012a00 0000000000012a00 ffff88000c9927c0
> [ 241.223196] ffff88000ec9e040 ffff88000ec9e040 ffff88000eca1b60 0000000000000002
> [ 241.226835] Call Trace:
> [ 241.227813] [<ffffffff8110e544>] ? __lock_acquire+0x1e74/0x1ef0
> [ 241.230123] [<ffffffff8110e544>] ? __lock_acquire+0x1e74/0x1ef0
> [ 241.232400] [<ffffffff81d39ea5>] schedule+0x55/0x60
> [ 241.234345] [<ffffffff81d38efb>] schedule_timeout+0x2b/0x200
> [ 241.236533] [<ffffffff8110a88e>] ? mark_held_locks+0x12e/0x150
> [ 241.238845] [<ffffffff81d3e577>] ? _raw_spin_unlock_irq+0x27/0x60
> [ 241.241356] [<ffffffff810f753d>] ? get_parent_ip+0xd/0x50
> [ 241.243790] [<ffffffff810f7603>] ? preempt_count_sub+0x33/0x50
> [ 241.246273] [<ffffffff81d3a9c7>] wait_for_common+0x167/0x1b0
> [ 241.248715] [<ffffffff810f9470>] ? try_to_wake_up+0x330/0x330
> [ 241.251291] [<ffffffff81124df0>] ? __call_rcu.constprop.42+0x3d0/0x3d0
> [ 241.254145] [<ffffffff81d3ab48>] wait_for_completion+0x18/0x20
> [ 241.256756] [<ffffffff8111ed16>] wait_rcu_gp+0x46/0x50
> [ 241.259000] [<ffffffff8111eb30>] ? ftrace_raw_output_rcu_barrier+0x70/0x70
> [ 241.262012] [<ffffffff811263dc>] synchronize_rcu+0x2c/0x30
> [ 241.264437] [<ffffffff81d1bd82>] _cpu_down+0x2b2/0x340
> [ 241.266760] [<ffffffff81d3afea>] ? mutex_lock_nested+0x3da/0x400
> [ 241.269311] [<ffffffff8106c832>] ? cpu_hotplug_driver_lock+0x12/0x20
> [ 241.272113] [<ffffffff81d1bec1>] cpu_down+0x31/0x50
> [ 241.274282] [<ffffffff81d1aaa7>] _debug_hotplug_cpu+0x47/0xb0
> [ 241.276853] [<ffffffff824251f3>] ? pci_iommu_alloc+0x6c/0x6c
> [ 241.279183] [<ffffffff82425200>] debug_hotplug_cpu+0xd/0x11
> [ 241.281679] [<ffffffff8241ce54>] do_one_initcall+0x7e/0x11b
> [ 241.284140] [<ffffffff810e9a1d>] ? parse_args+0x1dd/0x300
> [ 241.286665] [<ffffffff8241cfed>] kernel_init_freeable+0xfc/0x18d
> [ 241.289336] [<ffffffff8241c7ec>] ? do_early_param+0x8a/0x8a
> [ 241.291894] [<ffffffff81d1a8e0>] ? rest_init+0xc0/0xc0
> [ 241.294233] [<ffffffff81d1a8e9>] kernel_init+0x9/0x180
> [ 241.296494] [<ffffffff81d3f9cc>] ret_from_fork+0x7c/0xb0
> [ 241.298917] [<ffffffff81d1a8e0>] ? rest_init+0xc0/0xc0
> [ 241.303393] 3 locks held by swapper/0/1:
> [ 241.305121] #0: (x86_cpu_hotplug_driver_mutex){+.+.+.}, at: [<ffffffff8106c832>] cpu_hotplug_driver_lock+0x12/0x20
> [ 241.309967] #1: (cpu_add_remove_lock){+.+.+.}, at: [<ffffffff81d1bead>] cpu_down+0x1d/0x50
> [ 241.314045] #2: (cpu_hotplug.lock){+.+.+.}, at: [<ffffffff810cca86>] cpu_hotplug_begin+0x26/0x60
> [ 241.318328] sending NMI to all CPUs:
> [ 241.319960] NMI backtrace for cpu 1
> [ 241.321503] CPU: 1 PID: 29 Comm: khungtaskd Not tainted 3.12.0-02098-g1918664 #98
> [ 241.321618] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011
> [ 241.321618] task: ffff88000cf8e780 ti: ffff88000cf90000 task.ti: ffff88000cf90000
> [ 241.321618] RIP: 0010:[<ffffffff8107377d>] [<ffffffff8107377d>] flat_send_IPI_mask+0x8d/0xd0
> [ 241.321618] RSP: 0000:ffff88000cf91db8 EFLAGS: 00010046
> [ 241.321618] RAX: 0000000003000000 RBX: 0000000000000002 RCX: 0000000000000007
> [ 241.321618] RDX: 0000000000000c00 RSI: 0000000000000001 RDI: 0000000000000300
> [ 241.321618] RBP: ffff88000cf91dd8 R08: 0000000000000000 R09: 0000000000000000
> [ 241.321618] R10: 0000000000000001 R11: 0000000000000001 R12: 0000000000000246
> [ 241.321618] R13: 0000000000000c00 R14: 0000000000000003 R15: 00000000000003ff
> [ 241.321618] FS: 0000000000000000(0000) GS:ffff88000f100000(0000) knlGS:0000000000000000
> [ 241.321618] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b
> [ 241.321618] CR2: 00000000ffffffff CR3: 0000000002206000 CR4: 00000000000006a0
> [ 241.321618] Stack:
> [ 241.321618] 0000000000002710 ffff88000ec9e040 0000000000007fff ffff88000ec9e040
> [ 241.321618] ffff88000cf91de8 ffffffff810737da ffff88000cf91e00 ffffffff81070a3f
> [ 241.321618] 0000000000000078 ffff88000cf91e68 ffffffff8113e1a8 ffffffff8113df4b
> [ 241.321618] Call Trace:
> [ 241.321618] [<ffffffff810737da>] flat_send_IPI_all+0x1a/0x70
> [ 241.321618] [<ffffffff81070a3f>] arch_trigger_all_cpu_backtrace+0x5f/0xb0
> [ 241.321618] [<ffffffff8113e1a8>] watchdog+0x2d8/0x3d0
> [ 241.321618] [<ffffffff8113df4b>] ? watchdog+0x7b/0x3d0
> [ 241.321618] [<ffffffff8113ded0>] ? hung_task_panic+0x20/0x20
> [ 241.321618] [<ffffffff810eb431>] kthread+0xd1/0xe0
> [ 241.321618] [<ffffffff81d3e577>] ? _raw_spin_unlock_irq+0x27/0x60
> [ 241.321618] [<ffffffff810eb360>] ? __kthread_unpark+0x50/0x50
> [ 241.321618] [<ffffffff81d3f9cc>] ret_from_fork+0x7c/0xb0
> [ 241.321618] [<ffffffff810eb360>] ? __kthread_unpark+0x50/0x50
> [ 241.321618] Code: e6 10 75 f2 44 89 f0 c1 e0 18 89 04 25 10 b3 57 ff 89 da 44 09 ea 41 81 cd 00 04 00 00 83 fb 02 41 0f 44 d5 89 14 25 00 b3 57 ff <41> f7 c4 00 02 00 00 75 1a 4c 89 e7 57 9d 0f 1f 44 00 00 e8 8b
>
> git bisect start 1918664d0b2b6b2ad6b180e7c04a42872beb6691 d8524ae9d6f492a9c6db9f4d89c5f9b8782fa2d5 --
> git bisect good 4d9218daae2b00379b2bb81eaf4fe5143f372db1 # 11:54 27+ 0 Merge tag 'perf-core-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/core
> git bisect good f05ef3f6f78e6b97f5664a8290f1ac0717b1a1b0 # 12:09 27+ 2 kvm tools: Switch to using an enum for disk image types
> git bisect good 3784592e0c0f224e399605cfa2d1570f79c0cc29 # 12:17 27+ 6 config: fix make kvmconfig
> git bisect bad 9c028f0df729ba4958fe7b142d8521b66ee154d9 # 12:31 6- 4 Merge branch 'sched/core'
> git bisect good 04bb2f9475054298f0c67a89ca92cade42d3fe5e # 12:45 50+ 0 sched/numa: Adjust scan rate in task_numa_placement
> git bisect good ecf1f014325ba60f4df35edae1a357c67c5d4eb1 # 12:56 50+ 3 Merge branch 'core/rcu' into core/locking, to prepare the kernel/locking/ file move
> git bisect bad 01768b42dc97a67b4fb33a2535c49fc1969880df # 13:08 21- 18 locking: Move the mutex code to kernel/locking/
> git bisect bad c2d816443ef305aba8eaf0bf368f4d3d87494f06 # 13:22 4- 7 sched/wait: Introduce prepare_to_wait_event()
> git bisect good ec0ad3d01f99d5e5b56a99a58f7003b99250dc65 # 14:11 50+ 9 Merge branch 'core/urgent' into sched/core
> git bisect good 7c3f2ab7b844f1a859afbc3d41925e8a0faba5fa # 14:50 50+ 4 sched/rt: Add missing rmb()
> git bisect bad 6acce3ef84520537f8a09a12c9ddbe814a584dd2 # 14:55 13- 3 sched: Remove get_online_cpus() usage
> git bisect good 746023159c40c523b08a3bc3d213dac212385895 # 15:33 150+ 6 sched: Fix race in migrate_swap_stop()
> git bisect good 746023159c40c523b08a3bc3d213dac212385895 # 16:04 450+ 6 sched: Fix race in migrate_swap_stop()
> git bisect bad 1918664d0b2b6b2ad6b180e7c04a42872beb6691 # 16:04 0- 12 Merge branch 'x86/intel-mid'
> git bisect good 1ae1820bfe3f0964cf4bc3c5628055330e3cc538 # 16:47 450+ 0 Revert "sched: Remove get_online_cpus() usage"
> git bisect good 6c86ae2928f9e4cbf0d5844f5fcfd549e3450b8c # 17:26 450+ 63 Merge tag 'ftrace-urgent-3.12-v2' of git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace
> git bisect bad bad570d77024cba808126d68a2072f6f8ce64c27 # 17:37 32- 16 Add linux-next specific files for 20131108
> git bisect bad 1918664d0b2b6b2ad6b180e7c04a42872beb6691 # 17:37 0- 12 Merge branch 'x86/intel-mid'
>
> Thanks,
> Fengguang
>
>
>
> _______________________________________________
> LKP mailing list
> LKP@linux.intel.com
>



\
 
 \ /
  Last update: 2013-11-11 09:21    [W:0.079 / U:0.688 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site