[tip:perf/live] perf report: Introduce special handling for pipe input
in [Kernel]
|
Prev: [acpi:battery?] kernel oops on boot
Next: patch iwlwifi-counting-number-of-tfds-can-be-free-for-4965.patch added to 2.6.33-stable tree
From: tip-bot for Tom Zanussi on 14 Apr 2010 06:10 Commit-ID: 46656ac7fb3252f8a3db29b18638e0e8067849ba Gitweb: http://git.kernel.org/tip/46656ac7fb3252f8a3db29b18638e0e8067849ba Author: Tom Zanussi <tzanussi(a)gmail.com> AuthorDate: Thu, 1 Apr 2010 23:59:17 -0500 Committer: Ingo Molnar <mingo(a)elte.hu> CommitDate: Wed, 14 Apr 2010 11:56:06 +0200 perf report: Introduce special handling for pipe input Adds special treatment for stdin - if the user specifies '-i -' to perf report, the intent is that the event stream be written to stdin rather than from a disk file. The actual handling of the '-' filename is done by the session; this just adds a signal handler to stop reporting, and turns off interference by the pager. Signed-off-by: Tom Zanussi <tzanussi(a)gmail.com> Acked-by: Thomas Gleixner <tglx(a)linutronix.de> Cc: fweisbec(a)gmail.com Cc: rostedt(a)goodmis.org Cc: k-keiichi(a)bx.jp.nec.com Cc: acme(a)ghostprotocols.net LKML-Reference: <1270184365-8281-4-git-send-email-tzanussi(a)gmail.com> Signed-off-by: Ingo Molnar <mingo(a)elte.hu> --- tools/perf/builtin-report.c | 12 +++++++++++- 1 files changed, 11 insertions(+), 1 deletions(-) diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index daee082..00b358f 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -269,6 +269,13 @@ static struct perf_event_ops event_ops = { .read = process_read_event, }; +extern volatile int session_done; + +static void sig_handler(int sig __attribute__((__unused__))) +{ + session_done = 1; +} + static int __cmd_report(void) { int ret = -EINVAL; @@ -276,6 +283,8 @@ static int __cmd_report(void) struct rb_node *next; const char *help = "For a higher level overview, try: perf report --sort comm,dso"; + signal(SIGINT, sig_handler); + session = perf_session__new(input_name, O_RDONLY, force); if (session == NULL) return -ENOMEM; @@ -465,7 +474,8 @@ int cmd_report(int argc, const char **argv, const char *prefix __used) { argc = parse_options(argc, argv, options, report_usage, 0); - setup_browser(); + if (strcmp(input_name, "-") != 0) + setup_browser(); if (symbol__init() < 0) return -1; -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |