From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,URIBL_DBL_ABUSE_MALW, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C7198C2D0DB for ; Fri, 31 Jan 2020 16:00:05 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8A375206D3 for ; Fri, 31 Jan 2020 16:00:05 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="gQ732LVd" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8A375206D3 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=rXdHzerPamGwrrwZ3bgsBd2DZr+WduhOc0g2bHoRt+A=; b=gQ732LVdj/om0Ee8xwzCVG7FF chDM0HAMZqx6iUw9zyscE0YgqgFtXkntute6zjMw7Gp8mw7jbPARpDucXkj5RZ3y3k5HCESN9CMHB hmUUcz2T3xjH8tMBjY6GAHLfc73lq/kyykASmk2BJVUOiTuhH5YikVX0h0aTzs+mkQuR8bPSEk1zi /y3nW+UGLyYxNawjPWo0gt4xVSstDcgBRD4wAfToVut7MfzIepck7oJDUM0e+JhLuN//1vTOiG2AD bUYu+5hsn/TfvQaj5dWpjqLhoX/kQVLJ5SmRLiGJmsYAsq6+bMY8OiOL/yMhRK/2hGsX/zPNwVJNR 3VhWD54PQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1ixYiA-0004RQ-DS; Fri, 31 Jan 2020 16:00:02 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1ixYi5-0004QA-MT for linux-arm-kernel@lists.infradead.org; Fri, 31 Jan 2020 15:59:59 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DA5FDFEC; Fri, 31 Jan 2020 07:59:54 -0800 (PST) Received: from [10.37.12.54] (unknown [10.37.12.54]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 45BF73F68E; Fri, 31 Jan 2020 07:59:47 -0800 (PST) Subject: Re: [PATCH 1/3] ARM: exynos_defconfig: Enable SCHED_MC To: Krzysztof Kozlowski References: <20200127215453.15144-1-lukasz.luba@arm.com> <20200127215453.15144-2-lukasz.luba@arm.com> From: Lukasz Luba Message-ID: Date: Fri, 31 Jan 2020 15:59:30 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200131_075957_823281_F5858547 X-CRM114-Status: GOOD ( 20.41 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, "linux-samsung-soc@vger.kernel.org" , linux-pm@vger.kernel.org, =?UTF-8?Q?Bart=c5=82omiej_=c5=bbo=c5=82nierkiewicz?= , "linux-kernel@vger.kernel.org" , robh+dt@kernel.org, Chanwoo Choi , kyungmin.park@samsung.com, kgene@kernel.org, myungjoo.ham@samsung.com, dietmar.eggemann@arm.com, linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Krzysztof, Thank you for your review, please see my comments below. On 1/31/20 12:47 PM, Krzysztof Kozlowski wrote: > On Mon, 27 Jan 2020 at 22:55, wrote: >> >> From: Lukasz Luba >> >> Since the 'capacities-dmips-mhz' are present in the CPU nodes, make use of >> this knowledge in smarter decisions during scheduling. >> >> The values in 'capacities-dmips-mhz' are normilized, this means that i.e. >> when CPU0's capacities-dmips-mhz=100 and CPU1's 'capacities-dmips-mhz'=50, >> cpu0 is twice fast as CPU1, at the same frequency. The proper hirarchy >> in sched_domain topology could exploit the SoC architecture advantages >> like big.LITTLE. > > I do not quite get how this is related to rationale behind changing defconfig... It is not strictly about EAS, it is useful in general for big.LITTLE platform with clusters. > >> Enabling the SCHED_MC will create two levels in >> sched_domain hierarchy, which might be observed in: > > This is looks more convincing... but still what is the need? To work with EAS? It is not only for EAS, but in general for the scheduler (load balance, task's wake-up path, etc). The scheduler algorithms iterate CPUs in the sched groups. To make better decisions, the information about MC domain is needed. More about the scheduler domains and i.e. load_balance() you can find here: https://www.kernel.org/doc/html/latest/scheduler/sched-domains.html > >> grep . /proc/sys/kernel/sched_domain/cpu*/domain*/{name,flags} >> /proc/sys/kernel/sched_domain/cpu0/domain0/name:MC >> /proc/sys/kernel/sched_domain/cpu0/domain1/name:DIE >> ... >> /proc/sys/kernel/sched_domain/cpu0/domain0/flags:575 >> /proc/sys/kernel/sched_domain/cpu0/domain1/flags:4223 > > Not related to defconfig change and not visible after this commit. Without this patch there is only one domain: 'domain0' -> 'DIE' cat /proc/sys/kernel/sched_domain/cpu0/domain0/name DIE When you apply this patch you will get two: 'domain0, 'domain1' grep . /proc/sys/kernel/sched_domain/cpu0/domain?/name /proc/sys/kernel/sched_domain/cpu0/domain0/name:MC /proc/sys/kernel/sched_domain/cpu0/domain1/name:DIE I can remove it this information, but it is the most important to spot this difference out. This is also the main reason I haven't merge the patch 1 + 3. Regards, Lukasz > > Best regards, > Krzysztof > >> >> Signed-off-by: Lukasz Luba >> --- >> arch/arm/configs/exynos_defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig >> index e7e4bb5ad8d5..1db857056992 100644 >> --- a/arch/arm/configs/exynos_defconfig >> +++ b/arch/arm/configs/exynos_defconfig >> @@ -8,6 +8,7 @@ CONFIG_PERF_EVENTS=y >> CONFIG_ARCH_EXYNOS=y >> CONFIG_CPU_ICACHE_MISMATCH_WORKAROUND=y >> CONFIG_SMP=y >> +CONFIG_SCHED_MC=y >> CONFIG_BIG_LITTLE=y >> CONFIG_NR_CPUS=8 >> CONFIG_HIGHMEM=y >> -- >> 2.17.1 >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel