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=-12.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham 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 2BBFFC4332B for ; Thu, 28 Jan 2021 04:16:00 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E307464DD6 for ; Thu, 28 Jan 2021 04:15:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231172AbhA1EP5 (ORCPT ); Wed, 27 Jan 2021 23:15:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52514 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231332AbhA1EPR (ORCPT ); Wed, 27 Jan 2021 23:15:17 -0500 Received: from mail-pf1-x42c.google.com (mail-pf1-x42c.google.com [IPv6:2607:f8b0:4864:20::42c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 716E5C061573 for ; Wed, 27 Jan 2021 20:14:37 -0800 (PST) Received: by mail-pf1-x42c.google.com with SMTP id 11so3168836pfu.4 for ; Wed, 27 Jan 2021 20:14:37 -0800 (PST) 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:content-transfer-encoding:in-reply-to :user-agent; bh=lCXx50lgTRJjoLzvkZ4CSd/Xgq+HdduyctrNeK/ybOA=; b=L2zVgS72yknF/uJKUQTIxQpzR/YqSxlccL48rWBWjWBOow9nT7HL0FSBy3LN3SjdyK P32Nbd86UaYWAhSlCnaM/nfV3+dFAdxO5/7OtpSeir/skN3NAhYBlaTBM09Va7zSX0xI Q8L5M11D2DtQsFRf8i2TAQc+5LfhXxxpUvT3lXemfVPdXYFKlqxzbHiJ6BbRYjSJBlsW 4mGG7fNTjbcHntcuSwsVqsNT7ViisPkmUMiJUL9zQ3n/RGMPK4qP/5WG8DY+olSyVZVh M2CjVCaa+oo4GJMMpzn3dyODmpn/JGzl9lsm9Qxfmdie1HnJgZcKP2vT2/rGSsx/GWDz Nsng== 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:content-transfer-encoding :in-reply-to:user-agent; bh=lCXx50lgTRJjoLzvkZ4CSd/Xgq+HdduyctrNeK/ybOA=; b=OIwOfnDN3xD0stV/U9F9qVIf1EbB8a9T6+/3MBQkynCJmAX/SGWAxky/de2hCaBEeB spCd3BFlOZe4ewa9X2sfWd0XDgOjEhBTOGvJYA9swODmpC/YnUefWrTn6ZEbhwbU7xAz SYEP5MrVyE1HoDL4hcusWUL+TkX8oGvyDcHd9RAXUjn4S2kxX4MbGnIlgRCAbJkPQZ/3 XI0WYYcDcEAmiZZEny2/5JBlrT6tPtF8Nwy8g6mEg2jc2RSyLDHQfIdpBsGyWzysP70e 7OT2Oh8PxDNjGe4mpmaaLESxKaVFfiFwNWrcyhfXs1H6si5rwKpy5OrcCkrqS/pUrbYD +vGA== X-Gm-Message-State: AOAM533pPvE5TvGm3tADpSWnDScI+rHO9sugKZVY5KqlKQHYJaS4wCYr QbTLkhsxW8R+ieV9TJ+baprf8A== X-Google-Smtp-Source: ABdhPJywuWr5tT7F1kzEfWsEJJfA+0MY7q2KsZKqDelepQKDtrXtxMVcyU2H9WnPXcXc88aXjcUr0A== X-Received: by 2002:a65:458d:: with SMTP id o13mr14441303pgq.450.1611807277019; Wed, 27 Jan 2021 20:14:37 -0800 (PST) Received: from localhost ([122.172.59.240]) by smtp.gmail.com with ESMTPSA id t25sm3995172pgv.30.2021.01.27.20.14.35 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 27 Jan 2021 20:14:35 -0800 (PST) Date: Thu, 28 Jan 2021 09:44:31 +0530 From: Viresh Kumar To: Akhil P Oommen Cc: Dmitry Osipenko , Viresh Kumar , Nishanth Menon , Stephen Boyd , linux-pm@vger.kernel.org, Vincent Guittot , Rafael Wysocki , Sibi Sankar , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 03/13] opp: Keep track of currently programmed OPP Message-ID: <20210128041431.rnfp3yrh7mp7e2gb@vireshk-i7> References: <96b57316a2a307a5cc5ff7302b3cd0084123a2ed.1611227342.git.viresh.kumar@linaro.org> <20210122044532.pc7cpcgy3kjbqmls@vireshk-i7> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20180716-391-311a52 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 27-01-21, 22:01, Akhil P Oommen wrote: > On 1/22/2021 10:15 AM, Viresh Kumar wrote: > > On 22-01-21, 00:41, Dmitry Osipenko wrote: > > > 21.01.2021 14:17, Viresh Kumar пишет: > > > > @@ -1074,15 +1091,18 @@ int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq) > > > > if (!ret) { > > > > ret = _set_opp_bw(opp_table, opp, dev, false); > > > > - if (!ret) > > > > + if (!ret) { > > > > opp_table->enabled = true; > > > > + dev_pm_opp_put(old_opp); > > > > + > > > > + /* Make sure current_opp doesn't get freed */ > > > > + dev_pm_opp_get(opp); > > > > + opp_table->current_opp = opp; > > > > + } > > > > } > > > > > > I'm a bit surprised that _set_opp_bw() isn't used similarly to > > > _set_opp_voltage() in _generic_set_opp_regulator(). > > > > > > I'd expect the BW requirement to be raised before the clock rate goes UP. > > > > I remember discussing that earlier when this stuff came in, and this I > > believe is the reason for that. > > > > We need to scale regulators before/after frequency because when we > > increase the frequency a regulator may _not_ be providing enough power > > to sustain that (even for a short while) and this may have undesired > > effects on the hardware and so it is important to prevent that > > malfunction. > > > > In case of bandwidth such issues will not happen (AFAIK) and doing it > > just once is normally enough. It is just about allowing more data to > > be transmitted, and won't make the hardware behave badly. > > > I agree with Dmitry. BW is a shared resource in a lot of architectures. > Raising clk before increasing the bw can lead to a scenario where this > client saturate the entire BW for whatever small duration it may be. This > will impact the latency requirements of other clients. I see. I will make the necessary changes then to fix it. Thanks guys. -- viresh 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=-10.6 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 5DCD1C433DB for ; Thu, 28 Jan 2021 04:15:59 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id F3B3D64DD1 for ; Thu, 28 Jan 2021 04:15:58 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org F3B3D64DD1 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=HZ7pGaSSPIUa+o7sSSrKOwzW7tMzhlc6rlveq/ivNPk=; b=CqOA8k72E7zlMsVzXGwZIOxX6 pBEjvSnsZFbx5YamFOCK8GN5UdCJRxNaLFjd+7c+d6J56b+b1swmsDhDwRlXPWgeZUJ6kM/rPKz9+ CUTdzPA18sVIjE1NkrM98ufyKfc1DzXFe0FdJxQFppi9XMFvNGii9WVAHhIuG25hbwTzJv04OvZqe KcYGfaKM+zT27syxh+yXejJzlhMVa7XK+q8XI9RbY7sNeNZLsz+00j58pceWCQVE99SiRYF1qqBc4 Y6NySJbgLa4LY//HP3ewMY6CnarGbu6ZVnuQ2ihapxNdzJXyZGlmuD1Feroek0aaJAg+PMpihuJ52 Vd1UE9Ycg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l4yhg-00084O-Da; Thu, 28 Jan 2021 04:14:44 +0000 Received: from mail-pg1-x529.google.com ([2607:f8b0:4864:20::529]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1l4yhc-00083m-W0 for linux-arm-kernel@lists.infradead.org; Thu, 28 Jan 2021 04:14:42 +0000 Received: by mail-pg1-x529.google.com with SMTP id g15so3429868pgu.9 for ; Wed, 27 Jan 2021 20:14:38 -0800 (PST) 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:content-transfer-encoding:in-reply-to :user-agent; bh=lCXx50lgTRJjoLzvkZ4CSd/Xgq+HdduyctrNeK/ybOA=; b=L2zVgS72yknF/uJKUQTIxQpzR/YqSxlccL48rWBWjWBOow9nT7HL0FSBy3LN3SjdyK P32Nbd86UaYWAhSlCnaM/nfV3+dFAdxO5/7OtpSeir/skN3NAhYBlaTBM09Va7zSX0xI Q8L5M11D2DtQsFRf8i2TAQc+5LfhXxxpUvT3lXemfVPdXYFKlqxzbHiJ6BbRYjSJBlsW 4mGG7fNTjbcHntcuSwsVqsNT7ViisPkmUMiJUL9zQ3n/RGMPK4qP/5WG8DY+olSyVZVh M2CjVCaa+oo4GJMMpzn3dyODmpn/JGzl9lsm9Qxfmdie1HnJgZcKP2vT2/rGSsx/GWDz Nsng== 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:content-transfer-encoding :in-reply-to:user-agent; bh=lCXx50lgTRJjoLzvkZ4CSd/Xgq+HdduyctrNeK/ybOA=; b=XoIXC4Vo3tY0GmrvozyZDQhz1GUBDiBbOO6JM7MIIIfYryQ3WF4XhEBuX52rOJOtA+ 6m0pztOKE38M1/NUuqDKvDGU+I41bnDi1JRXUU5PUyHIYxOyXik2RATPIGmqSXrIm8Oa lZtzrBzX03JO7EHbjNtYpKrwX2c+FZ8GxLEn4sCzhzfY9S3U7X2I2GoAlyTclZ02izAV lYlE3X2XJ/JhYAn+gbgwyQnCAC8Bk7LKsOoP0igKWP6G4nRUDbrs/B6+8w1IBHxpwqCp 9JXwsXLjWcUB0dLnkljCS4Yon6yFNSecbMdxdH3f/hm5ApmzKVE0DE1+McG1JT+hGXJH HSuA== X-Gm-Message-State: AOAM531G3FPDdMxZcTwhAb7lxRMPjoVZzUEpcdkRA5dl/ZLWkKYLBSUh Q84dFxgu92Ms9tNFY6/v9YgnPw== X-Google-Smtp-Source: ABdhPJywuWr5tT7F1kzEfWsEJJfA+0MY7q2KsZKqDelepQKDtrXtxMVcyU2H9WnPXcXc88aXjcUr0A== X-Received: by 2002:a65:458d:: with SMTP id o13mr14441303pgq.450.1611807277019; Wed, 27 Jan 2021 20:14:37 -0800 (PST) Received: from localhost ([122.172.59.240]) by smtp.gmail.com with ESMTPSA id t25sm3995172pgv.30.2021.01.27.20.14.35 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 27 Jan 2021 20:14:35 -0800 (PST) Date: Thu, 28 Jan 2021 09:44:31 +0530 From: Viresh Kumar To: Akhil P Oommen Subject: Re: [PATCH 03/13] opp: Keep track of currently programmed OPP Message-ID: <20210128041431.rnfp3yrh7mp7e2gb@vireshk-i7> References: <96b57316a2a307a5cc5ff7302b3cd0084123a2ed.1611227342.git.viresh.kumar@linaro.org> <20210122044532.pc7cpcgy3kjbqmls@vireshk-i7> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716-391-311a52 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210127_231441_334085_82AF7ADA X-CRM114-Status: GOOD ( 22.73 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nishanth Menon , Vincent Guittot , linux-pm@vger.kernel.org, Stephen Boyd , Viresh Kumar , Rafael Wysocki , linux-kernel@vger.kernel.org, Sibi Sankar , Dmitry Osipenko , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gMjctMDEtMjEsIDIyOjAxLCBBa2hpbCBQIE9vbW1lbiB3cm90ZToKPiBPbiAxLzIyLzIwMjEg MTA6MTUgQU0sIFZpcmVzaCBLdW1hciB3cm90ZToKPiA+IE9uIDIyLTAxLTIxLCAwMDo0MSwgRG1p dHJ5IE9zaXBlbmtvIHdyb3RlOgo+ID4gPiAyMS4wMS4yMDIxIDE0OjE3LCBWaXJlc2ggS3VtYXIg 0L/QuNGI0LXRgjoKPiA+ID4gPiBAQCAtMTA3NCwxNSArMTA5MSwxOCBAQCBpbnQgZGV2X3BtX29w cF9zZXRfcmF0ZShzdHJ1Y3QgZGV2aWNlICpkZXYsIHVuc2lnbmVkIGxvbmcgdGFyZ2V0X2ZyZXEp Cj4gPiA+ID4gICAJaWYgKCFyZXQpIHsKPiA+ID4gPiAgIAkJcmV0ID0gX3NldF9vcHBfYncob3Bw X3RhYmxlLCBvcHAsIGRldiwgZmFsc2UpOwo+ID4gPiA+IC0JCWlmICghcmV0KQo+ID4gPiA+ICsJ CWlmICghcmV0KSB7Cj4gPiA+ID4gICAJCQlvcHBfdGFibGUtPmVuYWJsZWQgPSB0cnVlOwo+ID4g PiA+ICsJCQlkZXZfcG1fb3BwX3B1dChvbGRfb3BwKTsKPiA+ID4gPiArCj4gPiA+ID4gKwkJCS8q IE1ha2Ugc3VyZSBjdXJyZW50X29wcCBkb2Vzbid0IGdldCBmcmVlZCAqLwo+ID4gPiA+ICsJCQlk ZXZfcG1fb3BwX2dldChvcHApOwo+ID4gPiA+ICsJCQlvcHBfdGFibGUtPmN1cnJlbnRfb3BwID0g b3BwOwo+ID4gPiA+ICsJCX0KPiA+ID4gPiAgIAl9Cj4gPiA+IAo+ID4gPiBJJ20gYSBiaXQgc3Vy cHJpc2VkIHRoYXQgX3NldF9vcHBfYncoKSBpc24ndCB1c2VkIHNpbWlsYXJseSB0bwo+ID4gPiBf c2V0X29wcF92b2x0YWdlKCkgaW4gX2dlbmVyaWNfc2V0X29wcF9yZWd1bGF0b3IoKS4KPiA+ID4g Cj4gPiA+IEknZCBleHBlY3QgdGhlIEJXIHJlcXVpcmVtZW50IHRvIGJlIHJhaXNlZCBiZWZvcmUg dGhlIGNsb2NrIHJhdGUgZ29lcyBVUC4KPiA+IAo+ID4gSSByZW1lbWJlciBkaXNjdXNzaW5nIHRo YXQgZWFybGllciB3aGVuIHRoaXMgc3R1ZmYgY2FtZSBpbiwgYW5kIHRoaXMgSQo+ID4gYmVsaWV2 ZSBpcyB0aGUgcmVhc29uIGZvciB0aGF0Lgo+ID4gCj4gPiBXZSBuZWVkIHRvIHNjYWxlIHJlZ3Vs YXRvcnMgYmVmb3JlL2FmdGVyIGZyZXF1ZW5jeSBiZWNhdXNlIHdoZW4gd2UKPiA+IGluY3JlYXNl IHRoZSBmcmVxdWVuY3kgYSByZWd1bGF0b3IgbWF5IF9ub3RfIGJlIHByb3ZpZGluZyBlbm91Z2gg cG93ZXIKPiA+IHRvIHN1c3RhaW4gdGhhdCAoZXZlbiBmb3IgYSBzaG9ydCB3aGlsZSkgYW5kIHRo aXMgbWF5IGhhdmUgdW5kZXNpcmVkCj4gPiBlZmZlY3RzIG9uIHRoZSBoYXJkd2FyZSBhbmQgc28g aXQgaXMgaW1wb3J0YW50IHRvIHByZXZlbnQgdGhhdAo+ID4gbWFsZnVuY3Rpb24uCj4gPiAKPiA+ IEluIGNhc2Ugb2YgYmFuZHdpZHRoIHN1Y2ggaXNzdWVzIHdpbGwgbm90IGhhcHBlbiAoQUZBSUsp IGFuZCBkb2luZyBpdAo+ID4ganVzdCBvbmNlIGlzIG5vcm1hbGx5IGVub3VnaC4gSXQgaXMganVz dCBhYm91dCBhbGxvd2luZyBtb3JlIGRhdGEgdG8KPiA+IGJlIHRyYW5zbWl0dGVkLCBhbmQgd29u J3QgbWFrZSB0aGUgaGFyZHdhcmUgYmVoYXZlIGJhZGx5Lgo+ID4gCj4gSSBhZ3JlZSB3aXRoIERt aXRyeS4gQlcgaXMgYSBzaGFyZWQgcmVzb3VyY2UgaW4gYSBsb3Qgb2YgYXJjaGl0ZWN0dXJlcy4K PiBSYWlzaW5nIGNsayBiZWZvcmUgaW5jcmVhc2luZyB0aGUgYncgY2FuIGxlYWQgdG8gYSBzY2Vu YXJpbyB3aGVyZSB0aGlzCj4gY2xpZW50IHNhdHVyYXRlIHRoZSBlbnRpcmUgQlcgZm9yIHdoYXRl dmVyIHNtYWxsIGR1cmF0aW9uIGl0IG1heSBiZS4gVGhpcwo+IHdpbGwgaW1wYWN0IHRoZSBsYXRl bmN5IHJlcXVpcmVtZW50cyBvZiBvdGhlciBjbGllbnRzLgoKSSBzZWUuIEkgd2lsbCBtYWtlIHRo ZSBuZWNlc3NhcnkgY2hhbmdlcyB0aGVuIHRvIGZpeCBpdC4gVGhhbmtzIGd1eXMuCgotLSAKdmly ZXNoCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51 eC1hcm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVh ZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1h cm0ta2VybmVsCg==