From patchwork Thu Dec 16 22:53:18 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Qais Yousef X-Patchwork-Id: 524839 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E311EC433EF for ; Thu, 16 Dec 2021 22:53:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236535AbhLPWxl (ORCPT ); Thu, 16 Dec 2021 17:53:41 -0500 Received: from foss.arm.com ([217.140.110.172]:49486 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231652AbhLPWxl (ORCPT ); Thu, 16 Dec 2021 17:53:41 -0500 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 6CB3612FC; Thu, 16 Dec 2021 14:53:40 -0800 (PST) Received: from e107158-lin.cambridge.arm.com (unknown [10.1.197.40]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 01B9A3F73B; Thu, 16 Dec 2021 14:53:38 -0800 (PST) From: Qais Yousef To: "Rafael J. Wysocki" , Viresh Kumar , "Peter Zijlstra (Intel)" , Ingo Molnar Cc: Dietmar Eggemann , Vincent Guittot , Beata Michalska , Ionela Voinescu , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Qais Yousef Subject: [PATCH 0/2] uclamp_max vs schedutil fixes Date: Thu, 16 Dec 2021 22:53:18 +0000 Message-Id: <20211216225320.2957053-1-qais.yousef@arm.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org On systems that use sugov_update_single_{freq, perf}(), uclamp_max was ineffective because of 'busy' filter which ignores requests to change frequency if there's no idle time. A condition that is not true if uclamp is being used on this system. Similarly, io heavy tasks that are capped by uclamp_max can still obtain higher frequencies because sugov_iowait_apply() doesn't clamp the boost with uclamp_rq_util_with(). Patches 1 and 2 address these 2 problems. Thanks! Acked-by: Rafael J. Wysocki --- Qais Yousef Qais Yousef (2): sched/sugov: Ignore 'busy' filter when uclamp_is_used() sched/uclamp: Fix iowait boost escaping uclamp restriction kernel/sched/cpufreq_schedutil.c | 11 ++- kernel/sched/sched.h | 139 +++++++++++++++++-------------- 2 files changed, 87 insertions(+), 63 deletions(-)