From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757675AbcHWONz (ORCPT ); Tue, 23 Aug 2016 10:13:55 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:42872 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755434AbcHWONx (ORCPT ); Tue, 23 Aug 2016 10:13:53 -0400 Date: Tue, 23 Aug 2016 16:13:41 +0200 From: Peter Zijlstra To: Vincent Guittot Cc: Yuyang Du , Ingo Molnar , linux-kernel , Benjamin Segall , Paul Turner , Morten Rasmussen , Dietmar Eggemann , Matt Fleming , Mike Galbraith Subject: Re: [PATCH v1 00/10] Optimize sched avgs computation and implement flat util hierarchy Message-ID: <20160823141341.GP10153@twins.programming.kicks-ass.net> References: <1470788095-2125-1-git-send-email-yuyang.du@intel.com> <20160810002352.GB3273@intel.com> <20160822232637.GC3273@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23.1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 23, 2016 at 03:28:19PM +0200, Vincent Guittot wrote: > I still wonder if using a flat util hierarchy is the right solution to > solve this problem with utilization and task group. I have noticed > exact same issues with load that generates weird task placement > decision and i think that we should probably try to solve both wrong > behavior with same mechanism. but this is not possible with flat > hierarchy for load > > Let me take an example. > TA is a always running task on CPU1 in group /root/level1/ > TB wakes up on CPU0 and moves TA into group /root/level2/ > Even if TA stays on CPU1, runnable_load_avg of CPU1 root cfs rq will become 0. Because while we migrate the load_avg on /root/level2, we do not propagate the load_avg up the hierarchy? And always propagating everyrthing up will indeed also fix the utilization issue. Of course, doing that propagation has its costs.. Didn't you post a patch doing just this a while ago?