[{"id":1769921,"web_url":"http://patchwork.ozlabs.org/comment/1769921/","msgid":"<20170918073946.GA9156@verge.net.au>","list_archive_url":null,"date":"2017-09-18T07:39:46","subject":"Re: [PATCH 4/4] arm: dts: r8a7790: add cpu capacity-dmips-mhz\n\tinformation","submitter":{"id":106,"url":"http://patchwork.ozlabs.org/api/people/106/","name":"Simon Horman","email":"horms@verge.net.au"},"content":"On Wed, Aug 30, 2017 at 03:41:20PM +0100, Dietmar Eggemann wrote:\n> The following 'capacity-dmips-mhz' dt property values are used:\n> \n> Cortex-A15: 1024, Cortex-A7: 539\n> \n> They have been derived form the cpu_efficiency values:\n> \n> Cortex-A15: 3891, Cortex-A7: 2048\n> \n> by scaling them so that the Cortex-A15s (big cores) use 1024.\n> \n> The cpu_efficiency values were originally derived from the \"Big.LITTLE\n> Processing with ARM Cortex™-A15 & Cortex-A7\" white paper\n> (http://www.cl.cam.ac.uk/~rdm34/big.LITTLE.pdf). Table 1 lists 1.9x\n> (3891/2048) as the Cortex-A15 vs Cortex-A7 performance ratio for the\n> Dhrystone benchmark.\n> \n> The following platform is affected once cpu-invariant accounting\n> support is re-connected to the task scheduler:\n\nThanks, applied for v4.15.\n\nMy understanding from the following comment in the cover letter is that not\ncurrently the case and this there is no behavioural change in applying this\npatch.\n\nFor the record I observed the following with and without this patch\napplied. I believe this is the expected result.\n\nv4.14-rc1\n# cat /sys/devices/system/cpu/cpu*/cpu_capacity\n1535\n1535\n1535\n1535\n1024\n1024\n1024\n1024\n\nv4.14-rc1 + patch\n# cat /sys/devices/system/cpu/cpu*/cpu_capacity\n1024\n1024\n1024\n1024\n539\n539\n539\n539","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"gimKClLv\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=verge.net.au header.i=@verge.net.au\n\theader.b=\"Vc8mgN4y\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xwdGH4jVlz9s3w\n\tfor ;\n\tMon, 18 Sep 2017 17:40:39 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dtqfU-0004QT-2L; Mon, 18 Sep 2017 07:40:36 +0000","from kirsty.vergenet.net ([202.4.237.240])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dtqfP-0003YU-Bg for linux-arm-kernel@lists.infradead.org;\n\tMon, 18 Sep 2017 07:40:33 +0000","from penelope.horms.nl (unknown [217.111.208.18])\n\tby kirsty.vergenet.net (Postfix) with ESMTPA id AE83725B80D;\n\tMon, 18 Sep 2017 17:40:04 +1000 (AEST)","by penelope.horms.nl (Postfix, from userid 7100)\n\tid BF45AE231ED; Mon, 18 Sep 2017 03:39:46 -0400 (EDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=TMfcGLgcyVmBXwpjXz9HAVZzKfqyE9e0Zy/9lstggH4=;\n\tb=gimKClLvF4/01Y\n\tOBX72s8ZW3ros2qIsRkmWfsyin5itqEwWcxvJj2zo9pvgrakufCKkPbnvNW/J+ZZ9g1x+qCDucmZG\n\tjjsI/r6floN4Z+YGXWN/TwR67FYZZTZ4uLN63RQwV3k/OfimNZ6tiOEUC3bKtQjF3VjMOrtiVQQW9\n\tbuWFOwhQllyI66pJkmeSqS5JGi8Uu7LY+aA2PT0dttH+aJ621aaWi+LaR7D0H47fn5ovdzk/AhAlW\n\tWczTgyJ1PP1EsN1gWIpXCw0kLIqdZo9A8T9Nx/IrJT1UH5uH2b36ui29EPw1nGdLFNzbcHFYSahI/\n\tprziMjJNItpfJ1kS+tvw==;","v=1; a=rsa-sha256; c=simple/simple; d=verge.net.au; s=mail; \n\tt=1505720405; bh=Oc/9rw/Zthnvc5QLOG4VvIm/CE6MDJ/U0Bxfy85EY6g=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=Vc8mgN4yPKq2ZNiMS0VCxTD7h3qT4AC6YqEYLMU6FTW16l6u2eHcs1HaZPzIdXbHh\n\tlXdx94QH/cy9Cp15Mt2TqMi4Bri9O/EQexHMdl2DZuacuyPdzi08PzQGmuRC/N+uwp\n\tTKs3tteohu0KBKXUa0X/dIilx3n1mWjcd+ndbZhs="],"Date":"Mon, 18 Sep 2017 09:39:46 +0200","From":"Simon Horman ","To":"Dietmar Eggemann ","Subject":"Re: [PATCH 4/4] arm: dts: r8a7790: add cpu capacity-dmips-mhz\n\tinformation","Message-ID":"<20170918073946.GA9156@verge.net.au>","References":"<20170830144120.9312-1-dietmar.eggemann@arm.com>\n\t<20170830144120.9312-5-dietmar.eggemann@arm.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170830144120.9312-5-dietmar.eggemann@arm.com>","Organisation":"Horms Solutions BV","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170918_004031_732274_738BC9F6 ","X-CRM114-Status":"GOOD ( 11.61 )","X-Spam-Score":"-4.3 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-4.3 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\n\tmedium trust [202.4.237.240 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Mark Rutland , devicetree@vger.kernel.org,\n\tlinux-samsung-soc@vger.kernel.org,\n\tVincent Guittot ,\n\tJuri Lelli , \n\tlinux-kernel@vger.kernel.org, Krzysztof Kozlowski , \n\tRussell King , linux-renesas-soc@vger.kernel.org, \n\tRob Herring , Kukjin Kim ,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1783105,"web_url":"http://patchwork.ozlabs.org/comment/1783105/","msgid":"","list_archive_url":null,"date":"2017-10-09T17:55:07","subject":"Re: [PATCH 4/4] arm: dts: r8a7790: add cpu capacity-dmips-mhz\n\tinformation","submitter":{"id":64545,"url":"http://patchwork.ozlabs.org/api/people/64545/","name":"Dietmar Eggemann","email":"dietmar.eggemann@arm.com"},"content":"On 18/09/17 08:39, Simon Horman wrote:\n> On Wed, Aug 30, 2017 at 03:41:20PM +0100, Dietmar Eggemann wrote:\n>> The following 'capacity-dmips-mhz' dt property values are used:\n>>\n>> Cortex-A15: 1024, Cortex-A7: 539\n>>\n>> They have been derived form the cpu_efficiency values:\n>>\n>> Cortex-A15: 3891, Cortex-A7: 2048\n>>\n>> by scaling them so that the Cortex-A15s (big cores) use 1024.\n>>\n>> The cpu_efficiency values were originally derived from the \"Big.LITTLE\n>> Processing with ARM Cortex™-A15 & Cortex-A7\" white paper\n>> (http://www.cl.cam.ac.uk/~rdm34/big.LITTLE.pdf). Table 1 lists 1.9x\n>> (3891/2048) as the Cortex-A15 vs Cortex-A7 performance ratio for the\n>> Dhrystone benchmark.\n>>\n>> The following platform is affected once cpu-invariant accounting\n>> support is re-connected to the task scheduler:\n> \n> Thanks, applied for v4.15.\n> \n> My understanding from the following comment in the cover letter is that not\n> currently the case and this there is no behavioural change in applying this\n> patch.\n> \n> For the record I observed the following with and without this patch\n> applied. I believe this is the expected result.\n> \n> v4.14-rc1\n> # cat /sys/devices/system/cpu/cpu*/cpu_capacity\n> 1535\n> 1535\n> 1535\n> 1535\n> 1024\n> 1024\n> 1024\n> 1024\n> \n> v4.14-rc1 + patch\n> # cat /sys/devices/system/cpu/cpu*/cpu_capacity\n> 1024\n> 1024\n> 1024\n> 1024\n> 539\n> 539\n> 539\n> 539\n\nThanks Simon! Yes, that is the expected behaviour. And sorry for not\nresponding earlier!\n\nWith exynos542{0,2} and r8a7790 switching to the 'capacity-dmips-mhz'\nbased solution in v4.15, I can push for removal of the cpu_efficency\ncode [patch 1/4].","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"S420x7A2\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y9nw86FWPz9tY0\n\tfor ;\n\tTue, 10 Oct 2017 04:55:36 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1cH9-0006cI-5R; Mon, 09 Oct 2017 17:55:35 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e1cH4-0006Tn-TH for linux-arm-kernel@lists.infradead.org;\n\tMon, 09 Oct 2017 17:55:32 +0000","from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])\n\tby usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BA2821529;\n\tMon, 9 Oct 2017 10:55:10 -0700 (PDT)","from [10.1.210.41] (e107985-lin.cambridge.arm.com [10.1.210.41])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id\n\t9E2E83F483; Mon, 9 Oct 2017 10:55:08 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=hiEFagbhsNzvztBssQpIyc1XZCChueY2NcBbDCQ61wE=;\n\tb=S420x7A2upZfcJ\n\t5W1e/7xK5zIkBYG1CeEdNzu7GeaJ5PE13vBK61eFfNimI89XrBsJS9lzSlbM6dFe5UvKrn1h3yxbL\n\tar4h/WiAMj/KJfNg+GNE0VTJqP1XAoNuGXKNhco1Hh0gfMXv0Qaevs7OaOMHoME1sIADL7spfLgLs\n\t4vxa2Uf/p98sRvCeTgj8ZPm8PETGsLSfjsfVBVPVp1PSaFyfgf6u3jhVO0sAhGf55XIrl9uhALXM1\n\tajnTKgs0qdb5Vh+YS0BtRc27yeIFPzs5RTLADziu71L0bSGqUrypeCgwRmj7N/ySRQtjTuQb4vLWJ\n\t5G/tzPAYdo+smU8FDLPA==;","Subject":"Re: [PATCH 4/4] arm: dts: r8a7790: add cpu capacity-dmips-mhz\n\tinformation","To":"Simon Horman ","References":"<20170830144120.9312-1-dietmar.eggemann@arm.com>\n\t<20170830144120.9312-5-dietmar.eggemann@arm.com>\n\t<20170918073946.GA9156@verge.net.au>","From":"Dietmar Eggemann ","Message-ID":"","Date":"Mon, 9 Oct 2017 18:55:07 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170918073946.GA9156@verge.net.au>","Content-Language":"en-GB","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171009_105530_954803_8476C714 ","X-CRM114-Status":"GOOD ( 13.97 )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [217.140.101.70 listed in list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Mark Rutland , devicetree@vger.kernel.org,\n\tlinux-samsung-soc@vger.kernel.org,\n\tVincent Guittot ,\n\tJuri Lelli , \n\tlinux-kernel@vger.kernel.org, Krzysztof Kozlowski , \n\tRussell King , linux-renesas-soc@vger.kernel.org, \n\tRob Herring , Kukjin Kim ,\n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]