From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932066AbcGMTRT (ORCPT ); Wed, 13 Jul 2016 15:17:19 -0400 Received: from smtprelay0063.hostedemail.com ([216.40.44.63]:49134 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751580AbcGMTRI (ORCPT ); Wed, 13 Jul 2016 15:17:08 -0400 X-Session-Marker: 726F737465647440676F6F646D69732E6F7267 X-Spam-Summary: 2,0,0,,d41d8cd98f00b204,rostedt@goodmis.org,:::::::::::::::::::::::::::,RULES_HIT:41:355:379:541:599:800:960:966:968:973:988:989:1260:1277:1311:1313:1314:1345:1359:1431:1437:1515:1516:1518:1535:1543:1593:1594:1605:1711:1730:1747:1777:1792:2194:2196:2199:2200:2393:2553:2559:2562:2693:3138:3139:3140:3141:3142:3311:3622:3865:3866:3867:3868:3870:3871:3872:3873:3874:4250:4321:4385:5007:6119:6261:6742:7875:7903:8660:8957:9108:10004:10400:10848:10903:10967:11232:11658:11914:12296:12517:12519:12663:12740:13141:13148:13161:13180:13229:13230:13439:13618:14096:14097:14181:14659:14721:21080:21324:21433:30012:30054:30064:30070:30079: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: bulb64_3d7813189fc32 X-Filterd-Recvd-Size: 5481 Date: Wed, 13 Jul 2016 15:17:00 -0400 From: Steven Rostedt To: Arnaldo Carvalho de Melo Cc: Ingo Molnar , linux-kernel@vger.kernel.org, Arnaldo Carvalho de Melo , Adrian Hunter , David Ahern , Jiri Olsa , Namhyung Kim , Wang Nan , Clark Williams , Mauro Carvalho Chehab , Arjan van de Ven , Alexandra Yates , Nivedita Swaminathan Subject: Re: [PATCH 29/66] tools lib traceevent: Use str_error_r() Message-ID: <20160713151700.3cc8f65e@gandalf.local.home> In-Reply-To: <20160713135125.GA27879@kernel.org> References: <1468363241-14555-1-git-send-email-acme@kernel.org> <1468363241-14555-30-git-send-email-acme@kernel.org> <20160712191108.63d23fae@gandalf.local.home> <20160712231424.GA26691@kernel.org> <20160712192519.601b1fae@gandalf.local.home> <20160712233218.GB26691@kernel.org> <20160713083001.1dfa2b0e@gandalf.local.home> <20160713135125.GA27879@kernel.org> X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.30; 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, 13 Jul 2016 10:51:25 -0300 Arnaldo Carvalho de Melo wrote: > Em Wed, Jul 13, 2016 at 08:30:01AM -0400, Steven Rostedt escreveu: > > On Tue, 12 Jul 2016 20:32:18 -0300 > > Arnaldo Carvalho de Melo wrote: > > > > > Forgot about the out of tree copy :-\ > > > > > Yeah, we really need to make this into a real library. I haven't had > > > > the time to do that. Hopefully in August I can talk with some people at > > > > What exactly do you mean by that? To grab a copy of what is in tools/ > > > and have it turned into a library somewhere else? > > > > Or to freeze its interfaces and create an .so with a committed to ABI? > > > Yes, this is what we need to do. > > > > I kind of like the way it is now... :-) > > > You like the current ABI? Or the fact that we have 4 tools with each > > I like the lack of an ABI, i.e. I like to use the same rationale for > when a out of tree driver breaks because we improve kernel internals. As > soon as we commit to an ABI for those libraries, that becomes way more > difficult. Yes, ABI is hard. But it can be done. Let's not get lazy. The code came from external sources to begin with. I've been working hard to try to keep the ABI fixed. There's still a few things I would like to change before doing it officially, and that's because this code is quite mature now, and I have an idea of what to change. One thing I want to do is change the event_format structure to "pevent_event" and the format_field to "pevent_field" just to be more consistent. There's probably a few other changes as well. But once they are done, I think we should get this out into a library. > > And since we're still adding more and more stuff (write_backward, eBPF > for kernel and userspace, etc, etc) I fear lots are still very much in > flux to commit to something like that. We can always add new functions. The old ones will have to be maintained. Anyway, I'm only looking to what is being used externally from perf. We can slowly add functionality. The .so releases should only increment the version if something new is added. I refuse to follow the gtk crap that they find ways to remove or modify existing functions every single release. > > So if we know what are the needs of those out of the tree tools (can't > they be brought in tree? I'm digressing, couldn't resist, sorry ;-)), Well, I Cc'd some of the maintainers. We can see what they think. > then we could, if absolutely required, try, without much enthusiasm, > relutanctly, see what could be done besides what has been already > underway (moving stuff out of tools/perf and into tools/lib). Note, I'm only looking at making a library out of what is currently in tools/lib. We don't need to move stuff out from perf unless you want to. There's a trace-cmd library I want to have as well, which I would love to have in the kernel tools/lib directory too. I'd call it libftrace.so. This would include the ways to parse the ftrace ring buffer, as well as parsing and creating trace.dat files. > > It would add more burden to me for perhaps reducing the burden to who > would use those libraries. Well the event-parse.* code has already been pretty stable. I do want to do the renames as I mentioned above before making it official. But once that is done, I would want us to get a libtraceevent.so that distros can provide. > > > their own copies of the library? (and maybe even more) > > Which ones? perf, trace-cmd/kernelshark, powertop and ras-daemon -- Steve > > - Arnaldo > > > > > LinuxCon to see the best way to go about doing that. > > > > > Anyway, I may just take that file and port it to trace-cmd. > > > > Yeah, that would solve this specific case. > > > OK, I'll do this.