From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755444AbdCGNUB (ORCPT ); Tue, 7 Mar 2017 08:20:01 -0500 Received: from mail-oi0-f67.google.com ([209.85.218.67]:36125 "EHLO mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754790AbdCGNTd (ORCPT ); Tue, 7 Mar 2017 08:19:33 -0500 MIME-Version: 1.0 In-Reply-To: <20170307103108.GA4526@vireshk-i7> References: <639aad743bac7f3292146738f44dbd1480169c8e.1488437503.git.viresh.kumar@linaro.org> <1772276.4tCnP8C0XV@aspire.rjw.lan> <1668614.4zDQWLsnmH@aspire.rjw.lan> <20170306044553.GE8206@vireshk-i7> <20170307103108.GA4526@vireshk-i7> From: "Rafael J. Wysocki" Date: Tue, 7 Mar 2017 14:19:01 +0100 X-Google-Sender-Auth: j0g3j3WzSHq3iM1xyBOp0qw38c4 Message-ID: Subject: Re: [PATCH 3/3] cpufreq: schedutil: remove redundant code from sugov_next_freq_shared() To: Viresh Kumar Cc: "Rafael J. Wysocki" , "Rafael J. Wysocki" , Ingo Molnar , Peter Zijlstra , Lists linaro-kernel , Linux PM , Linux Kernel Mailing List , Vincent Guittot Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 7, 2017 at 11:31 AM, Viresh Kumar wrote: > On 06-03-17, 13:24, Rafael J. Wysocki wrote: >> On Mon, Mar 6, 2017 at 5:45 AM, Viresh Kumar wrote: >> > On 04-03-17, 01:11, Rafael J. Wysocki wrote: >> >> So one idea is that if SCHED_CPUFREQ_RT_DL is set in flags, we don't even >> >> need to start the loop which is quite a cost to simply notice that there's >> >> nothing to do. >> > >> > Hmm. Isn't the probability of this flag being set, same for all CPUs in the >> > policy? >> >> No, I don't think so. > > Why do you think so? I thought all CPU in the policy can have the RT/DL flag set > and the probability of all of them is just the same. Well, yes, but if the current CPU has that flag set already, we surely don't need to check the other ones in the policy? >> So to the point, the code was written this way on purpose and not just >> by accident as your changelog suggests and > > I didn't wanted to convey that really and I knew that it was written on purpose. > >> if you want to change it, you need numbers. > > What kind of numbers can we get for such a change ? I tried to take the running > average of the time it takes to execute this routine over 10000 samples, but it > varies a lot even with the same build. Any tests like hackbench, etc wouldn't be > of any help as well. So why do you think it needs to be changed, but really? Is that because it is particularly hard to follow or similar? Thanks, Rafael