[{"id":1633162,"web_url":"http://patchwork.ozlabs.org/comment/1633162/","msgid":"","date":"2017-04-13T07:24:28","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":5234,"url":"http://patchwork.ozlabs.org/api/people/5234/","name":"Christophe Leroy","email":"christophe.leroy@c-s.fr"},"content":"Hi Baldir\n\nLe 13/04/2017 à 09:03, Balbir Singh a écrit :\n> We expect to have these configs on by default, most\n> distros turn them off, its always good to have them on\n> so that we can use them and hopefully not break them\n\nIsn't it the purpose of the target allyesconfig to allow such tests ?\n\nFTRACE is quite CPU consumming, shouldn't it really be on by default ?\n\nChristophe\n\n>\n> Signed-off-by: Balbir Singh \n> ---\n> arch/powerpc/configs/powernv_defconfig | 4 ++++\n> arch/powerpc/configs/ppc64_defconfig | 3 +++\n> arch/powerpc/configs/pseries_defconfig | 3 +++\n> 3 files changed, 10 insertions(+)\n>\n> diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc/configs/powernv_defconfig\n> index 0695ce0..eff3968 100644\n> --- a/arch/powerpc/configs/powernv_defconfig\n> +++ b/arch/powerpc/configs/powernv_defconfig\n> @@ -295,7 +295,11 @@ CONFIG_DEBUG_STACK_USAGE=y\n> CONFIG_DEBUG_STACKOVERFLOW=y\n> CONFIG_LOCKUP_DETECTOR=y\n> CONFIG_LATENCYTOP=y\n> +CONFIG_FTRACE=y\n> +CONFIG_FUNCTION_TRACER=y\n> +CONFIG_FUNCTION_GRAPH_TRACER=y\n> CONFIG_SCHED_TRACER=y\n> +CONFIG_FTRACE_SYSCALLS=y\n> CONFIG_BLK_DEV_IO_TRACE=y\n> CONFIG_UPROBE_EVENT=y\n> CONFIG_CODE_PATCHING_SELFTEST=y\n> diff --git a/arch/powerpc/configs/ppc64_defconfig b/arch/powerpc/configs/ppc64_defconfig\n> index e353168f9..17c6c12 100644\n> --- a/arch/powerpc/configs/ppc64_defconfig\n> +++ b/arch/powerpc/configs/ppc64_defconfig\n> @@ -328,6 +328,9 @@ CONFIG_DEBUG_STACKOVERFLOW=y\n> CONFIG_LOCKUP_DETECTOR=y\n> CONFIG_DEBUG_MUTEXES=y\n> CONFIG_LATENCYTOP=y\n> +CONFIG_FTRACE=y\n> +CONFIG_FUNCTION_TRACER=y\n> +CONFIG_FUNCTION_GRAPH_TRACER=y\n> CONFIG_SCHED_TRACER=y\n> CONFIG_BLK_DEV_IO_TRACE=y\n> CONFIG_UPROBE_EVENT=y\n> diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/configs/pseries_defconfig\n> index 1a61aa2..a445960 100644\n> --- a/arch/powerpc/configs/pseries_defconfig\n> +++ b/arch/powerpc/configs/pseries_defconfig\n> @@ -293,6 +293,9 @@ CONFIG_DEBUG_STACK_USAGE=y\n> CONFIG_DEBUG_STACKOVERFLOW=y\n> CONFIG_LOCKUP_DETECTOR=y\n> CONFIG_LATENCYTOP=y\n> +CONFIG_FTRACE=y\n> +CONFIG_FUNCTION_TRACER=y\n> +CONFIG_FUNCTION_GRAPH_TRACER=y\n> CONFIG_SCHED_TRACER=y\n> CONFIG_BLK_DEV_IO_TRACE=y\n> CONFIG_UPROBE_EVENT=y\n>","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w3XPj23Tkz9s7t\n\tfor ;\n\tThu, 13 Apr 2017 17:25:29 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w3XPj07rZzDqKg\n\tfor ;\n\tThu, 13 Apr 2017 17:25:29 +1000 (AEST)","from pegase1.c-s.fr (pegase1.c-s.fr [93.17.236.30])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w3XNd5SznzDq7g\n\tfor ;\n\tThu, 13 Apr 2017 17:24:33 +1000 (AEST)","from localhost (mailhub1-int [192.168.12.234])\n\tby localhost (Postfix) with ESMTP id 3w3XNL6DW2z9ttfr;\n\tThu, 13 Apr 2017 09:24:18 +0200 (CEST)","from pegase1.c-s.fr ([192.168.12.234])\n\tby localhost (pegase1.c-s.fr [192.168.12.234]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id gj2xfObagTNn; Thu, 13 Apr 2017 09:24:18 +0200 (CEST)","from messagerie.si.c-s.fr (messagerie.si.c-s.fr [192.168.25.192])\n\tby pegase1.c-s.fr (Postfix) with ESMTP id 3w3XNL5dlMz9ttfn;\n\tThu, 13 Apr 2017 09:24:18 +0200 (CEST)","from localhost (localhost [127.0.0.1])\n\tby messagerie.si.c-s.fr (Postfix) with ESMTP id DB9FF8B7BB;\n\tThu, 13 Apr 2017 09:24:28 +0200 (CEST)","from messagerie.si.c-s.fr ([127.0.0.1])\n\tby localhost (messagerie.si.c-s.fr [127.0.0.1]) (amavisd-new,\n\tport 10023)\n\twith ESMTP id S-zeVnx_4_si; Thu, 13 Apr 2017 09:24:28 +0200 (CEST)","from PO15451 (po15451.idsi0.si.c-s.fr [172.25.231.16])\n\tby messagerie.si.c-s.fr (Postfix) with ESMTP id BB16A8B7AA;\n\tThu, 13 Apr 2017 09:24:28 +0200 (CEST)"],"X-Virus-Scanned":["Debian amavisd-new at c-s.fr","amavisd-new at c-s.fr"],"Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","To":"Balbir Singh , mpe@ellerman.id.au","References":"<20170413070309.10497-1-bsingharora@gmail.com>","From":"Christophe LEROY ","Message-ID":"","Date":"Thu, 13 Apr 2017 09:24:28 +0200","User-Agent":"Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101\n\tThunderbird/45.8.0","MIME-Version":"1.0","In-Reply-To":"<20170413070309.10497-1-bsingharora@gmail.com>","Content-Type":"text/plain; charset=windows-1252; format=flowed","Content-Transfer-Encoding":"8bit","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1633207,"web_url":"http://patchwork.ozlabs.org/comment/1633207/","msgid":"<1492071568.8380.20.camel@gmail.com>","date":"2017-04-13T08:19:28","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":9347,"url":"http://patchwork.ozlabs.org/api/people/9347/","name":"Balbir Singh","email":"bsingharora@gmail.com"},"content":"On Thu, 2017-04-13 at 09:24 +0200, Christophe LEROY wrote:\n> Hi Baldir\n\nHi Christophe, It's Balbir\n\n> \n> Le 13/04/2017 à 09:03, Balbir Singh a écrit :\n> > We expect to have these configs on by default, most\n> > distros turn them off, its always good to have them on\n> > so that we can use them and hopefully not break them\n> \n> Isn't it the purpose of the target allyesconfig to allow such tests ?\n> \n\nAs a developer I don't use allyesconfig for submitting patches or testing\nthe kernel.\n\n> FTRACE is quite CPU consumming, shouldn't it really be on by default ?\n\nIt does some work at boot to NOP out function entry points at _mcount\nlocations. Is that what you are referring to? Or the overhead of the\ncode in terms of size? Most distro kernels have tracing on by default.\n\nThe rest of the overhead is enablement based.\n\nThanks for the review!\nBalbir Singh","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w3Ydp4M0vz9s7B\n\tfor ;\n\tThu, 13 Apr 2017 18:21:02 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w3Ydp38kGzDqFR\n\tfor ;\n\tThu, 13 Apr 2017 18:21:02 +1000 (AEST)","from mail-pg0-x230.google.com (mail-pg0-x230.google.com\n\t[IPv6:2607:f8b0:400e:c05::230])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w3Yc803SYzDq7g\n\tfor ;\n\tThu, 13 Apr 2017 18:19:36 +1000 (AEST)","by mail-pg0-x230.google.com with SMTP id g2so27040299pge.3\n\tfor ;\n\tThu, 13 Apr 2017 01:19:35 -0700 (PDT)","from firefly.ozlabs.ibm.com ([122.99.82.10])\n\tby smtp.googlemail.com with ESMTPSA id\n\tu29sm40753870pfa.123.2017.04.13.01.19.31\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tThu, 13 Apr 2017 01:19:32 -0700 (PDT)"],"Authentication-Results":["ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"PD4ZrBnn\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"PD4ZrBnn\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"PD4ZrBnn\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=message-id:subject:from:to:cc:date:in-reply-to:references\n\t:mime-version:content-transfer-encoding;\n\tbh=VyR3BdYiiyawn+rqcZ9YTp3YH9dQaszV58eoo0yLIRw=;\n\tb=PD4ZrBnniRSahFm2JOhCLpDnS5Wt+rYYNQsZ4FpG1MvVbKOBBpthBrdO2i6b84OFpP\n\tzzTQc6AJnNq1kG4oZx7hVSXgef/kOmyBhYm8tn1St5rLX8AiIIh+CePr9HtXxb7NbwcY\n\tdFmX6XaApidPdK3U4XXo4iNgiFkLuqtk7YG3NjXjtkkZv23ekpWqoganA5tJJ3V2meso\n\tiUlzsWc4PY0/A36JRIpucIDfaeEr1zNQhnjgAstiYl6arXxA/T2Qcuo66OYHvsVR39ZM\n\tT2N6btzxZ2EV2R6nJkOz5kUi+UN/UyiQNfIzp7YoNeznlTgD5iSJMYqAYm/rUEw+789J\n\tm63w==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to\n\t:references:mime-version:content-transfer-encoding;\n\tbh=VyR3BdYiiyawn+rqcZ9YTp3YH9dQaszV58eoo0yLIRw=;\n\tb=rFAVZ5sZa7YBQTVWTNgkkVr/L8gAnWBZKGktOVhfJuzImor9Tt+OsBqd02Q2OdH9Aa\n\tdu31g9Tb6K1fX0RNS66suJbhV8YWwmn5SY8/n5uO3F2OZTT5nLTKZb2GI1gz3ptrlPPp\n\t7kkeznGcc2R+tqL9/T+Io/DvknPWiiGxPJWS6aKONNk4zYoyLWRaPdvB6RB/ff/0tvJ9\n\tJXMX5kLcC+0aRYCslTK1lQvuU9b+CJfF14o0LtKEG7r2pIi4465EAm1rGxQWQCAL3wYM\n\t91b4+7hpKiIptqwO1J4f0XaBjie8dS+dN9efXUCyRdBJLE0rzPqViwXBAPljgG29OOPm\n\tLhcA==","X-Gm-Message-State":"AN3rC/4dTu1369ew+MupEBPX+RhSSEpjI59vs+ohbKirtiS3Dp2Mt9pS\n\td1aUynOQORwyew==","X-Received":"by 10.84.228.207 with SMTP id y15mr2669755pli.168.1492071574076; \n\tThu, 13 Apr 2017 01:19:34 -0700 (PDT)","Message-ID":"<1492071568.8380.20.camel@gmail.com>","Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","From":"Balbir Singh ","To":"Christophe LEROY , mpe@ellerman.id.au","Date":"Thu, 13 Apr 2017 18:19:28 +1000","In-Reply-To":"","References":"<20170413070309.10497-1-bsingharora@gmail.com>\n\t","Content-Type":"text/plain; charset=\"UTF-8\"","X-Mailer":"Evolution 3.22.6 (3.22.6-2.fc25) ","Mime-Version":"1.0","Content-Transfer-Encoding":"8bit","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1633216,"web_url":"http://patchwork.ozlabs.org/comment/1633216/","msgid":"<585eeaa5-52bd-7fe4-2179-3582570966e5@c-s.fr>","date":"2017-04-13T08:41:32","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":5234,"url":"http://patchwork.ozlabs.org/api/people/5234/","name":"Christophe Leroy","email":"christophe.leroy@c-s.fr"},"content":"Le 13/04/2017 à 10:19, Balbir Singh a écrit :\n> On Thu, 2017-04-13 at 09:24 +0200, Christophe LEROY wrote:\n>> Hi Baldir\n>\n> Hi Christophe, It's Balbir\n\nSorry\n\n>\n>>\n>> Le 13/04/2017 à 09:03, Balbir Singh a écrit :\n>>> We expect to have these configs on by default, most\n>>> distros turn them off, its always good to have them on\n>>> so that we can use them and hopefully not break them\n>>\n>> Isn't it the purpose of the target allyesconfig to allow such tests ?\n>>\n>\n> As a developer I don't use allyesconfig for submitting patches or testing\n> the kernel.\n\nI do it most of the time to check compilation doesn't fail.\n\n>\n>> FTRACE is quite CPU consumming, shouldn't it really be on by default ?\n>\n> It does some work at boot to NOP out function entry points at _mcount\n> locations. Is that what you are referring to? Or the overhead of the\n> code in terms of size? Most distro kernels have tracing on by default.\n\nYou said in you patch text: \"most distros turn them off\".\n\nIf most distros turn it ON (and not OFF as you said), then I can \nunderstand we have it ON as well in defconfigs.\n\nChristophe\n\n>\n> The rest of the overhead is enablement based.\n>\n> Thanks for the review!\n> Balbir Singh\n>","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w3Z6b2TKxz9s87\n\tfor ;\n\tThu, 13 Apr 2017 18:42:31 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w3Z6b1d36zDqKR\n\tfor ;\n\tThu, 13 Apr 2017 18:42:31 +1000 (AEST)","from pegase1.c-s.fr (pegase1.c-s.fr [93.17.236.30])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w3Z5X2KndzDq7g\n\tfor ;\n\tThu, 13 Apr 2017 18:41:36 +1000 (AEST)","from localhost (mailhub1-int [192.168.12.234])\n\tby localhost (Postfix) with ESMTP id 3w3Z5G12wBz9ttfv;\n\tThu, 13 Apr 2017 10:41:22 +0200 (CEST)","from pegase1.c-s.fr ([192.168.12.234])\n\tby localhost (pegase1.c-s.fr [192.168.12.234]) (amavisd-new,\n\tport 10024)\n\twith ESMTP id REOulSRfKQyQ; Thu, 13 Apr 2017 10:41:22 +0200 (CEST)","from messagerie.si.c-s.fr (messagerie.si.c-s.fr [192.168.25.192])\n\tby pegase1.c-s.fr (Postfix) with ESMTP id 3w3Z5G0Ty9z9ttfs;\n\tThu, 13 Apr 2017 10:41:22 +0200 (CEST)","from localhost (localhost [127.0.0.1])\n\tby messagerie.si.c-s.fr (Postfix) with ESMTP id 73D738B859;\n\tThu, 13 Apr 2017 10:41:32 +0200 (CEST)","from messagerie.si.c-s.fr ([127.0.0.1])\n\tby localhost (messagerie.si.c-s.fr [127.0.0.1]) (amavisd-new,\n\tport 10023)\n\twith ESMTP id TEloqIbZ6dSA; Thu, 13 Apr 2017 10:41:32 +0200 (CEST)","from PO15451 (po15451.idsi0.si.c-s.fr [172.25.231.16])\n\tby messagerie.si.c-s.fr (Postfix) with ESMTP id 5572A8B85C;\n\tThu, 13 Apr 2017 10:41:32 +0200 (CEST)"],"X-Virus-Scanned":["Debian amavisd-new at c-s.fr","amavisd-new at c-s.fr"],"Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","To":"Balbir Singh , mpe@ellerman.id.au","References":"<20170413070309.10497-1-bsingharora@gmail.com>\n\t\n\t<1492071568.8380.20.camel@gmail.com>","From":"Christophe LEROY ","Message-ID":"<585eeaa5-52bd-7fe4-2179-3582570966e5@c-s.fr>","Date":"Thu, 13 Apr 2017 10:41:32 +0200","User-Agent":"Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101\n\tThunderbird/45.8.0","MIME-Version":"1.0","In-Reply-To":"<1492071568.8380.20.camel@gmail.com>","Content-Type":"text/plain; charset=utf-8; format=flowed","Content-Transfer-Encoding":"8bit","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1633359,"web_url":"http://patchwork.ozlabs.org/comment/1633359/","msgid":"","date":"2017-04-13T11:51:37","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":9347,"url":"http://patchwork.ozlabs.org/api/people/9347/","name":"Balbir Singh","email":"bsingharora@gmail.com"},"content":"On Thu, Apr 13, 2017 at 6:41 PM, Christophe LEROY\n wrote:\n>\n>\n> Le 13/04/2017 à 10:19, Balbir Singh a écrit :\n>>\n>> On Thu, 2017-04-13 at 09:24 +0200, Christophe LEROY wrote:\n>>>\n>>> Hi Baldir\n>>\n>>\n>> Hi Christophe, It's Balbir\n>\n>\n> Sorry\n>\nNo worries!\n\n>>\n>>>\n>>> Le 13/04/2017 à 09:03, Balbir Singh a écrit :\n>>>>\n>>>> We expect to have these configs on by default, most\n>>>> distros turn them off, its always good to have them on\n>>>> so that we can use them and hopefully not break them\n>>>\n>>>\n>>> Isn't it the purpose of the target allyesconfig to allow such tests ?\n>>>\n>>\n>> As a developer I don't use allyesconfig for submitting patches or testing\n>> the kernel.\n>\n>\n> I do it most of the time to check compilation doesn't fail.\n>\n>>\n>>> FTRACE is quite CPU consumming, shouldn't it really be on by default ?\n>>\n>>\n>> It does some work at boot to NOP out function entry points at _mcount\n>> locations. Is that what you are referring to? Or the overhead of the\n>> code in terms of size? Most distro kernels have tracing on by default.\n>\n>\n> You said in you patch text: \"most distros turn them off\".\n>\n\nMy bad.. sometimes one can type faster than one can think, this is one\nof those occasions :)\n\n> If most distros turn it ON (and not OFF as you said), then I can understand\n\nYes\n\nBalbir Singh.","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w3fLT4ZgZz9sNW\n\tfor ;\n\tThu, 13 Apr 2017 21:53:05 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w3fLT3WnQzDq8B\n\tfor ;\n\tThu, 13 Apr 2017 21:53:05 +1000 (AEST)","from mail-lf0-x22a.google.com (mail-lf0-x22a.google.com\n\t[IPv6:2a00:1450:4010:c07::22a])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w3fJs1MNhzDq7j\n\tfor ;\n\tThu, 13 Apr 2017 21:51:41 +1000 (AEST)","by mail-lf0-x22a.google.com with SMTP id s141so28309784lfe.3\n\tfor ;\n\tThu, 13 Apr 2017 04:51:41 -0700 (PDT)","by 10.46.81.10 with HTTP; Thu, 13 Apr 2017 04:51:37 -0700 (PDT)"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"LlqaBMB8\"; dkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"LlqaBMB8\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"LlqaBMB8\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc:content-transfer-encoding;\n\tbh=6QbmNo2hcKh8DcYQbe+B/Vslu1c0qnEDZJXRctUgkqI=;\n\tb=LlqaBMB8Zx19vzQpmPayMHt4WOCNkf6vLR3IUYWw0AXVd2ikOT1ZnZXz9ME/iXaNBJ\n\tp89iUA+NVeNYaOrtR9BA4QxI0jFimSxI8y/RQgzKKE8cQxUC21hQQLfFWg8S0Eeeel5K\n\tnx2riAlGRoaSb8wsW7L0CUv0oqTajiaKRVkE/R42W/jTMn4nXY3OU/7vT63OT+ehy/Zj\n\tXrbpfbRtgV90oKss0iWPB2CqoFd0HTJWhkBZ0kzJxvDo8HVirm6AwMmoKvx/L2VmqTRj\n\tOxNN/q+H7jnk8BTIc5cW+UKhfjBHqVDKhd0Fu5jTNJU7AkPPBXwC3cjNXM8nh1MPlZSw\n\tTqQQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc:content-transfer-encoding;\n\tbh=6QbmNo2hcKh8DcYQbe+B/Vslu1c0qnEDZJXRctUgkqI=;\n\tb=kPYnifFIMhZVelSHbJ3mdgRQhcGeQPUEq8lq+zKKrfDb/fUeDnB1ZiaZU0z9GjsPdz\n\tBuncGhg1jKMbDBFcRpjCmgwGrJlcOHnMQ4647+Dxft6tM+M961R9RXOmkI9XM9PkC5r9\n\tY68m+DyEpn+2JaidSmIwUcw7BeM3fnlvFIBnqI/CvaHkDLWk8oWMlnm5rYMOLPiAJUV+\n\toEa7NeDZfSMRmOsV3L3c9bqko/IrvdInZ47fmPcpe1X9SkDLtJ5Js/Pe7WTy5+pewJmB\n\tWOcgrCMTalETIBUrQN2vI0y3C8J6GPD83mMJM473KRNnPESUe8Uj0cUyu6qjGTyElEjq\n\toXhQ==","X-Gm-Message-State":"AN3rC/6/RhX0j9lYoX4fXfhg1Z1VoP35XFhiVseLfVm3QpKW/ocVXJeK\n\tlYmY1JsqpvjrZJMiDVdIvDyvv8ajyw==","X-Received":"by 10.25.41.209 with SMTP id p200mr1020259lfp.55.1492084298098; \n\tThu, 13 Apr 2017 04:51:38 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<585eeaa5-52bd-7fe4-2179-3582570966e5@c-s.fr>","References":"<20170413070309.10497-1-bsingharora@gmail.com>\n\t\n\t<1492071568.8380.20.camel@gmail.com>\n\t<585eeaa5-52bd-7fe4-2179-3582570966e5@c-s.fr>","From":"Balbir Singh ","Date":"Thu, 13 Apr 2017 21:51:37 +1000","Message-ID":"","Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","To":"Christophe LEROY ","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"quoted-printable","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"\"open list:LINUX FOR POWERPC \\(32-BIT AND 64-BIT\\)\"\n\t","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1633513,"web_url":"http://patchwork.ozlabs.org/comment/1633513/","msgid":"<1492096766.sziet58dj7.astroid@naverao1-tp.none>","date":"2017-04-13T15:21:41","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":15185,"url":"http://patchwork.ozlabs.org/api/people/15185/","name":"Naveen N. Rao","email":"naveen.n.rao@linux.vnet.ibm.com"},"content":"Excerpts from Balbir Singh's message of April 13, 2017 12:33:\n> We expect to have these configs on by default, most\n> distros turn them off, its always good to have them on\n> so that we can use them and hopefully not break them\n\nThanks! I find myself constantly enabling these options.\n\n> \n> Signed-off-by: Balbir Singh \n> ---\n> arch/powerpc/configs/powernv_defconfig | 4 ++++\n> arch/powerpc/configs/ppc64_defconfig | 3 +++\n> arch/powerpc/configs/pseries_defconfig | 3 +++\n> 3 files changed, 10 insertions(+)\n> \n> diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc/configs/powernv_defconfig\n> index 0695ce0..eff3968 100644\n> --- a/arch/powerpc/configs/powernv_defconfig\n> +++ b/arch/powerpc/configs/powernv_defconfig\n> @@ -295,7 +295,11 @@ CONFIG_DEBUG_STACK_USAGE=y\n> CONFIG_DEBUG_STACKOVERFLOW=y\n> CONFIG_LOCKUP_DETECTOR=y\n> CONFIG_LATENCYTOP=y\n> +CONFIG_FTRACE=y\n> +CONFIG_FUNCTION_TRACER=y\n> +CONFIG_FUNCTION_GRAPH_TRACER=y\n> CONFIG_SCHED_TRACER=y\n> +CONFIG_FTRACE_SYSCALLS=y\n\nAny reason to not enable this for ppc64 and pseries defconfigs?\nApart from that, for this patch:\nAcked-by: Naveen N. Rao \n\n\n- Naveen\n\n> CONFIG_BLK_DEV_IO_TRACE=y\n> CONFIG_UPROBE_EVENT=y\n> CONFIG_CODE_PATCHING_SELFTEST=y\n> diff --git a/arch/powerpc/configs/ppc64_defconfig b/arch/powerpc/configs/ppc64_defconfig\n> index e353168f9..17c6c12 100644\n> --- a/arch/powerpc/configs/ppc64_defconfig\n> +++ b/arch/powerpc/configs/ppc64_defconfig\n> @@ -328,6 +328,9 @@ CONFIG_DEBUG_STACKOVERFLOW=y\n> CONFIG_LOCKUP_DETECTOR=y\n> CONFIG_DEBUG_MUTEXES=y\n> CONFIG_LATENCYTOP=y\n> +CONFIG_FTRACE=y\n> +CONFIG_FUNCTION_TRACER=y\n> +CONFIG_FUNCTION_GRAPH_TRACER=y\n> CONFIG_SCHED_TRACER=y\n> CONFIG_BLK_DEV_IO_TRACE=y\n> CONFIG_UPROBE_EVENT=y\n> diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/configs/pseries_defconfig\n> index 1a61aa2..a445960 100644\n> --- a/arch/powerpc/configs/pseries_defconfig\n> +++ b/arch/powerpc/configs/pseries_defconfig\n> @@ -293,6 +293,9 @@ CONFIG_DEBUG_STACK_USAGE=y\n> CONFIG_DEBUG_STACKOVERFLOW=y\n> CONFIG_LOCKUP_DETECTOR=y\n> CONFIG_LATENCYTOP=y\n> +CONFIG_FTRACE=y\n> +CONFIG_FUNCTION_TRACER=y\n> +CONFIG_FUNCTION_GRAPH_TRACER=y\n> CONFIG_SCHED_TRACER=y\n> CONFIG_BLK_DEV_IO_TRACE=y\n> CONFIG_UPROBE_EVENT=y\n> -- \n> 2.9.3\n> \n>","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w3l0h470bz9sN6\n\tfor ;\n\tFri, 14 Apr 2017 01:23:00 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w3l0h3NFXzDqJq\n\tfor ;\n\tFri, 14 Apr 2017 01:23:00 +1000 (AEST)","from mx0a-001b2d01.pphosted.com (mx0a-001b2d01.pphosted.com\n\t[148.163.156.1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w3kzZ4S3czDq5b\n\tfor ;\n\tFri, 14 Apr 2017 01:22:02 +1000 (AEST)","from pps.filterd (m0098410.ppops.net [127.0.0.1])\n\tby mx0a-001b2d01.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id\n\tv3DFEPAv113247\n\tfor ; Thu, 13 Apr 2017 11:21:52 -0400","from e28smtp05.in.ibm.com (e28smtp05.in.ibm.com [125.16.236.5])\n\tby mx0a-001b2d01.pphosted.com with ESMTP id 29t69a1aqw-1\n\t(version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT)\n\tfor ; Thu, 13 Apr 2017 11:21:52 -0400","from localhost\n\tby e28smtp05.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use\n\tOnly! Violators will be prosecuted\n\tfor from\n\t; Thu, 13 Apr 2017 20:51:48 +0530","from d28relay04.in.ibm.com (9.184.220.61)\n\tby e28smtp05.in.ibm.com (192.168.1.135) with IBM ESMTP SMTP Gateway:\n\tAuthorized Use Only! Violators will be prosecuted; \n\tThu, 13 Apr 2017 20:51:46 +0530","from d28av08.in.ibm.com (d28av08.in.ibm.com [9.184.220.148])\n\tby d28relay04.in.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id\n\tv3DFLkp018743336\n\tfor ; Thu, 13 Apr 2017 20:51:46 +0530","from d28av08.in.ibm.com (localhost [127.0.0.1])\n\tby d28av08.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id\n\tv3DFLj8b014896\n\tfor ; Thu, 13 Apr 2017 20:51:45 +0530","from localhost ([9.79.182.14])\n\tby d28av08.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVin) with ESMTP id\n\tv3DFLiad014892; Thu, 13 Apr 2017 20:51:44 +0530"],"Date":"Thu, 13 Apr 2017 15:21:41 +0000","From":"\"Naveen N. Rao\" ","Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","To":"Balbir Singh , mpe@ellerman.id.au","References":"<20170413070309.10497-1-bsingharora@gmail.com>","In-Reply-To":"<20170413070309.10497-1-bsingharora@gmail.com>","User-Agent":"astroid/0.8 (https://github.com/astroidmail/astroid)","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8; format=flowed","Content-Transfer-Encoding":"quoted-printable","X-TM-AS-MML":"disable","x-cbid":"17041315-0016-0000-0000-00000409444A","X-IBM-AV-DETECTION":"SAVI=unused REMOTE=unused XFE=unused","x-cbparentid":"17041315-0017-0000-0000-000028F2D0E6","Message-Id":"<1492096766.sziet58dj7.astroid@naverao1-tp.none>","X-Proofpoint-Virus-Version":"vendor=fsecure engine=2.50.10432:, ,\n\tdefinitions=2017-04-13_11:, , signatures=0","X-Proofpoint-Spam-Details":"rule=outbound_notspam policy=outbound score=0\n\tspamscore=0 suspectscore=0\n\tmalwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam\n\tadjust=0 reason=mlx scancount=1 engine=8.0.1-1702020001\n\tdefinitions=main-1704130130","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1637511,"web_url":"http://patchwork.ozlabs.org/comment/1637511/","msgid":"<20170419211346.5e228370@kryten>","date":"2017-04-19T11:13:46","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":1918,"url":"http://patchwork.ozlabs.org/api/people/1918/","name":"Anton Blanchard","email":"anton@samba.org"},"content":"Hi Balbir,\n\n> > FTRACE is quite CPU consumming, shouldn't it really be on by\n> > default ? \n> \n> It does some work at boot to NOP out function entry points at _mcount\n> locations. Is that what you are referring to? Or the overhead of the\n> code in terms of size? Most distro kernels have tracing on by default.\n> \n> The rest of the overhead is enablement based.\n\nUnfortunately the overhead is somewhat high without\nCONFIG_MPROFILE_KERNEL, and enabling that option will break old\ntoolchains. It would be great if we could automatically enable it based\non the toolchain.\n\nEven with CONFIG_MPROFILE_KERNEL enabled, we aren't noping out the\nredundant mflr at the start of each function.\n\nAnton","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w7KDC3KcKz9s0m\n\tfor ;\n\tWed, 19 Apr 2017 21:15:23 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w7KDC29BkzDqFq\n\tfor ;\n\tWed, 19 Apr 2017 21:15:23 +1000 (AEST)","from hr2.samba.org (hr2.samba.org [IPv6:2a01:4f8:192:486::147:1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w7KBb4Pq9zDq5W\n\tfor ;\n\tWed, 19 Apr 2017 21:13:59 +1000 (AEST)","from [127.0.0.2] (localhost [127.0.0.1])\n\tby hr2.samba.org with esmtpsa\n\t(TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)\n\t(Exim) id 1d0nYj-0002Vd-3c; Wed, 19 Apr 2017 11:14:05 +0000"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tsecure) header.d=samba.org header.i=@samba.org header.b=\"jynZvRZ0\";\n\tdkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tsecure) header.d=samba.org header.i=@samba.org header.b=\"jynZvRZ0\";\n\tdkim-atps=neutral","lists.ozlabs.org; dkim=pass (1024-bit key;\n\tsecure) header.d=samba.org header.i=@samba.org header.b=\"jynZvRZ0\"; \n\tdkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=samba.org;\n\ts=42627210; h=Message-ID:Cc:To:From:Date;\n\tbh=gG7cRbnk3nM+lfz00Dr85cLbz9+V8OtQ8h4ValdMiXw=;\n\tb=jynZvRZ0S8WByGiwMd2eJyDKvc\n\tVGG31xmVl3uokvj0nxDYgRdnbE3CJUJL44uBshWrsUJgRsIj7T/XfypUtGgn5Fb4efhyF1FeTDNyB\n\t8321XLYFQzQRss2MxISCs7kqtPRNdmMwUPxtilpORGzLZG6ERS/od6STTnIIG4xLfmOE=;","Date":"Wed, 19 Apr 2017 21:13:46 +1000","From":"Anton Blanchard ","To":"Balbir Singh ","Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","Message-ID":"<20170419211346.5e228370@kryten>","In-Reply-To":"<1492071568.8380.20.camel@gmail.com>","References":"<20170413070309.10497-1-bsingharora@gmail.com>\n\t\n\t<1492071568.8380.20.camel@gmail.com>","X-Mailer":"Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu)","MIME-Version":"1.0","Content-Type":"text/plain; charset=US-ASCII","Content-Transfer-Encoding":"7bit","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1637601,"web_url":"http://patchwork.ozlabs.org/comment/1637601/","msgid":"<1492606204.7284.1.camel@gmail.com>","date":"2017-04-19T12:50:04","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":9347,"url":"http://patchwork.ozlabs.org/api/people/9347/","name":"Balbir Singh","email":"bsingharora@gmail.com"},"content":"On Wed, 2017-04-19 at 21:13 +1000, Anton Blanchard wrote:\n> Hi Balbir,\n> \n> > > FTRACE is quite CPU consumming, shouldn't it really be on by\n> > > default ? \n> > \n> > It does some work at boot to NOP out function entry points at _mcount\n> > locations. Is that what you are referring to? Or the overhead of the\n> > code in terms of size? Most distro kernels have tracing on by default.\n> > \n> > The rest of the overhead is enablement based.\n> \n> Unfortunately the overhead is somewhat high without\n> CONFIG_MPROFILE_KERNEL, and enabling that option will break old\n> toolchains. It would be great if we could automatically enable it based\n> on the toolchain.\n> \n> Even with CONFIG_MPROFILE_KERNEL enabled, we aren't noping out the\n> redundant mflr at the start of each function.\n>\n\nVery good catch! I sent the enablement assuming that we want to have these\nenabled to ensure we generally have these enabled in the distro kernels\nand ideally want to test with these enabled and many of us turn them\non in any case. Do you see an issue with this being enabled by default?\nI presume most workloads run on kernels that have them enabled these days?\n\nCONFIG_MPROFILE_KERNEL depends on CC_USING_MPROFILE_KERNEL which is\nautomatically detected and is LE only.\n\nBalbir Singh","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w7MNX1tC6z9s0m\n\tfor ;\n\tWed, 19 Apr 2017 22:52:44 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w7MNX0p4vzDq8t\n\tfor ;\n\tWed, 19 Apr 2017 22:52:44 +1000 (AEST)","from mail-io0-x22c.google.com (mail-io0-x22c.google.com\n\t[IPv6:2607:f8b0:4001:c06::22c])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w7MLb6rTPzDq8M\n\tfor ;\n\tWed, 19 Apr 2017 22:51:03 +1000 (AEST)","by mail-io0-x22c.google.com with SMTP id o22so21051172iod.3\n\tfor ;\n\tWed, 19 Apr 2017 05:51:03 -0700 (PDT)","from 350D (14-202-194-140.static.tpgi.com.au. [14.202.194.140])\n\tby smtp.googlemail.com with ESMTPSA id\n\td3sm4657301pgc.37.2017.04.19.05.50.55\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tWed, 19 Apr 2017 05:51:00 -0700 (PDT)"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"gG04sRCq\"; dkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"gG04sRCq\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"gG04sRCq\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=message-id:subject:from:to:cc:date:in-reply-to:references\n\t:mime-version:content-transfer-encoding;\n\tbh=9/qLA03CaCsQ8znC4qFnDyBIrwMN/Qp9AXXAQiBSUHo=;\n\tb=gG04sRCqjFxxRH9w9rOAAYDtSco2rotIXTYL9P7BI6G26SBeYbhs/qegFcYdPWz2rA\n\tXMayq4DKke0TpnQysvjnSRZ6lxxoNbnRpcTyqli//kYE/fHU89+eqPO4r7SAPZE4jjrG\n\tZApdr92A2sf+YYAOR5eMHUoLk4aEM5FcfKTj9gj74+rj/rZCIZ3LsMJP44CQaYVsfSHt\n\tKBJ/fJuWNipyAMQtkrSY9otLz4GiD9OY7r4/xQazpK7kp02GhgkSkXD/R9dH1vEdaCAn\n\tuHhMNFrwpntAJxiTF2PyCYV22OECRdlvRz1kmdQNARYAH+pKVu+AA2JDQ169OFwD9qIS\n\t/thA==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to\n\t:references:mime-version:content-transfer-encoding;\n\tbh=9/qLA03CaCsQ8znC4qFnDyBIrwMN/Qp9AXXAQiBSUHo=;\n\tb=For/xi+twMtTUWGUekxRfAm2JwRuvOyHNUmHLCsc9n0WTxThSDLpSY/u06SyisPZED\n\tAZUmMSbkZopJdBKtdA59D0cKAxHRLbuZrX97YJBvt7/VkEKgOdsb6tbMpQkSOux3hIbk\n\tQ2YrXTOAZ1kEIetptNnZwj2lGDBfK/rj4vAF8c3i/LwEWpFVELyqTlWnwRFxkickrAyD\n\tk1leglUvutXTcuVi1Y81lYO0KD2Oww10+VOHkcGNrlCL6mR4trDLB33se8UUKxJJwl5u\n\tVTwWoBr1wVu+ToPkIW6jU1LAxNYUlFpw3cHNFjejMuI5J2erIgAUP9zZehFqSqQIzerh\n\tboBQ==","X-Gm-Message-State":"AN3rC/7EKVaIMwu4oqmO3ZAMdvOAUNRLeMaLX+Q9PPh6DuQ3ojRLrH8v\n\tQD7X+RO2k1iphw==","X-Received":"by 10.98.24.68 with SMTP id 65mr2891901pfy.191.1492606261458;\n\tWed, 19 Apr 2017 05:51:01 -0700 (PDT)","Message-ID":"<1492606204.7284.1.camel@gmail.com>","Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","From":"Balbir Singh ","To":"Anton Blanchard ","Date":"Wed, 19 Apr 2017 22:50:04 +1000","In-Reply-To":"<20170419211346.5e228370@kryten>","References":"<20170413070309.10497-1-bsingharora@gmail.com>\n\t\n\t<1492071568.8380.20.camel@gmail.com> <20170419211346.5e228370@kryten>","Content-Type":"text/plain; charset=\"UTF-8\"","X-Mailer":"Evolution 3.22.6-1ubuntu1 ","Mime-Version":"1.0","Content-Transfer-Encoding":"7bit","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1637726,"web_url":"http://patchwork.ozlabs.org/comment/1637726/","msgid":"<87lgqw5x2w.fsf@concordia.ellerman.id.au>","date":"2017-04-19T13:46:47","subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","submitter":{"id":46580,"url":"http://patchwork.ozlabs.org/api/people/46580/","name":"Michael Ellerman","email":"mpe@ellerman.id.au"},"content":"Balbir Singh writes:\n\n> On Wed, 2017-04-19 at 21:13 +1000, Anton Blanchard wrote:\n>> Hi Balbir,\n>> \n>> > > FTRACE is quite CPU consumming, shouldn't it really be on by\n>> > > default ? \n>> > \n>> > It does some work at boot to NOP out function entry points at _mcount\n>> > locations. Is that what you are referring to? Or the overhead of the\n>> > code in terms of size? Most distro kernels have tracing on by default.\n>> > \n>> > The rest of the overhead is enablement based.\n>> \n>> Unfortunately the overhead is somewhat high without\n>> CONFIG_MPROFILE_KERNEL, and enabling that option will break old\n>> toolchains. It would be great if we could automatically enable it based\n>> on the toolchain.\n>> \n>> Even with CONFIG_MPROFILE_KERNEL enabled, we aren't noping out the\n>> redundant mflr at the start of each function.\n>\n> Very good catch! I sent the enablement assuming that we want to have these\n> enabled to ensure we generally have these enabled in the distro kernels\n> and ideally want to test with these enabled and many of us turn them\n> on in any case. Do you see an issue with this being enabled by default?\n> I presume most workloads run on kernels that have them enabled these days?\n\nNo definitely not. Most distro kernels don't even have the code, it only\nwent into 4.6.\n\n> CONFIG_MPROFILE_KERNEL depends on CC_USING_MPROFILE_KERNEL which is\n\nNo it doesn't:\n\nconfig MPROFILE_KERNEL\n\tdepends on PPC64 && CPU_LITTLE_ENDIAN\n\tdef_bool !DISABLE_MPROFILE_KERNEL\n\n> automatically detected and is LE only.\n\nIt's automatically detected at build time, which is too late, so our\nonly option is to break the build:\n\nifdef CONFIG_MPROFILE_KERNEL\n ifeq ($(shell $(srctree)/arch/powerpc/scripts/gcc-check-mprofile-kernel.sh $(CC) -I$(srctree)/include -D__KERNEL__),OK)\n CC_FLAGS_FTRACE := -pg -mprofile-kernel\n KBUILD_CPPFLAGS += -DCC_USING_MPROFILE_KERNEL\n else\n # If the user asked for mprofile-kernel but the toolchain doesn't\n # support it, emit a warning and deliberately break the build later\n # with mprofile-kernel-not-supported. We would prefer to make this an\n # error right here, but then the user would never be able to run\n # oldconfig to change their configuration.\n $(warning Compiler does not support mprofile-kernel, set CONFIG_DISABLE_MPROFILE_KERNEL)\n CC_FLAGS_FTRACE := -mprofile-kernel-not-supported\n endif\nendif\n\n\nSo in short it's a big PITA.\n\ncheers","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3w7Nc35xnYz9ryk\n\tfor ;\n\tWed, 19 Apr 2017 23:47:47 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3w7Nc359G8zDqRc\n\tfor ;\n\tWed, 19 Apr 2017 23:47:47 +1000 (AEST)","from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3w7NZx1YRgzDq8M\n\tfor ;\n\tWed, 19 Apr 2017 23:46:49 +1000 (AEST)","from authenticated.ozlabs.org (localhost [127.0.0.1])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPSA id 3w7NZw6Sx0z9ryk;\n\tWed, 19 Apr 2017 23:46:48 +1000 (AEST)"],"From":"Michael Ellerman ","To":"Balbir Singh , Anton Blanchard ","Subject":"Re: [PATCH] powerpc/configs: Enable function trace by default","In-Reply-To":"<1492606204.7284.1.camel@gmail.com>","References":"<20170413070309.10497-1-bsingharora@gmail.com>\n\t\n\t<1492071568.8380.20.camel@gmail.com> <20170419211346.5e228370@kryten>\n\t<1492606204.7284.1.camel@gmail.com>","User-Agent":"Notmuch/0.21 (https://notmuchmail.org)","Date":"Wed, 19 Apr 2017 23:46:47 +1000","Message-ID":"<87lgqw5x2w.fsf@concordia.ellerman.id.au>","MIME-Version":"1.0","Content-Type":"text/plain","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1760861,"web_url":"http://patchwork.ozlabs.org/comment/1760861/","msgid":"<3xjgKz5BGDz9t2Q@ozlabs.org>","date":"2017-08-31T11:35:51","subject":"Re: powerpc/configs: Enable function trace by default","submitter":{"id":69509,"url":"http://patchwork.ozlabs.org/api/people/69509/","name":"Michael Ellerman","email":"patch-notifications@ellerman.id.au"},"content":"On Thu, 2017-04-13 at 07:03:09 UTC, Balbir Singh wrote:\n> We expect to have these configs on by default, most\n> distros turn them off, its always good to have them on\n> so that we can use them and hopefully not break them\n> \n> Signed-off-by: Balbir Singh \n> Acked-by: Naveen N. Rao \n\nApplied to powerpc next, thanks.\n\nhttps://git.kernel.org/powerpc/c/539df7fcb303f2cbe8021a27839928\n\ncheers","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xjgTJ5HJMz9t1t\n\tfor ;\n\tThu, 31 Aug 2017 21:42:12 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xjgTJ4PJ8zDqgB\n\tfor ;\n\tThu, 31 Aug 2017 21:42:12 +1000 (AEST)","from ozlabs.org (bilbo.ozlabs.org [103.22.144.67])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xjgKz5klGzDqGZ\n\tfor ;\n\tThu, 31 Aug 2017 21:35:51 +1000 (AEST)","by ozlabs.org (Postfix, from userid 1034)\n\tid 3xjgKz5BGDz9t2Q; Thu, 31 Aug 2017 21:35:51 +1000 (AEST)"],"X-powerpc-patch-notification":"thanks","X-powerpc-patch-commit":"539df7fcb303f2cbe8021a27839928485937cd6b","In-Reply-To":"<20170413070309.10497-1-bsingharora@gmail.com>","To":"Balbir Singh ","From":"Michael Ellerman ","Subject":"Re: powerpc/configs: Enable function trace by default","Message-Id":"<3xjgKz5BGDz9t2Q@ozlabs.org>","Date":"Thu, 31 Aug 2017 21:35:51 +1000 (AEST)","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}}]