From patchwork Tue Feb 23 01:22:45 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steve Muckle X-Patchwork-Id: 62646 Delivered-To: patch@linaro.org Received: by 10.112.43.199 with SMTP id y7csp1568270lbl; Mon, 22 Feb 2016 17:23:23 -0800 (PST) X-Received: by 10.98.72.70 with SMTP id v67mr42258460pfa.100.1456190603589; Mon, 22 Feb 2016 17:23:23 -0800 (PST) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id tc5si43245491pab.176.2016.02.22.17.23.22; Mon, 22 Feb 2016 17:23:23 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dkim=pass header.i=@linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932478AbcBWBXT (ORCPT + 30 others); Mon, 22 Feb 2016 20:23:19 -0500 Received: from mail-pa0-f48.google.com ([209.85.220.48]:35324 "EHLO mail-pa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932277AbcBWBXA (ORCPT ); Mon, 22 Feb 2016 20:23:00 -0500 Received: by mail-pa0-f48.google.com with SMTP id ho8so103056326pac.2 for ; Mon, 22 Feb 2016 17:23:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=ni/VM2xC3QmmJI4/7fJh4BVdQeu7gON1EhNPUJ6GSW0=; b=fk4L9dzmE5BuuiUwUPjQOGVIOVCmfF4qeyy5c+1zXdZRDH01OWMaDAPyMkQ57iAVHD NluZ5QLDw4KmLMauNdBUAxBqh8/irXu2vFbYc3qCJNgcxwkP1ZXWc6wFTHfEdyO8IJiz Fbt+Uejx3fta2+tAEk6U/wRUorYpCIeJr0kpc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=ni/VM2xC3QmmJI4/7fJh4BVdQeu7gON1EhNPUJ6GSW0=; b=jzweTBaOsyRCFjgyf4mxvVYGye5eOFz0s2Ml2+BdpmGzG2cNHCiR1yARJAqp0i/okD nKCXwnT4XvJIU1KntIdm4EMLCZI3ZyQCElwNyYe8murV0/x/jsbdLqgikmqmqgsLKHZ3 24JsWHod8zehYidBqYKSbN7JmwJ5gZQ3evh91s1pQ7njVEy8jtGKYXEyYZ/0X7Lt1BYh Im+buxM53ZXV1OnxjV6d6XxORYE7dG6JLsKGIhmGBK0RldifJWElU1eU4qzx9sNMw0Sm scQa9NVuTZhAxuSCzjB6EDAijSgmwaaLCrXCMhRKInwb71meVrBZcIufYeAKrt1NdHK8 YO0Q== X-Gm-Message-State: AG10YOQSHBPWz84v1NaIGA2eulQSnTGwt2tsJf4K1O1q7pKnrq7wgX/TAJMDX8vRTaGBQV3t X-Received: by 10.66.144.4 with SMTP id si4mr42570113pab.0.1456190579970; Mon, 22 Feb 2016 17:22:59 -0800 (PST) Received: from graphite.smuckle.net (cpe-75-80-155-7.san.res.rr.com. [75.80.155.7]) by smtp.gmail.com with ESMTPSA id t29sm39626789pfi.8.2016.02.22.17.22.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 22 Feb 2016 17:22:59 -0800 (PST) From: Steve Muckle X-Google-Original-From: Steve Muckle To: Peter Zijlstra , Ingo Molnar , "Rafael J. Wysocki" Cc: linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, Vincent Guittot , Morten Rasmussen , Dietmar Eggemann , Juri Lelli , Patrick Bellasi , Michael Turquette , Juri Lelli Subject: [RFCv7 PATCH 05/10] sched/{core, fair}: trigger OPP change request on fork() Date: Mon, 22 Feb 2016 17:22:45 -0800 Message-Id: <1456190570-4475-6-git-send-email-smuckle@linaro.org> X-Mailer: git-send-email 2.4.10 In-Reply-To: <1456190570-4475-1-git-send-email-smuckle@linaro.org> References: <1456190570-4475-1-git-send-email-smuckle@linaro.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Juri Lelli Patch "sched/fair: add triggers for OPP change requests" introduced OPP change triggers for enqueue_task_fair(), but the trigger was operating only for wakeups. Fact is that it makes sense to consider wakeup_new also (i.e., fork()), as we don't know anything about a newly created task and thus we most certainly want to jump to max OPP to not harm performance too much. However, it is not currently possible (or at least it wasn't evident to me how to do so :/) to tell new wakeups from other (non wakeup) operations. This patch introduces an additional flag in sched.h that is only set at fork() time and it is then consumed in enqueue_task_fair() for our purpose. cc: Ingo Molnar cc: Peter Zijlstra Signed-off-by: Juri Lelli Signed-off-by: Steve Muckle --- kernel/sched/core.c | 2 +- kernel/sched/fair.c | 9 +++------ kernel/sched/sched.h | 1 + 3 files changed, 5 insertions(+), 7 deletions(-) -- 2.4.10 diff --git a/kernel/sched/core.c b/kernel/sched/core.c index 87ca0be..86297a2 100644 --- a/kernel/sched/core.c +++ b/kernel/sched/core.c @@ -2553,7 +2553,7 @@ void wake_up_new_task(struct task_struct *p) #endif rq = __task_rq_lock(p); - activate_task(rq, p, 0); + activate_task(rq, p, ENQUEUE_WAKEUP_NEW); p->on_rq = TASK_ON_RQ_QUEUED; trace_sched_wakeup_new(p); check_preempt_curr(rq, p, WF_FORK); diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c index f1f00a4..e7fab8f 100644 --- a/kernel/sched/fair.c +++ b/kernel/sched/fair.c @@ -4308,7 +4308,8 @@ enqueue_task_fair(struct rq *rq, struct task_struct *p, int flags) { struct cfs_rq *cfs_rq; struct sched_entity *se = &p->se; - int task_new = !(flags & ENQUEUE_WAKEUP); + int task_new = flags & ENQUEUE_WAKEUP_NEW; + int task_wakeup = flags & ENQUEUE_WAKEUP; for_each_sched_entity(se) { if (se->on_rq) @@ -4349,12 +4350,8 @@ enqueue_task_fair(struct rq *rq, struct task_struct *p, int flags) * because we get here also during load balancing, but * in these cases it seems wise to trigger as single * request after load balancing is done. - * - * XXX: how about fork()? Do we need a special - * flag/something to tell if we are here after a - * fork() (wakeup_task_new)? */ - if (!task_new) + if (task_new || task_wakeup) update_capacity_of(cpu_of(rq)); } hrtick_update(rq); diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h index 17908dd..9c26be2 100644 --- a/kernel/sched/sched.h +++ b/kernel/sched/sched.h @@ -1140,6 +1140,7 @@ extern const u32 sched_prio_to_wmult[40]; #endif #define ENQUEUE_REPLENISH 0x08 #define ENQUEUE_RESTORE 0x10 +#define ENQUEUE_WAKEUP_NEW 0x20 #define DEQUEUE_SLEEP 0x01 #define DEQUEUE_SAVE 0x02