From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755915AbaJHLWJ (ORCPT ); Wed, 8 Oct 2014 07:22:09 -0400 Received: from mail-oi0-f51.google.com ([209.85.218.51]:33606 "EHLO mail-oi0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754192AbaJHLWH (ORCPT ); Wed, 8 Oct 2014 07:22:07 -0400 MIME-Version: 1.0 In-Reply-To: <20141008110054.GA1788@e105550-lin.cambridge.arm.com> References: <1411403047-32010-1-git-send-email-morten.rasmussen@arm.com> <1411403047-32010-2-git-send-email-morten.rasmussen@arm.com> <20140925172343.GX23693@e103034-lin> <20141002203428.GI2849@worktop.programming.kicks-ass.net> <20141008110054.GA1788@e105550-lin.cambridge.arm.com> From: Vincent Guittot Date: Wed, 8 Oct 2014 13:21:45 +0200 Message-ID: Subject: Re: [PATCH 1/7] sched: Introduce scale-invariant load tracking To: Morten Rasmussen Cc: Peter Zijlstra , "mingo@redhat.com" , Dietmar Eggemann , Paul Turner , Benjamin Segall , Nicolas Pitre , Mike Turquette , "rjw@rjwysocki.net" , linux-kernel Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8 October 2014 13:00, Morten Rasmussen wrote: > On Thu, Oct 02, 2014 at 09:34:28PM +0100, Peter Zijlstra wrote: >> On Thu, Sep 25, 2014 at 06:23:43PM +0100, Morten Rasmussen wrote: >> >> > > Why haven't you used arch_scale_freq_capacity which has a similar >> > > purpose in scaling the CPU capacity except the additional sched_domain >> > > pointer argument ? >> > >> > To be honest I'm not happy with introducing another arch-function >> > either and I'm happy to change that. It wasn't really clear to me which >> > functions that would remain after your cpu_capacity rework patches, so I >> > added this one. Now that we have most of the patches for capacity >> > scaling and scale-invariant load-tracking on the table I think we have a >> > better chance of figuring out which ones are needed and exactly how they >> > are supposed to work. >> > >> > arch_scale_load_capacity() compensates for both frequency scaling and >> > micro-architectural differences, while arch_scale_freq_capacity() only >> > for frequency. As long as we can use arch_scale_cpu_capacity() to >> > provide the micro-architecture scaling we can just do the scaling in two >> > operations rather than one similar to how it is done for capacity in >> > update_cpu_capacity(). I can fix that in the next version. It will cost >> > an extra function call and multiplication though. >> > >> > To make sure that runnable_avg_{sum, period} are still bounded by >> > LOAD_AVG_MAX, arch_scale_{cpu,freq}_capacity() must both return a factor >> > in the range 0..SCHED_CAPACITY_SCALE. >> >> I would certainly like some words in the Changelog on how and that the >> math is still free of overflows. Clearly you've thought about it, so >> please feel free to elucidate the rest of us :-) > > Sure. The easiest way to avoid introducing overflows is to ensure that > we always scale by a factor >= 1.0. That should be true as long as > arch_scale_{cpu,freq}_capacity() never returns anything greater than > SCHED_CAPACITY_SCALE (= 1024 = 1.0). the current ARM arch_scale_cpu is in the range [1536..0] which is free of overflow AFAICT > > If we take big.LITTLE is an example, the max cpu capacity of a big cpu > would be 1024 and since we multiply the scaling factors (as in > update_cpu_capacity()) the max frequency scaling capacity factor would > be 1024. The result is a 1.0 (1.0 * 1.0) scaling factor when a task is > running on a big cpu at the highest frequency. At 50% frequency, the > scaling factor is 0.5 (1.0 * 0.5). > > For a little cpu arch_scale_cpu_capacity() would return something less > than 1024, 512 for example. The max frequency scaling capacity factor is > 1024. A task running on a little cpu at max frequency would have its > load scaled by 0.5 (0.5 * 1.0). At 50% frequency, it would be 0.25 (0.5 > * 0.5). > > However, as said earlier (below), we have to go through the load-balance > code to ensure that it doesn't blow up when cpu capacities get small > (huge.TINY), but the load-tracking code itself should be fine I think. > >> >> > > If we take the example of an always running task, its runnable_avg_sum >> > > should stay at the LOAD_AVG_MAX value whatever the frequency of the >> > > CPU on which it runs. But your change links the max value of >> > > runnable_avg_sum with the current frequency of the CPU so an always >> > > running task will have a load contribution of 25% >> > > your proposed scaling is fine with usage_avg_sum which reflects the >> > > effective running time on the CPU but the runnable_avg_sum should be >> > > able to reach LOAD_AVG_MAX whatever the current frequency is >> > >> > I don't think it makes sense to scale one metric and not the other. You >> > will end up with two very different (potentially opposite) views of the >> > cpu load/utilization situation in many scenarios. As I see it, >> > scale-invariance and load-balancing with scale-invariance present can be >> > done in two ways: >> > >> > 1. Leave runnable_avg_sum unscaled and scale running_avg_sum. >> > se->avg.load_avg_contrib will remain unscaled and so will >> > cfs_rq->runnable_load_avg, cfs_rq->blocked_load_avg, and >> > weighted_cpuload(). Essentially all the existing load-balancing code >> > will continue to use unscaled load. When we want to improve cpu >> > utilization and energy-awareness we will have to bypass most of this >> > code as it is likely to lead us on the wrong direction since it has a >> > potentially wrong view of the cpu load due to the lack of >> > scale-invariance. >> > >> > 2. Scale both runnable_avg_sum and running_avg_sum. All existing load >> > metrics including weighted_cpuload() are scaled and thus more accurate. >> > The difference between se->avg.load_avg_contrib and >> > se->avg.usage_avg_contrib is the priority scaling and whether or not >> > runqueue waiting time is counted. se->avg.load_avg_contrib can only >> > reach se->load.weight when running on the fastest cpu at the highest >> > frequency, but it is now scale-invariant so we have much better idea >> > about how much load we are pulling when load-balancing two cpus running >> > at different frequencies. The load-balance code-path still has to be >> > audited to see if anything blows up due to the scaling. I haven't >> > finished doing that yet. This patch set doesn't include patches to >> > address such issues (yet). IMHO, by scaling runnable_avg_sum we can more >> > easily make the existing load-balancing code do the right thing. >> > >> > For both options we have to go through the existing load-balancing code >> > to either change it to use the scale-invariant metric (running_avg_sum) >> > when appropriate or to fix bits that don't work properly with a >> > scale-invariant runnable_avg_sum and reuse the existing code. I think >> > the latter is less intrusive, but I might be wrong. >> > >> > Opinions? >> >> /me votes #2, I think the example in the reply is a false one, an always >> running task will/should ramp up the cpufreq and get us at full speed >> (and yes I'm aware of the case where you're memory bound and raising the >> cpu freq isn't going to actually improve performance, but I'm not sure >> we want to get/be that smart, esp. at this stage). > > Okay, and agreed that memory bound task smarts are out of scope for the > time being. > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/