From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752181AbcHQIml (ORCPT ); Wed, 17 Aug 2016 04:42:41 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:35551 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750922AbcHQImi (ORCPT ); Wed, 17 Aug 2016 04:42:38 -0400 MIME-Version: 1.0 In-Reply-To: <1469453670-2660-6-git-send-email-morten.rasmussen@arm.com> References: <1469453670-2660-1-git-send-email-morten.rasmussen@arm.com> <1469453670-2660-6-git-send-email-morten.rasmussen@arm.com> From: Wanpeng Li Date: Wed, 17 Aug 2016 16:42:36 +0800 Message-ID: Subject: Re: [PATCH v3 05/13] sched: Introduce SD_ASYM_CPUCAPACITY sched_domain topology flag To: Morten Rasmussen Cc: Peter Zijlstra , Ingo Molnar , Dietmar Eggemann , Yuyang Du , Vincent Guittot , Mike Galbraith , sgurrappadi@nvidia.com, freedom.tan@mediatek.com, keita.kobayashi.ym@renesas.com, "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2016-07-25 21:34 GMT+08:00 Morten Rasmussen : > Add a topology flag to the sched_domain hierarchy indicating the lowest > domain level where the full range of cpu capacities is represented by > the domain members for asymmetric capacity topologies (e.g. ARM > big.LITTLE). > > The flag is intended to indicate that extra care should be taken when > placing tasks on cpus and this level spans all the different types of > cpus found in the system (no need to look further up the domain > hierarchy). This information is currently only available through > iterating through the capacities of all the cpus at parent levels in the > sched_domain hierarchy. > > SD 2 [ 0 1 2 3] SD_ASYM_CPUCAPACITY > > SD 1 [ 0 1] [ 2 3] !SD_ASYM_CPUCAPACITY > > cpu: 0 1 2 3 > capacity: 756 756 1024 1024 > > If the topology in the example above is duplicated to create an eight > cpu example with third sched_domain level on top (SD 3), this level > should not have the flag set (!SD_ASYM_CPUCAPACITY) as its two group > would both have all cpu capacities represented within them. I didn't find the place where set SD_ASYM_CPUCAPACITY to any SDs in this patchset, but you have testing result in cover letter, where I miss? Regards, Wanpeng Li