From: Steven Rostedt <rostedt@goodmis.org>
To: LKML <linux-kernel@vger.kernel.org>, <linux-trace-users@vger.kernel.org>
Cc: John Kacur <jkacur@redhat.com>,
Christian Neukirchen <chneukirchen@gmail.com>,
Troy Engel <troyengel@gmail.com>
Subject: Re: [ANNOUNCE] trace-cmd 2.5.2 (and others)
Date: Fri, 6 Feb 2015 09:35:21 -0500 [thread overview]
Message-ID: <20150206093521.26348f7a@gandalf.local.home> (raw)
In-Reply-To: <20150205173237.673691d0@gandalf.local.home>
On Thu, 5 Feb 2015 17:32:37 -0500
Steven Rostedt <rostedt@goodmis.org> wrote:
>
> You can get trace-cmd from:
>
> https://git.kernel.org/cgit/linux/kernel/git/rostedt/trace-cmd.git/
>
>
> Tag: trace-cmd-v2.5.2
>
It has been reported to me that version 2.5.2 broke the build of
kernelshark. I just fixed that and pushed to master, and the
trace-cmd-stable-v2.5 branch.
I added the tag trace-cmd-v2.5.2-1 that holds the change. I did not
make a 2.5.3 because no functionality change or fixes have been done to
trace-cmd itself. Only a fix for kernelshark was made.
-- Steve
>From cd0d29f41041d627fcc0d0446bfca697c20ee3c3 Mon Sep 17 00:00:00 2001
Subject: [PATCH] trace-cmd build: Add trace-hooks.o to library files
kernelshark links against libtracecmd.a. The hook functions are now
part of that, but trace-hooks.o was only linked to the trace-cmd object
itself and not the library. This broke the build of kernelshark.
Reported-by: Christian Neukirchen <chneukirchen@gmail.com>
Reported-by: Troy Engel <troyengel@gmail.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
---
Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index 1e9dc44e6011..55930eeaae12 100644
--- a/Makefile
+++ b/Makefile
@@ -308,7 +308,7 @@ TRACE_GUI_OBJS = trace-filter.o trace-compat.o trace-filter-hash.o trace-dialog.
trace-xml.o
TRACE_CMD_OBJS = trace-cmd.o trace-record.o trace-read.o trace-split.o trace-listen.o \
trace-stack.o trace-hist.o trace-mem.o trace-snapshot.o trace-stat.o \
- trace-hash.o trace-profile.o trace-stream.o trace-hooks.o
+ trace-hash.o trace-profile.o trace-stream.o
TRACE_VIEW_OBJS = trace-view.o trace-view-store.o
TRACE_GRAPH_OBJS = trace-graph.o trace-plot.o trace-plot-cpu.o trace-plot-task.o
TRACE_VIEW_MAIN_OBJS = trace-view-main.o $(TRACE_VIEW_OBJS) $(TRACE_GUI_OBJS)
@@ -320,7 +320,7 @@ PEVENT_LIB_OBJS = event-parse.o trace-seq.o parse-filter.o parse-utils.o
TCMD_LIB_OBJS = $(PEVENT_LIB_OBJS) trace-util.o trace-input.o trace-ftrace.o \
trace-output.o trace-record.o trace-recorder.o \
trace-restore.o trace-usage.o trace-blk-hack.o \
- kbuffer-parse.o event-plugin.o
+ kbuffer-parse.o event-plugin.o trace-hooks.o
PLUGIN_OBJS =
PLUGIN_OBJS += plugin_jbd2.o
--
2.1.4
prev parent reply other threads:[~2015-02-06 14:35 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-05 22:32 Steven Rostedt
2015-02-06 14:35 ` Steven Rostedt [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150206093521.26348f7a@gandalf.local.home \
--to=rostedt@goodmis.org \
--cc=chneukirchen@gmail.com \
--cc=jkacur@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-trace-users@vger.kernel.org \
--cc=troyengel@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®