Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751405AbdCPFtb (ORCPT ); Thu, 16 Mar 2017 01:49:31 -0400 Received: from szxga03-in.huawei.com ([45.249.212.189]:4414 "EHLO dggrg03-dlp.huawei.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751127AbdCPFt3 (ORCPT ); Thu, 16 Mar 2017 01:49:29 -0400 Subject: Re: change uprobe_events default ? Was: [PATCH] perf: Rename CONFIG_[UK]PROBE_EVENT to CONFIG_[UK]PROBE_EVENTS To: Arnaldo Carvalho de Melo , Alexei Starovoitov References: <20170216060050.20866-1-anton@ozlabs.org> <20170316005817.GA6805@ast-mbp.thefacebook.com> <20170316010651.GP12825@kernel.org> CC: Anton Blanchard , David Miller , Peter Zijlstra , Ingo Molnar , "Hemant Kumar" , Masami Hiramatsu , Steven Rostedt , Srikar Dronamraju , Namhyung Kim , David Ahern , Jiri Olsa , , , From: "Wangnan (F)" Message-ID: <5bdd6b73-136b-aafd-3ed3-2a1970310b63@huawei.com> Date: Thu, 16 Mar 2017 13:46:54 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <20170316010651.GP12825@kernel.org> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.111.194.139] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020202.58CA26DD.0100,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 00be02700f86b5a6897d4063c122dfc5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1024 Lines: 30 On 2017/3/16 9:06, Arnaldo Carvalho de Melo wrote: > Added more people to the CC list. > > Em Wed, Mar 15, 2017 at 05:58:19PM -0700, Alexei Starovoitov escreveu: >> On Thu, Feb 16, 2017 at 05:00:50PM +1100, Anton Blanchard wrote: >>> We have uses of CONFIG_UPROBE_EVENT and CONFIG_KPROBE_EVENT as >>> well as CONFIG_UPROBE_EVENTS and CONFIG_KPROBE_EVENTS. Consistently >>> use the plurals. > >> this rename made me notice that UPROBE_EVENTS still defaults to 'n'. >> this is key feature that all distros enable, so having default 'n' >> is kinda saying that it's not something that should be turned on >> or used widely. which is obviously not the case. >> imo it's time to change it to 'y'. >> Thoughts? > Agreed, I also found it strange that it was disabled by default when I > recently did a 'make oldconfig' :-\ > > - Arnaldo +1. Some internal distros in my company now have started discussing supporting user space probing. Turning uprobes on for upstream kernel helps us saving a lot of discussion. Thank you.