From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id ECE24C433E0 for ; Tue, 7 Jul 2020 11:39:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D4CFC206DF for ; Tue, 7 Jul 2020 11:39:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728246AbgGGLjy (ORCPT ); Tue, 7 Jul 2020 07:39:54 -0400 Received: from foss.arm.com ([217.140.110.172]:42780 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726805AbgGGLjx (ORCPT ); Tue, 7 Jul 2020 07:39:53 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id EFA3E1FB; Tue, 7 Jul 2020 04:39:52 -0700 (PDT) Received: from e113632-lin (e113632-lin.cambridge.arm.com [10.1.194.46]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 878203F71E; Tue, 7 Jul 2020 04:39:50 -0700 (PDT) References: <20200706142839.26629-1-qais.yousef@arm.com> <20200706142839.26629-2-qais.yousef@arm.com> User-agent: mu4e 0.9.17; emacs 26.3 From: Valentin Schneider To: Qais Yousef Cc: Ingo Molnar , Peter Zijlstra , Doug Anderson , Jonathan Corbet , Juri Lelli , Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Luis Chamberlain , Kees Cook , Iurii Zaikin , Quentin Perret , Patrick Bellasi , Pavan Kondeti , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v6 1/2] sched/uclamp: Add a new sysctl to control RT default boost value In-reply-to: <20200706142839.26629-2-qais.yousef@arm.com> Date: Tue, 07 Jul 2020 12:39:48 +0100 Message-ID: MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/07/20 15:28, Qais Yousef wrote: > RT tasks by default run at the highest capacity/performance level. When > uclamp is selected this default behavior is retained by enforcing the > requested uclamp.min (p->uclamp_req[UCLAMP_MIN]) of the RT tasks to be > uclamp_none(UCLAMP_MAX), which is SCHED_CAPACITY_SCALE; the maximum > value. > > This is also referred to as 'the default boost value of RT tasks'. > > See commit 1a00d999971c ("sched/uclamp: Set default clamps for RT tasks"). > > On battery powered devices, it is desired to control this default > (currently hardcoded) behavior at runtime to reduce energy consumed by > RT tasks. > > For example, a mobile device manufacturer where big.LITTLE architecture > is dominant, the performance of the little cores varies across SoCs, and > on high end ones the big cores could be too power hungry. > > Given the diversity of SoCs, the new knob allows manufactures to tune > the best performance/power for RT tasks for the particular hardware they > run on. > > They could opt to further tune the value when the user selects > a different power saving mode or when the device is actively charging. > > The runtime aspect of it further helps in creating a single kernel image > that can be run on multiple devices that require different tuning. > > Keep in mind that a lot of RT tasks in the system are created by the > kernel. On Android for instance I can see over 50 RT tasks, only > a handful of which created by the Android framework. > > To control the default behavior globally by system admins and device > integrator, introduce the new sysctl_sched_uclamp_util_min_rt_default > to change the default boost value of the RT tasks. > > I anticipate this to be mostly in the form of modifying the init script > of a particular device. > Sorry for going at this again, but I feel like I can squeeze more juice out of this. This being mainly tweaked in init scripts makes me question why we should harden this for runtime tweaking. Yes, there's the whole single image thing, but there's other ways to have different init-time values on a single image (e.g. cmdline, although that one is usually a bit controversial). For instance, Android could set the min to 0 and then go about its life tweaking the clamp of individual / all RT tasks at runtime, using the existing uclamp API.