ARM64: sched: fix bug: remove printk while schedule is in progress
authorChen Liang <cl@rock-chips.com>
Fri, 8 Jul 2016 02:37:53 +0000 (10:37 +0800)
committerHuang, Tao <huangtao@rock-chips.com>
Fri, 8 Jul 2016 06:25:13 +0000 (14:25 +0800)
It will cause deadlock and while(1) if call printk while schedule
is in progress. The block state like as below:

cpu0(hold the console sem):
printk->console_unlock->up_sem->spin_lock(&sem->lock)->wake_up_process(cpu1)
->try_to_wake_up(cpu1)->while(p->on_cpu).

cpu1(request console sem):
console_lock->down_sem->schedule->idle_banlance->update_cpu_capacity->
printk->console_trylock->spin_lock(&sem->lock).

p->on_cpu will be 1 forever, because the task is still running on cpu1,
so cpu0 is blocked in while(p->on_cpu), but cpu1 could not get
spin_lock(&sem->lock), it is blocked too, it means the task will running
on cpu1 forever.

Change-Id: I60d02d8c957273872f97939632bdd235accdad4e
Signed-off-by: Chen Liang <cl@rock-chips.com>
kernel/sched/fair.c

index 13fe9d8fac61e84a816338cca33ab47edbb39dc5..6d84df9366edaa0535b24636ad41f3f4d6a36d37 100644 (file)
@@ -6808,7 +6808,7 @@ static void update_cpu_capacity(struct sched_domain *sd, int cpu)
                mcc->cpu = cpu;
 #ifdef CONFIG_SCHED_DEBUG
                raw_spin_unlock_irqrestore(&mcc->lock, flags);
-               pr_info("CPU%d: update max cpu_capacity %lu\n", cpu, capacity);
+               //pr_info("CPU%d: update max cpu_capacity %lu\n", cpu, capacity);
                goto skip_unlock;
 #endif
        }