From patchwork Tue Oct 16 10:14:58 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Quentin Perret X-Patchwork-Id: 10643315 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 366DC1057 for ; Tue, 16 Oct 2018 10:15:27 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 281712856D for ; Tue, 16 Oct 2018 10:15:27 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 1AFA329212; Tue, 16 Oct 2018 10:15:27 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 28E4F2920A for ; Tue, 16 Oct 2018 10:15:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726553AbeJPSFH (ORCPT ); Tue, 16 Oct 2018 14:05:07 -0400 Received: from foss.arm.com ([217.140.101.70]:33678 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726165AbeJPSFH (ORCPT ); Tue, 16 Oct 2018 14:05:07 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 47E90341; Tue, 16 Oct 2018 03:15:24 -0700 (PDT) Received: from queper01-lin.local (unknown [10.1.195.48]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id E28B03F59C; Tue, 16 Oct 2018 03:15:19 -0700 (PDT) From: Quentin Perret To: peterz@infradead.org, rjw@rjwysocki.net, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Cc: gregkh@linuxfoundation.org, mingo@redhat.com, dietmar.eggemann@arm.com, morten.rasmussen@arm.com, chris.redpath@arm.com, patrick.bellasi@arm.com, valentin.schneider@arm.com, vincent.guittot@linaro.org, thara.gopinath@linaro.org, viresh.kumar@linaro.org, tkjos@google.com, joel@joelfernandes.org, smuckle@google.com, adharmap@codeaurora.org, skannan@codeaurora.org, pkondeti@codeaurora.org, juri.lelli@redhat.com, edubezval@gmail.com, srinivas.pandruvada@linux.intel.com, currojerez@riseup.net, javi.merino@kernel.org, quentin.perret@arm.com Subject: [PATCH v8 00/15] Energy Aware Scheduling Date: Tue, 16 Oct 2018 11:14:58 +0100 Message-Id: <20181016101513.26919-1-quentin.perret@arm.com> X-Mailer: git-send-email 2.19.1 MIME-Version: 1.0 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP This patch series introduces Energy Aware Scheduling (EAS) for CFS tasks on platforms with asymmetric CPU topologies (e.g. Arm big.LITTLE). For more details about the ideas behind it and the overall design, please refer to the cover letter of version 5 [1]. 1. Version History ------------------ Changes v7[2]->v8: - Added kerneldoc to enum schedutil_type (patch 02) - Added 'max' argument to schedutil_freq_util() (patch 02) - Added schedutil_energy_util() wrapper (patch 02) - Added smp_store_release() to the EM loading code (patch 03) - Renamed 'obj' field of struct perf_domain to 'em_pd' (patch 05) - Added plain WARN when the EM is too large for EAS (patch 07) - Added dmesg warning when EAS gets disabled by switching from sugov (patch 08) - Replaced sched_feat(ENERGY_AWARE) by a sysctl + static key (patches 09 and 10) - Improved/refactored find_energy_efficient_cpu() and compute_energy() for readability (patches 13 and 14) Changes v6[3]->v7: - Replaced the sched_energy_present static key by a sched_feat - Replaced the CPUFreq notifier in the dependency between sugov and EAS by a function call - Squashed all sugov-refactoring patches into patch 02 - Clarified comment in em_fd_energy() to explain the choice of “energy” over “power” - Added kerneldoc to structs in include/linux/energy_model.h - Removed unnecessary memory barrier from the EM framework - Fixed corner case in find_energy_efficient_cpu when prev_cpu is overutilized (and prev_energy = ULONG_MAX) Changes v5[1]->v6: - Rebased on Peter’s sched/core branch (that includes Morten's misfit patches [4] and the automatic detection of SD_ASYM_CPUCAPACITY [5]) - Removed patch 13/14 (not needed with the automatic flag detection) - Added patch creating a dependency between sugov and EAS - Renamed frequency domains to performance domains to avoid creating too deep assumptions in the code about the HW - Renamed the sd_ea shortcut sd_asym_cpucapacity - Added comment to explain why new tasks are not accounted when detecting the 'overutilized' flag - Added comment explaining why forkees don’t go in find_energy_efficient_cpu() Changes v4[6]->v5: - Removed the RCU protection of the EM tables and the associated need for em_rescale_cpu_capacity(). - Factorized schedutil’s PELT aggregation function with EAS - Improved comments/doc in the EM framework - Added check on the uarch of CPUs in one fd in the EM framework - Reduced CONFIG_ENERGY_MODEL ifdefery in kernel/sched/topology.c - Cleaned-up update_sg_lb_stats parameters - Improved comments in compute_energy() to explain the multi-rd scenarios Changes v3[7]->v4: - Replaced spinlock in EM framework by smp_store_release/READ_ONCE - Fixed missing locks to protect rcu_assign_pointer in EM framework - Fixed capacity calculation in EM framework on 32 bits system - Fixed compilation issue for CONFIG_ENERGY_MODEL=n - Removed cpumask from struct em_freq_domain, now dynamically allocated - Power costs of the EM are specified in milliwatts - Added example of CPUFreq driver modification - Added doc/comments in the EM framework and better commit header - Fixed integration issue with util_est in cpu_util_next() - Changed scheduler topology code to have one freq. dom. list per rd - Split sched topology patch in smaller patches - Added doc/comments explaining the heuristic in the wake-up path - Changed energy threshold for migration to from 1.5% to 6% Changes v2[8]->v3: - Removed the PM_OPP dependency by implementing a new EM framework - Modified the scheduler topology code to take references on the EM data structures - Simplified the overutilization mechanism into a system-wide flag - Reworked the integration in the wake-up path using the sd_ea shortcut - Rebased on tip/sched/core (247f2f6f3c70 "sched/core: Don't schedule threads on pre-empted vCPUs") Changes v1[9]->v2: - Reworked interface between fair.c and energy.[ch] (Remove #ifdef CONFIG_PM_OPP from energy.c) (Greg KH) - Fixed licence & header issue in energy.[ch] (Greg KH) - Reordered EAS path in select_task_rq_fair() (Joel) - Avoid prev_cpu if not allowed in select_task_rq_fair() (Morten/Joel) - Refactored compute_energy() (Patrick) - Account for RT/IRQ pressure in task_fits() (Patrick) - Use UTIL_EST and DL utilization during OPP estimation (Patrick/Juri) - Optimize selection of CPU candidates in the energy-aware wake-up path - Rebased on top of tip/sched/core (commit b720342849fe “sched/core: Update Preempt_notifier_key to modern API”) 2. Test results --------------- Two fundamentally different tests were executed. Firstly the energy test case shows the impact on energy consumption this patch-set has using a synthetic set of tasks. Secondly the performance test case provides the conventional hackbench metric numbers. The tests run on two arm64 big.LITTLE platforms: Hikey960 (4xA73 + 4xA53) and Juno r0 (2xA57 + 4xA53). Base kernel is 4.19-rc5, with some Hikey960 and Juno specific patches, and Morten's patches from tip/sched/core for detecting the SD_ASYM_CPUCAPACITY flag. Test branch: [10]. 2.1 Energy test case 10 iterations of between 10 and 50 periodic rt-app tasks (16ms period, 5% duty-cycle) for 30 seconds with energy measurement. Unit is Joules. The goal is to save energy, so lower is better. 2.1.1 Hikey960 Energy is measured with an ACME Cape on an instrumented board. Numbers include consumption of big and little CPUs, LPDDR memory, GPU and most of the other small components on the board. They do not include consumption of the radio chip (turned-off anyway) and external connectors. +----------+-----------------+-------------------------+ |          | Without patches | With patches            | +----------+--------+--------+------------------+------+ | Tasks nb |  Mean | RSD* | Mean             | RSD* | +----------+--------+--------+------------------+------+ |       10 | 34.35 |   4.4% | 29.37 (-14.50%) | 1.6% | |       20 | 52.07 |   2.2% | 43.08 (-17.27%) | 1.8% | |       30 | 66.24 |   5.2% | 58.69 (-11.40%) | 4.0% | |       40 | 90.25 |   4.3% | 82.95 (-8.09%)  | 5.9% | |       50 | 135.49 |  10.2% | 105.99 (-21.77%) | 1.0% | +----------+-----------------+-------------------------+ 2.1.2 Juno r0 Energy is measured with the onboard energy meter. Numbers include consumption of big and little CPUs. +----------+-----------------+-------------------------+ |          | Without patches | With patches            | +----------+--------+--------+------------------+------+ | Tasks nb |  Mean | RSD* | Mean             | RSD* | +----------+--------+--------+------------------+------+ |       10 | 9.41 |   2.4% | 6.87 (-26.99%) | 2.0% | |       20 | 17.54 |   2.7% | 14.08 (-19.73%) | 4.0% | |       30 | 30.34 |   3.7% | 22.00 (-27.49%) | 4.1% | |       40 | 42.06 |   3.2% | 32.13 (-23.61%) | 5.1% | |       50 | 53.62 |   0.8% | 51.24 (-4.44%)  | 2.5% | +----------+-----------------+-------------------------+ 2.2 Performance test case 30 iterations of perf bench sched messaging --pipe --thread --group G --loop L with G=[1 2 4 8] and L=50000 (Hikey960)/16000 (Juno r0). 2.2.1 Hikey960 The impact of thermal capping was mitigated thanks to a heatsink, a fan, and a 30 sec delay between two successive executions. IPA is disabled to reduce the stddev. +----------------+-----------------+------------------------+ |                | Without patches | With patches           | +--------+-------+---------+-------+----------------+-------+ | Groups | Tasks | Mean    | RSD* | Mean | RSD*  | +--------+-------+---------+-------+----------------+-------+ |      1 | 40 |    8.29 | 0.83% | 8.46 (+2.05%) | 0.75% | |      2 | 80 |   15.27 | 0.54% | 15.43 (+0.73%) | 0.61% | |      4 | 160 |   31.75 | 0.41% | 32.04 (+0.91%) | 0.49% | |      8 | 320 |   66.92 | 0.54% | 66.98 (+0.09%) | 0.43% | +--------+-------+---------+-------+----------------+-------+ 2.2.2 Juno r0 +----------------+-----------------+------------------------+ |                | Without patches | With patches           | +--------+-------+---------+-------+----------------+-------+ | Groups | Tasks | Mean    | RSD* | Mean | RSD*  | +--------+-------+---------+-------+----------------+-------+ |      1 | 40 |    8.29 | 1.21% | 8.34 (+0.60%) | 0.95% | |      2 | 80 |   15.46 | 0.95% | 15.45 (+0.00%) | 0.94% | |      4 | 160 |   29.48 | 0.87% | 29.47 (+0.00%) | 0.65% | |      8 | 320 |   59.30 | 0.63% | 59.59 (+0.49%) | 0.85% | +--------+-------+---------+-------+----------------+-------+ *RSD: Relative Standard Deviation (std dev / mean) [1] https://marc.info/?l=linux-kernel&m=153243513908731&w=2 (V5) [2] https://marc.info/?l=linux-kernel&m=153674360525432&w=2 (V7) [3] https://marc.info/?l=linux-kernel&m=153476300928169&w=2 (V6) [4] https://marc.info/?l=linux-kernel&m=153069968022982&w=2 (Misfit) [5] https://marc.info/?l=linux-kernel&m=153209362826476&w=2 (SD_ASYM_CPUCAPACITY) [6] https://marc.info/?l=linux-kernel&m=153018606728533&w=2 (V4) [7] https://marc.info/?l=linux-kernel&m=152691273111941&w=2 (V3) [8] https://marc.info/?l=linux-kernel&m=152302902427143&w=2 (V2) [9] https://marc.info/?l=linux-kernel&m=152153905805048&w=2 (V1) [10] http://www.linux-arm.org/git?p=linux-qp.git;a=shortlog;h=refs/heads/upstream/eas_v8 Morten Rasmussen (1): sched: Add over-utilization/tipping point indicator Quentin Perret (14): sched: Relocate arch_scale_cpu_capacity sched/cpufreq: Prepare schedutil for Energy Aware Scheduling PM: Introduce an Energy Model management framework PM / EM: Expose the Energy Model in sysfs sched/topology: Reference the Energy Model of CPUs when available sched/topology: Lowest CPU asymmetry sched_domain level pointer sched/topology: Disable EAS on inappropriate platforms sched/topology: Make Energy Aware Scheduling depend on schedutil sched: Introduce sched_energy_present static key sched: Introduce a sysctl for Energy Aware Scheduling sched/fair: Clean-up update_sg_lb_stats parameters sched/fair: Introduce an energy estimation helper function sched/fair: Select an energy-efficient CPU on task wake-up OPTIONAL: cpufreq: dt: Register an Energy Model drivers/cpufreq/cpufreq-dt.c | 48 ++++- drivers/cpufreq/cpufreq.c | 2 + include/linux/energy_model.h | 189 +++++++++++++++++++ include/linux/sched/cpufreq.h | 15 ++ include/linux/sched/sysctl.h | 7 + include/linux/sched/topology.h | 19 ++ kernel/power/Kconfig | 15 ++ kernel/power/Makefile | 2 + kernel/power/energy_model.c | 291 +++++++++++++++++++++++++++++ kernel/sched/cpufreq_schedutil.c | 129 +++++++++---- kernel/sched/fair.c | 305 +++++++++++++++++++++++++++++-- kernel/sched/sched.h | 84 ++++++--- kernel/sched/topology.c | 269 ++++++++++++++++++++++++++- kernel/sysctl.c | 11 ++ 14 files changed, 1304 insertions(+), 82 deletions(-) create mode 100644 include/linux/energy_model.h create mode 100644 kernel/power/energy_model.c