Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp4227675pxv; Tue, 29 Jun 2021 01:55:56 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzpgjVUF87jHZx+MY89ngH6b34iKoxbquTWh1urYi1vY22ogq2bUjcgI9E/+IR8yojA55xM X-Received: by 2002:a17:906:538c:: with SMTP id g12mr2243619ejo.245.1624956956075; Tue, 29 Jun 2021 01:55:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1624956956; cv=none; d=google.com; s=arc-20160816; b=e/pUNWrt4s2uf5rt6axrXDin+CtQWdSK++BMPi2yQ2DsOKnYZKy58X8NhROBtU1JSy lwS+3uUcai0xspCcsR+S+/rXK2TRNZlBRUbg4Llr0ifnNGqN4ri2Zu/kn0uafclU1aD9 cJiHXsM6jNxjRC0usK1rTxrHjOEctAP9Tx6mUag/6cB9fnwppsurEuBGS+n3xTMcuR2J 0cZ+QdYPWIXhaFMkAtQKnKfWXpSfKgNqjW4s8HH4ZvaaKkdVcPJI9WKiMbxaW87xswel kaAYLE9oXIcUmgWiAiixOTjGpDrPs+/4xWf6Ni+mwB8gOTt0uQQe6Ew6fOt2ktmvfl2Y YT0Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=Y/8aWcR8pQLRr1AneyecLvILPcZP/SywuwqtFiKRs6o=; b=P8b+fIBH/bDttqN3skxPjpT/Cba8i7KEjbxrBfvw1ANn+bWBMQqFbhkJffzB7ykonz cDvgbgEUUqs5fgIrRfTPQeqRfuq2tS+a6C1E8pnB13PTPc4qp2GM15on92R3NlMcS+FT W2Y+1FfGdHv7cwVFOdLvR9vubJFYhzSIGtHqwtVRoSLuX8ntu8d5ZQCD4TftPtw9O6BI ivN1k+pNCxLpJdb9QsDxzKN+ZveiHu/C40vknBe+J5A3BoRtybXDm0P3702EYjMpOD0p t5IyCDE65+xZ6bVeI4YcY9aevIUgjamikKiZIAy/U6HRebVWo30BPJD158N+TDzyIo6S hrnQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=UERVdQrp; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q19si17011115edv.125.2021.06.29.01.55.33; Tue, 29 Jun 2021 01:55:56 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=UERVdQrp; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232662AbhF2Izp (ORCPT + 99 others); Tue, 29 Jun 2021 04:55:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55222 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232653AbhF2Izo (ORCPT ); Tue, 29 Jun 2021 04:55:44 -0400 Received: from mail-pj1-x1030.google.com (mail-pj1-x1030.google.com [IPv6:2607:f8b0:4864:20::1030]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 55843C061574 for ; Tue, 29 Jun 2021 01:53:15 -0700 (PDT) Received: by mail-pj1-x1030.google.com with SMTP id g6-20020a17090adac6b029015d1a9a6f1aso1545653pjx.1 for ; Tue, 29 Jun 2021 01:53:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=Y/8aWcR8pQLRr1AneyecLvILPcZP/SywuwqtFiKRs6o=; b=UERVdQrpA6QxI7RQYA+eUhOfbgXgeUFYb4pvURcjwQKvNlRQqHAOYqkWee5zs+Xjb0 Ah3t28sJQF+zhkVQHRqUV46YTiZXGBh15MtEwUIevK9C7jHP7Fn7YYpts7mxMjBYXvfV 47iOj3Tl6L2SOmEYsEKhu1xRsUleykMhGR2fkj71nvNEvo29dZejYi93pRIysLFg/Jyj JIwu/CkRn40298AVPZw0CFNk+2U+cYskRpjwI/yVZ8wTDOqXyUmLMtHYRw69DrGY1b7o BrSUColfv2WDuJLrTcLbkUPwpk6iOu0/QE8q5I/rcsxj598vsuZMvfGGMrfhnqAJGbzj noXw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=Y/8aWcR8pQLRr1AneyecLvILPcZP/SywuwqtFiKRs6o=; b=H93aH+HO6wGmrNjYT13qrPP2f3SHKcy7lER0bBAQBP5B6fdvgFMen8jOl1899vTBUf TOhAoZnfGOkgntA8bgekKaI8nCyARR7GOSyFS1PgR7YG/GiMr9S6kTchkaT9BoCwhltN GGRA/DL1preqXxGAfJHupbbkF3ruTcoIxkQtM55XHmhj9T9kGyBObzvD5ebOmACjWsuf +1/wb72URTcmYWQuuhLXcWgWeEEhIeRwtkWDuvVFVG03wwzpGbZuVFHPFPVfjm+Y04Jl m6PJphEQf7/04Ft+KUOiW8wJ0LDmffIz4DS9HCdVTCy3rbBvk7xeeOHlBLucWtc6aePx 2JLQ== X-Gm-Message-State: AOAM533FLRR8ISYwrl5G/4bQW9fzkGKqmER3NRGIAZxuMcb2GU6kBLcg ZmPeyhFTs/3Ycwi46MHsVNmlvA== X-Received: by 2002:a17:90a:5907:: with SMTP id k7mr33269795pji.196.1624956794956; Tue, 29 Jun 2021 01:53:14 -0700 (PDT) Received: from localhost ([136.185.134.182]) by smtp.gmail.com with ESMTPSA id i188sm6191585pfe.30.2021.06.29.01.53.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 29 Jun 2021 01:53:14 -0700 (PDT) Date: Tue, 29 Jun 2021 14:23:12 +0530 From: Viresh Kumar To: Ionela Voinescu Cc: Rafael Wysocki , Sudeep Holla , Ingo Molnar , Peter Zijlstra , Juri Lelli , Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Daniel Bristot de Oliveira , linux-pm@vger.kernel.org, Qian Cai , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH V3 4/4] cpufreq: CPPC: Add support for frequency invariance Message-ID: <20210629085312.j6f4e4b6vuezsqvm@vireshk-i7> References: <20210624094812.GA6095@arm.com> <20210624130418.poiy4ph66mbv3y67@vireshk-i7> <20210625085454.GA15540@arm.com> <20210625165418.shi3gkebumqllxma@vireshk-i7> <20210628104929.GA29595@arm.com> <20210629043244.xkjat5dqqjaixkii@vireshk-i7> <20210629084737.GB2425@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210629084737.GB2425@arm.com> User-Agent: NeoMutt/20180716-391-311a52 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 29-06-21, 09:47, Ionela Voinescu wrote: > Okay, makes sense. I have not seen this code actually do anything wrong > so far, and the issues I see on ThunderX2 point more to misbehaving > counters for this purpose. This being said, I would have probably > preferred for this feature to be disabled by default, until we've tested > more, but that won't give the chance to anyone else to test. > > Thanks! > > Reviewed-by: Ionela Voinescu Thanks for understanding Ionela. -- viresh