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=-3.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,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 4F8C8C43331 for ; Fri, 8 Nov 2019 00:53:29 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 1793E2178F for ; Fri, 8 Nov 2019 00:53:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727366AbfKHAx2 (ORCPT ); Thu, 7 Nov 2019 19:53:28 -0500 Received: from cloudserver094114.home.pl ([79.96.170.134]:57212 "EHLO cloudserver094114.home.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726094AbfKHAx2 (ORCPT ); Thu, 7 Nov 2019 19:53:28 -0500 Received: from 79.184.254.83.ipv4.supernova.orange.pl (79.184.254.83) (HELO kreacher.localnet) by serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer 0.83.292) id ce3d5ac9ff617bdc; Fri, 8 Nov 2019 01:53:26 +0100 From: "Rafael J. Wysocki" To: Daniel Lezcano Cc: mathieu.poirier@linaro.org, mingo@redhat.com, peterz@infradead.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, ulf.hansson@linaro.org Subject: Re: [PATCH V6 1/3] cpuidle: play_idle: Make play_idle more flexible Date: Fri, 08 Nov 2019 01:53:25 +0100 Message-ID: <11162801.HHDnLhZHqx@kreacher> In-Reply-To: References: <20191030075141.1039-1-daniel.lezcano@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org On Wednesday, November 6, 2019 7:27:47 PM CET Daniel Lezcano wrote: > > Hi Rafael, > > > On 30/10/2019 08:51, Daniel Lezcano wrote: > > The play_idle function has two users, the intel powerclamp and the > > idle_injection. > > > > The idle injection cooling device uses the function via the > > idle_injection powercap's APIs. Unfortunately, play_idle is currently > > limited by the idle state depth: by default the deepest idle state is > > selected. On the ARM[64] platforms, most of the time it is the cluster > > idle state, the exit latency and the residency can be very high. That > > reduces the scope of the idle injection usage because the impact on > > the performances can be very significant. > > > > If the idle injection cycles can be done with a shallow state like a > > retention state, the cooling effect would eventually give similar > > results than the cpufreq cooling device. > > > > In order to prepare the function to receive an idle state parameter, > > let's replace the 'use_deepest_state' boolean field with 'use_state' > > and use this value to enter the specific idle state. > > > > The current code keeps the default behavior which is go to the deepest > > idle state. > > > > Signed-off-by: Daniel Lezcano > > Acked-by: Mathieu Poirier > > Reviewed-by: Ulf Hansson > > Is it possible to merge this series so I can make some progress on > upstreaming the idle cooling device which depends on these three patches? That would be possible if the series had no problems, but it appears to have some. Let me reply to the patches.