From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966816AbbBEIee (ORCPT ); Thu, 5 Feb 2015 03:34:34 -0500 Received: from szxga02-in.huawei.com ([119.145.14.65]:25053 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966099AbbBEIec (ORCPT ); Thu, 5 Feb 2015 03:34:32 -0500 Message-ID: <54D32AD4.1060003@huawei.com> Date: Thu, 5 Feb 2015 16:33:24 +0800 From: Zefan Li User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 MIME-Version: 1.0 To: Ingo Molnar CC: Peter Zijlstra , Mike Galbraith , LKML , Stefan Bader Subject: [PATCH] sched, autogroup: Fix failure when writing to cpu.rt_runtime_us X-Enigmail-Version: 1.6 Content-Type: text/plain; charset="GB2312" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.177.18.230] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is how to reproduce the bug: int main() { struct sched_param param = {.sched_priority=1}; if (fork() > 0) exit(0); setsid(); if (sched_setscheduler(0, SCHED_RR, ¶m) < 0){ perror("failed to sched_setscheduler()"); return -1; } while(1) ; } # ./test # mount -t cgroup -o cpu xxx /cgroup # cat /cgroup/cpu.rt_runtime_us 950000 # echo 940000 > /cgroup/cpu.rt_runtime_us Device or Resource busy An autogroup has been created and there's an RT task in it. RT tasks in autogroups are redirected to the root group and task_group() should return &root_task_group, but it's broken and returns the autogroup. We should reset p->sched_task_group when changing a normal task to an RT task which is in autogroup. Fixes: 8323f26ce342 ("sched: Fix race in task_group()") Cc: # 3.6+ Reported-by: Zhang Wei Signed-off-by: Zefan Li --- kernel/sched/core.c | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 89e7283..fccde96 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c @@ -3414,6 +3414,7 @@ static int __sched_setscheduler(struct task_struct *p, const struct sched_class *prev_class; struct rq *rq; int reset_on_fork; + bool reset_task_group = false; /* may grab non-irq protected spin_locks */ BUG_ON(in_interrupt()); @@ -3546,10 +3547,13 @@ change: * assigned. */ if (rt_bandwidth_enabled() && rt_policy(policy) && - task_group(p)->rt_bandwidth.rt_runtime == 0 && - !task_group_is_autogroup(task_group(p))) { - task_rq_unlock(rq, p, &flags); - return -EPERM; + task_group(p)->rt_bandwidth.rt_runtime == 0) { + if (!task_group_is_autogroup(task_group(p))) { + task_rq_unlock(rq, p, &flags); + return -EPERM; + } else { + reset_task_group = true; + } } #endif #ifdef CONFIG_SMP @@ -3615,6 +3619,9 @@ change: prev_class = p->sched_class; __setscheduler(rq, p, attr); + if (reset_task_group) + p->sched_task_group = &root_task_group; + if (running) p->sched_class->set_curr_task(rq); if (queued) { -- 1.8.0.2