From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965725AbbKDUcn (ORCPT ); Wed, 4 Nov 2015 15:32:43 -0500 Received: from smtprelay0054.hostedemail.com ([216.40.44.54]:40204 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S965377AbbKDUcm (ORCPT ); Wed, 4 Nov 2015 15:32:42 -0500 X-Session-Marker: 726F737465647440676F6F646D69732E6F7267 X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,rostedt@goodmis.org,:::::,RULES_HIT:41:355:379:541:599:800:960:968:973:988:989:1260:1277:1311:1313:1314:1345:1359:1437:1515:1516:1518:1534:1541:1593:1594:1711:1730:1747:1777:1792:2194:2198:2199:2200:2393:2553:2559:2562:2693:2731:3138:3139:3140:3141:3142:3353:3622:3865:3866:3867:3868:3870:3871:3872:3874:4250:5007:6261:7514:7875:7903:9108:9707:10004:10400:10848:10967:11026:11232:11658:11914:12296:12438:12517:12519:12740:13069:13161:13229:13255:13311:13357:14096:14097:14659:21080:30012:30036:30054:30070:30090:30091,0,RBL:none,CacheIP:none,Bayesian:0.5,0.5,0.5,Netcheck:none,DomainCache:0,MSF:not bulk,SPF:fn,MSBL:0,DNSBL:none,Custom_rules:0:0:0,LFtime:3,LUA_SUMMARY:none X-HE-Tag: pet52_17404d4d1e624 X-Filterd-Recvd-Size: 2501 Date: Wed, 4 Nov 2015 15:32:38 -0500 From: Steven Rostedt To: Greg Kroah-Hartman Cc: Jiaxing Wang , linux-kernel@vger.kernel.org Subject: Re: [PATCH RESEND] tracing: Make tracing work when debugfs is not compiled or initialized. Message-ID: <20151104153238.1fb3eb9d@gandalf.local.home> In-Reply-To: <20151104185442.GA19152@kroah.com> References: <20151102091637.0fcc05b2@gandalf.local.home> <1446599478-8723-1-git-send-email-hello.wjx@gmail.com> <20151104100339.583e1bc9@gandalf.local.home> <20151104185442.GA19152@kroah.com> X-Mailer: Claws Mail 3.12.0 (GTK+ 2.24.28; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 4 Nov 2015 10:54:42 -0800 Greg Kroah-Hartman wrote: > On Wed, Nov 04, 2015 at 10:03:39AM -0500, Steven Rostedt wrote: > > On Wed, 4 Nov 2015 09:11:18 +0800 > > Jiaxing Wang wrote: > > > > > Currently tracing_init_dentry() returns -ENODEV when debugfs is not > > > initialized, which causes tracefs not populated with tracing files and > > > directories, so we will get an empty directory even after we manually > > > mount tracefs. > > > > > > We can make tracing_init_dentry() return NULL as long as tracefs > > > is initialized and get a populated tracefs. > > > > > > We also need to make global_trace.dir not NULL in order to pass the checks > > > in tracing_get_dentry() and add_tracer_options(). > > > > > > Also added stub debugfs_create_automount() for when debugfs is not > > > configured in. > > The debugfs change should be split out into a separate patch, which I'll > be glad to take through my tree, it isn't dependant on the tracing > code at all. > I guess the tracing code is dependent on that change. As the tracing code will then be compiled without the select DEBUGFS, and will fail to compile if that change is not there. That said. I think it's best to split it up, and have the debugfs change go through your tree, and the other part go through mine without the remove of the "select DEBUGFS". After both are in mainline, then we can remove the select statement. -- Steve