From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758012AbbEaAfd (ORCPT ); Sat, 30 May 2015 20:35:33 -0400 Received: from blu004-omc1s20.hotmail.com ([65.55.116.31]:59489 "EHLO BLU004-OMC1S20.hotmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757838AbbEaAfC (ORCPT ); Sat, 30 May 2015 20:35:02 -0400 X-TMN: [VZstw8jflKN+2k7inYAhz3vMUCjHWbk3] X-Originating-Email: [wanpeng.li@hotmail.com] Message-ID: From: Wanpeng Li To: Ingo Molnar , Peter Zijlstra CC: Juri Lelli , linux-kernel@vger.kernel.org, Wanpeng Li Subject: [PATCH v3 5/5] sched/core: remove superfluous resetting of dl_throttled flag Date: Sun, 31 May 2015 08:27:01 +0800 X-Mailer: git-send-email 1.9.1 In-Reply-To: <1433032021-11127-1-git-send-email-wanpeng.li@hotmail.com> References: <1433032021-11127-1-git-send-email-wanpeng.li@hotmail.com> X-OriginalArrivalTime: 31 May 2015 00:35:00.0926 (UTC) FILETIME=[A0EFDDE0:01D09B39] MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Resetting dl_throttled flag in rt_mutex_setprio (for a task that is going to be boosted) is superfluous, as the natural place to do so is in replenish_dl_entity(). If the task was on the runqueue and it is boosted by a DL task, it will be enqueued back with ENQUEUE_REPLENISH flag set, which can guarantee that dl_throttled is reset in replenish_dl_entity(). This patch drops the resetting of throttled status in function rt_mutex_setprio(). Signed-off-by: Wanpeng Li --- v1 -> v2: * rewrite patch subject and changelog kernel/sched/core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 5140db6..ef2ddd6 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c @@ -3076,7 +3076,6 @@ void rt_mutex_setprio(struct task_struct *p, int prio) if (!dl_prio(p->normal_prio) || (pi_task && dl_entity_preempt(&pi_task->dl, &p->dl))) { p->dl.dl_boosted = 1; - p->dl.dl_throttled = 0; enqueue_flag = ENQUEUE_REPLENISH; } else p->dl.dl_boosted = 0; -- 1.9.1