From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753801AbcFPG5P (ORCPT ); Thu, 16 Jun 2016 02:57:15 -0400 Received: from nat-hk.nvidia.com ([203.18.50.4]:36604 "EHLO hkmmgate101.nvidia.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751490AbcFPG5M (ORCPT ); Thu, 16 Jun 2016 02:57:12 -0400 X-PGP-Universal: processed; by hkpgpgate102.nvidia.com on Wed, 15 Jun 2016 23:57:10 -0700 Subject: Re: [PATCH RFC] Revert "cpufreq: dt: Identify cpu-sharing for platforms without operating-points-v2" To: Viresh Kumar , "Rafael J. Wysocki" , Thierry Reding References: <20160616053524.19921-1-acourbot@nvidia.com> <20160616063529.GA5726@vireshk-i7> CC: , , , From: Alexandre Courbot Organization: NVIDIA X-Nvconfidentiality: public Message-ID: <5db9eb89-1026-e710-755b-c7a0fa87819e@nvidia.com> Date: Thu, 16 Jun 2016 15:57:06 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0 MIME-Version: 1.0 In-Reply-To: <20160616063529.GA5726@vireshk-i7> X-Originating-IP: [10.19.57.156] X-ClientProxiedBy: HKMAIL103.nvidia.com (10.18.16.12) To HKMAIL103.nvidia.com (10.18.16.12) Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/16/2016 03:35 PM, Viresh Kumar wrote: > +Rafael > > On 16-06-16, 14:35, Alexandre Courbot wrote: >> This is not an actual request for revert, but rather for comments about >> the observed behavior since I am not really familiar with cpufreq. >> >> I am observing a serious performance regression on Jetson TK1 since 4.7-rc1: >> namely, moving windows under X would become unsufferably slow, and graphical >> performance under X in general is seriously degraded. After bisecting, I >> found the regression to be visible since commit 1530b9963eeb ("cpufreq: dt: >> Identify cpu-sharing for platforms without operating-points-v2") >> >> If I revert this commit, I noticed that the CPU frequency immediately jumps to >> a higher frequency once I start moving windows (resulting in a smooth and >> responsive action), whereas enabling this commit causes the CPU frequency to >> remain low (typically 204 Mhz) in that case, resulting in CPU-bound slowness. >> >> What happens is that with 1530b9963eeb applied, dev_pm_opp_get_sharing_cpus() >> returns zero causing the fallback variable to remain false, whereas without it >> opp_v1 is set to true. >> >> It is not clear to me whether this is a cpufreq issue or a Tegra issue, so I am >> posting this in the hope to get clarifications from either side. >> >> Thanks! > > Yeah, you just hit a corner case. Sorry about that :( > > Can you try a patch that I sent to you instead? Its subject is: > > [PATCH] PM / OPP: 'UNKNOWN' status of opp-table->shared Thanks, this seems to solve the issue. > Also I would like to know one more thing. You can hit this bug only if > some other piece of code for your platform is creating OPP table for > the CPUs. Which code is doing that ? Good question. I am still educating myself about OPP - adding Thierry in the meantime who may have an answer.