blob: 483cb9466444a1d1ee0e2a215f071c221c53c7bb [file] [log] [blame]
Ingo Molnar8035e422009-06-06 15:19:13 +02001/*
2 * builtin-annotate.c
3 *
4 * Builtin annotate command: Analyze the perf.data input file,
5 * look up and read DSOs and symbol information and display
6 * a histogram of results, along various sorting keys.
7 */
8#include "builtin.h"
9
10#include "util/util.h"
Ingo Molnar8035e422009-06-06 15:19:13 +020011#include "util/color.h"
Arnaldo Carvalho de Melo5da50252009-07-01 14:46:08 -030012#include <linux/list.h>
Ingo Molnar8035e422009-06-06 15:19:13 +020013#include "util/cache.h"
Arnaldo Carvalho de Melo43cbcd82009-07-01 12:28:37 -030014#include <linux/rbtree.h>
Ingo Molnar8035e422009-06-06 15:19:13 +020015#include "util/symbol.h"
Ingo Molnar8035e422009-06-06 15:19:13 +020016
17#include "perf.h"
Frederic Weisbecker8f288272009-08-16 22:05:48 +020018#include "util/debug.h"
Ingo Molnar8035e422009-06-06 15:19:13 +020019
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -030020#include "util/evlist.h"
21#include "util/evsel.h"
Arnaldo Carvalho de Melo78f7def2011-02-04 09:45:46 -020022#include "util/annotate.h"
Arnaldo Carvalho de Melo62daacb2009-11-27 16:29:22 -020023#include "util/event.h"
Ingo Molnar8035e422009-06-06 15:19:13 +020024#include "util/parse-options.h"
25#include "util/parse-events.h"
Frederic Weisbecker6baa0a52009-08-14 12:21:53 +020026#include "util/thread.h"
John Kacurdd68ada2009-09-24 18:02:49 +020027#include "util/sort.h"
John Kacur3d1d07e2009-09-28 15:32:55 +020028#include "util/hist.h"
Arnaldo Carvalho de Melo94c744b2009-12-11 21:24:02 -020029#include "util/session.h"
Ingo Molnar8035e422009-06-06 15:19:13 +020030
Anton Blanchard5d67be92011-07-04 21:57:50 +100031#include <linux/bitmap.h>
32
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -020033struct perf_annotate {
34 struct perf_event_ops ops;
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -020035 char const *input_name;
36 bool force, use_tui, use_stdio;
37 bool full_paths;
38 bool print_line;
39 const char *sym_hist_filter;
40 const char *cpu_list;
41 DECLARE_BITMAP(cpu_bitmap, MAX_NR_CPUS);
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -020042};
Anton Blanchard5d67be92011-07-04 21:57:50 +100043
Arnaldo Carvalho de Melod04b35f2011-11-11 22:17:32 -020044static int perf_evsel__add_sample(struct perf_evsel *evsel,
45 struct perf_sample *sample,
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -020046 struct addr_location *al,
47 struct perf_annotate *ann)
Ingo Molnar8035e422009-06-06 15:19:13 +020048{
Arnaldo Carvalho de Melo628ada02010-02-25 12:57:40 -030049 struct hist_entry *he;
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -030050 int ret;
Arnaldo Carvalho de Melo628ada02010-02-25 12:57:40 -030051
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -020052 if (ann->sym_hist_filter != NULL &&
53 (al->sym == NULL ||
54 strcmp(ann->sym_hist_filter, al->sym->name) != 0)) {
Arnaldo Carvalho de Melo628ada02010-02-25 12:57:40 -030055 /* We're only interested in a symbol named sym_hist_filter */
56 if (al->sym != NULL) {
57 rb_erase(&al->sym->rb_node,
58 &al->map->dso->symbols[al->map->type]);
59 symbol__delete(al->sym);
60 }
61 return 0;
62 }
63
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -030064 he = __hists__add_entry(&evsel->hists, al, NULL, 1);
Arnaldo Carvalho de Melo9735abf2009-10-03 10:42:45 -030065 if (he == NULL)
Ingo Molnar8035e422009-06-06 15:19:13 +020066 return -ENOMEM;
Arnaldo Carvalho de Melo628ada02010-02-25 12:57:40 -030067
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -030068 ret = 0;
Arnaldo Carvalho de Melo2f525d02011-02-04 13:43:24 -020069 if (he->ms.sym != NULL) {
Arnaldo Carvalho de Melo2f525d02011-02-04 13:43:24 -020070 struct annotation *notes = symbol__annotation(he->ms.sym);
Arnaldo Carvalho de Melod04b35f2011-11-11 22:17:32 -020071 if (notes->src == NULL && symbol__alloc_hist(he->ms.sym) < 0)
Arnaldo Carvalho de Melo2f525d02011-02-04 13:43:24 -020072 return -ENOMEM;
73
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -030074 ret = hist_entry__inc_addr_samples(he, evsel->idx, al->addr);
Arnaldo Carvalho de Melo2f525d02011-02-04 13:43:24 -020075 }
76
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -030077 evsel->hists.stats.total_period += sample->period;
78 hists__inc_nr_events(&evsel->hists, PERF_RECORD_SAMPLE);
79 return ret;
Ingo Molnar8035e422009-06-06 15:19:13 +020080}
81
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -020082static int process_sample_event(struct perf_event_ops *ops,
83 union perf_event *event,
Arnaldo Carvalho de Melo8115d602011-01-29 14:01:45 -020084 struct perf_sample *sample,
Arnaldo Carvalho de Melo9e69c212011-03-15 15:44:01 -030085 struct perf_evsel *evsel,
Arnaldo Carvalho de Melo640c03c2010-12-02 14:10:21 -020086 struct perf_session *session)
Ingo Molnar8035e422009-06-06 15:19:13 +020087{
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -020088 struct perf_annotate *ann = container_of(ops, struct perf_annotate, ops);
Arnaldo Carvalho de Melo1ed091c2009-11-27 16:29:23 -020089 struct addr_location al;
Frederic Weisbecker6baa0a52009-08-14 12:21:53 +020090
Arnaldo Carvalho de Meloce6f4fa2011-02-08 13:27:39 -020091 if (perf_event__preprocess_sample(event, session, &al, sample,
92 symbol__annotate_init) < 0) {
Arnaldo Carvalho de Melo29a9f662010-02-03 16:52:06 -020093 pr_warning("problem processing %d event, skipping it.\n",
94 event->header.type);
Ingo Molnar8035e422009-06-06 15:19:13 +020095 return -1;
96 }
97
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -020098 if (ann->cpu_list && !test_bit(sample->cpu, ann->cpu_bitmap))
Anton Blanchard5d67be92011-07-04 21:57:50 +100099 return 0;
100
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200101 if (!al.filtered && perf_evsel__add_sample(evsel, sample, &al, ann)) {
Arnaldo Carvalho de Melo29a9f662010-02-03 16:52:06 -0200102 pr_warning("problem incrementing symbol count, "
103 "skipping event\n");
Arnaldo Carvalho de Meloec218fc2009-10-03 20:30:48 -0300104 return -1;
Ingo Molnar8035e422009-06-06 15:19:13 +0200105 }
Ingo Molnar8035e422009-06-06 15:19:13 +0200106
107 return 0;
108}
109
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200110static int hist_entry__tty_annotate(struct hist_entry *he, int evidx,
111 struct perf_annotate *ann)
Ingo Molnar0b73da32009-06-06 15:48:52 +0200112{
Arnaldo Carvalho de Melo2f525d02011-02-04 13:43:24 -0200113 return symbol__tty_annotate(he->ms.sym, he->ms.map, evidx,
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200114 ann->print_line, ann->full_paths, 0, 0);
Ingo Molnar0b73da32009-06-06 15:48:52 +0200115}
116
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200117static void hists__find_annotations(struct hists *self, int evidx,
118 struct perf_annotate *ann)
Ingo Molnar0b73da32009-06-06 15:48:52 +0200119{
Arnaldo Carvalho de Melob50e0032010-08-11 10:07:43 -0300120 struct rb_node *nd = rb_first(&self->entries), *next;
Arnaldo Carvalho de Melocf958002011-10-20 16:59:15 -0200121 int key = K_RIGHT;
Ingo Molnar0b73da32009-06-06 15:48:52 +0200122
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300123 while (nd) {
Arnaldo Carvalho de Meloed52ce22009-10-19 17:17:57 -0200124 struct hist_entry *he = rb_entry(nd, struct hist_entry, rb_node);
Arnaldo Carvalho de Melo78f7def2011-02-04 09:45:46 -0200125 struct annotation *notes;
Ingo Molnar0b73da32009-06-06 15:48:52 +0200126
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300127 if (he->ms.sym == NULL || he->ms.map->dso->annotate_warned)
128 goto find_next;
Ingo Molnar0b73da32009-06-06 15:48:52 +0200129
Arnaldo Carvalho de Melo78f7def2011-02-04 09:45:46 -0200130 notes = symbol__annotation(he->ms.sym);
Arnaldo Carvalho de Meloce6f4fa2011-02-08 13:27:39 -0200131 if (notes->src == NULL) {
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300132find_next:
Arnaldo Carvalho de Melocf958002011-10-20 16:59:15 -0200133 if (key == K_LEFT)
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300134 nd = rb_prev(nd);
135 else
136 nd = rb_next(nd);
Arnaldo Carvalho de Meloe4204992009-10-20 14:25:40 -0200137 continue;
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300138 }
Arnaldo Carvalho de Meloe4204992009-10-20 14:25:40 -0200139
Arnaldo Carvalho de Melo62e34362010-05-26 13:22:26 -0300140 if (use_browser > 0) {
Arnaldo Carvalho de Melod04b35f2011-11-11 22:17:32 -0200141 key = hist_entry__tui_annotate(he, evidx, NULL, NULL, 0);
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300142 switch (key) {
Arnaldo Carvalho de Melocf958002011-10-20 16:59:15 -0200143 case K_RIGHT:
Arnaldo Carvalho de Melob50e0032010-08-11 10:07:43 -0300144 next = rb_next(nd);
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300145 break;
Arnaldo Carvalho de Melocf958002011-10-20 16:59:15 -0200146 case K_LEFT:
Arnaldo Carvalho de Melob50e0032010-08-11 10:07:43 -0300147 next = rb_prev(nd);
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300148 break;
Arnaldo Carvalho de Melob50e0032010-08-11 10:07:43 -0300149 default:
150 return;
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300151 }
Arnaldo Carvalho de Melob50e0032010-08-11 10:07:43 -0300152
153 if (next != NULL)
154 nd = next;
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300155 } else {
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200156 hist_entry__tty_annotate(he, evidx, ann);
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300157 nd = rb_next(nd);
158 /*
159 * Since we have a hist_entry per IP for the same
Arnaldo Carvalho de Meloce6f4fa2011-02-08 13:27:39 -0200160 * symbol, free he->ms.sym->src to signal we already
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300161 * processed this symbol.
162 */
Arnaldo Carvalho de Meloce6f4fa2011-02-08 13:27:39 -0200163 free(notes->src);
164 notes->src = NULL;
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300165 }
Ingo Molnar0b73da32009-06-06 15:48:52 +0200166 }
Ingo Molnar0b73da32009-06-06 15:48:52 +0200167}
168
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200169static int __cmd_annotate(struct perf_annotate *ann)
Ingo Molnar8035e422009-06-06 15:19:13 +0200170{
Li Zefanbab81b62009-12-01 14:04:49 +0800171 int ret;
Arnaldo Carvalho de Melo75be6cf2009-12-15 20:04:39 -0200172 struct perf_session *session;
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -0300173 struct perf_evsel *pos;
174 u64 total_nr_samples;
Ingo Molnar8035e422009-06-06 15:19:13 +0200175
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200176 session = perf_session__new(ann->input_name, O_RDONLY,
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200177 ann->force, false, &ann->ops);
Arnaldo Carvalho de Melo94c744b2009-12-11 21:24:02 -0200178 if (session == NULL)
179 return -ENOMEM;
180
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200181 if (ann->cpu_list) {
182 ret = perf_session__cpu_bitmap(session, ann->cpu_list,
183 ann->cpu_bitmap);
Anton Blanchard5d67be92011-07-04 21:57:50 +1000184 if (ret)
185 goto out_delete;
186 }
187
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200188 ret = perf_session__process_events(session, &ann->ops);
Li Zefanbab81b62009-12-01 14:04:49 +0800189 if (ret)
Arnaldo Carvalho de Melo94c744b2009-12-11 21:24:02 -0200190 goto out_delete;
Ingo Molnar8035e422009-06-06 15:19:13 +0200191
Arnaldo Carvalho de Melo62daacb2009-11-27 16:29:22 -0200192 if (dump_trace) {
Arnaldo Carvalho de Meloc8446b92010-05-14 10:36:42 -0300193 perf_session__fprintf_nr_events(session, stdout);
Arnaldo Carvalho de Melo94c744b2009-12-11 21:24:02 -0200194 goto out_delete;
Arnaldo Carvalho de Melo62daacb2009-11-27 16:29:22 -0200195 }
Ingo Molnar8035e422009-06-06 15:19:13 +0200196
Arnaldo Carvalho de Meloda21d1b2009-10-07 10:49:00 -0300197 if (verbose > 3)
Arnaldo Carvalho de Melob3165f42009-12-13 19:50:28 -0200198 perf_session__fprintf(session, stdout);
Ingo Molnar8035e422009-06-06 15:19:13 +0200199
Arnaldo Carvalho de Meloda21d1b2009-10-07 10:49:00 -0300200 if (verbose > 2)
Arnaldo Carvalho de Melocbf69682010-04-27 21:22:44 -0300201 perf_session__fprintf_dsos(session, stdout);
Ingo Molnar8035e422009-06-06 15:19:13 +0200202
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -0300203 total_nr_samples = 0;
204 list_for_each_entry(pos, &session->evlist->entries, node) {
205 struct hists *hists = &pos->hists;
206 u32 nr_samples = hists->stats.nr_events[PERF_RECORD_SAMPLE];
Ingo Molnar8035e422009-06-06 15:19:13 +0200207
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -0300208 if (nr_samples > 0) {
209 total_nr_samples += nr_samples;
210 hists__collapse_resort(hists);
211 hists__output_resort(hists);
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200212 hists__find_annotations(hists, pos->idx, ann);
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -0300213 }
214 }
215
216 if (total_nr_samples == 0) {
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200217 ui__warning("The %s file has no samples!\n", ann->input_name);
Arnaldo Carvalho de Meloe248de32011-03-05 21:40:06 -0300218 goto out_delete;
219 }
220out_delete:
221 /*
222 * Speed up the exit process, for large files this can
223 * take quite a while.
224 *
225 * XXX Enable this when using valgrind or if we ever
226 * librarize this command.
227 *
228 * Also experiment with obstacks to see how much speed
229 * up we'll get here.
230 *
231 * perf_session__delete(session);
232 */
Li Zefanbab81b62009-12-01 14:04:49 +0800233 return ret;
Ingo Molnar8035e422009-06-06 15:19:13 +0200234}
235
236static const char * const annotate_usage[] = {
237 "perf annotate [<options>] <command>",
238 NULL
239};
240
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200241int cmd_annotate(int argc, const char **argv, const char *prefix __used)
242{
243 struct perf_annotate annotate = {
244 .ops = {
245 .sample = process_sample_event,
246 .mmap = perf_event__process_mmap,
247 .comm = perf_event__process_comm,
248 .fork = perf_event__process_task,
249 .ordered_samples = true,
250 .ordering_requires_timestamps = true,
251 },
252 .input_name = "perf.data",
253 };
254 const struct option options[] = {
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200255 OPT_STRING('i', "input", &annotate.input_name, "file",
Ingo Molnar8035e422009-06-06 15:19:13 +0200256 "input file name"),
Arnaldo Carvalho de Meloac73c5a2010-03-24 16:40:16 -0300257 OPT_STRING('d', "dsos", &symbol_conf.dso_list_str, "dso[,dso...]",
258 "only consider symbols in these dsos"),
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200259 OPT_STRING('s', "symbol", &annotate.sym_hist_filter, "symbol",
Ingo Molnar0b73da32009-06-06 15:48:52 +0200260 "symbol to annotate"),
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200261 OPT_BOOLEAN('f', "force", &annotate.force, "don't complain, do it"),
Ian Munsiec0555642010-04-13 18:37:33 +1000262 OPT_INCR('v', "verbose", &verbose,
Ingo Molnar8035e422009-06-06 15:19:13 +0200263 "be more verbose (show symbol address, etc)"),
264 OPT_BOOLEAN('D', "dump-raw-trace", &dump_trace,
265 "dump raw trace in ASCII"),
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200266 OPT_BOOLEAN(0, "tui", &annotate.use_tui, "Use the TUI interface"),
267 OPT_BOOLEAN(0, "stdio", &annotate.use_stdio, "Use the stdio interface"),
Arnaldo Carvalho de Melob32d1332009-11-24 12:05:15 -0200268 OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name,
269 "file", "vmlinux pathname"),
270 OPT_BOOLEAN('m', "modules", &symbol_conf.use_modules,
Mike Galbraith42976482009-07-02 08:09:46 +0200271 "load module symbols - WARNING: use only with -k and LIVE kernel"),
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200272 OPT_BOOLEAN('l', "print-line", &annotate.print_line,
Frederic Weisbecker301406b2009-06-13 00:11:21 +0200273 "print matching source lines (may be slow)"),
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200274 OPT_BOOLEAN('P', "full-paths", &annotate.full_paths,
Mike Galbraith42976482009-07-02 08:09:46 +0200275 "Don't shorten the displayed pathnames"),
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200276 OPT_STRING('c', "cpu", &annotate.cpu_list, "cpu", "list of cpus to profile"),
Stephane Eraniane71a0592011-07-30 01:20:40 +0200277 OPT_STRING(0, "symfs", &symbol_conf.symfs, "directory",
278 "Look for files with symbols relative to this directory"),
Arnaldo Carvalho de Melo64c6f0c2011-10-06 12:48:31 -0300279 OPT_BOOLEAN(0, "source", &symbol_conf.annotate_src,
Stephane Eranian3e6a2a72011-05-17 17:32:07 +0200280 "Interleave source code with assembly code (default)"),
Arnaldo Carvalho de Melo64c6f0c2011-10-06 12:48:31 -0300281 OPT_BOOLEAN(0, "asm-raw", &symbol_conf.annotate_asm_raw,
Stephane Eranian3e6a2a72011-05-17 17:32:07 +0200282 "Display raw encoding of assembly instructions (default)"),
Andi Kleenf69b64f2011-09-15 14:31:41 -0700283 OPT_STRING('M', "disassembler-style", &disassembler_style, "disassembler style",
284 "Specify disassembler style (e.g. -M intel for intel syntax)"),
Ingo Molnar8035e422009-06-06 15:19:13 +0200285 OPT_END()
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200286 };
Ingo Molnar8035e422009-06-06 15:19:13 +0200287
Arnaldo Carvalho de Melo655000e2009-12-15 20:04:40 -0200288 argc = parse_options(argc, argv, options, annotate_usage, 0);
289
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200290 if (annotate.use_stdio)
Arnaldo Carvalho de Melo8b9e74e2010-08-21 10:38:16 -0300291 use_browser = 0;
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200292 else if (annotate.use_tui)
Arnaldo Carvalho de Melo8b9e74e2010-08-21 10:38:16 -0300293 use_browser = 1;
294
Arnaldo Carvalho de Melo229ade92011-01-31 18:08:39 -0200295 setup_browser(true);
Arnaldo Carvalho de Melo46e3e052010-05-22 11:25:40 -0300296
Arnaldo Carvalho de Melo78f7def2011-02-04 09:45:46 -0200297 symbol_conf.priv_size = sizeof(struct annotation);
Arnaldo Carvalho de Melo75be6cf2009-12-15 20:04:39 -0200298 symbol_conf.try_vmlinux_path = true;
299
300 if (symbol__init() < 0)
Arnaldo Carvalho de Melob32d1332009-11-24 12:05:15 -0200301 return -1;
Ingo Molnar8035e422009-06-06 15:19:13 +0200302
Arnaldo Carvalho de Meloc8829c72009-12-14 20:09:29 -0200303 setup_sorting(annotate_usage, options);
Ingo Molnar8035e422009-06-06 15:19:13 +0200304
Ingo Molnar0b73da32009-06-06 15:48:52 +0200305 if (argc) {
306 /*
307 * Special case: if there's an argument left then assume tha
308 * it's a symbol filter:
309 */
310 if (argc > 1)
311 usage_with_options(annotate_usage, options);
312
Arnaldo Carvalho de Melo7009cc32011-11-17 12:33:21 -0200313 annotate.sym_hist_filter = argv[0];
Ingo Molnar0b73da32009-06-06 15:48:52 +0200314 }
315
John Kacurdd68ada2009-09-24 18:02:49 +0200316 if (field_sep && *field_sep == '.') {
Arnaldo Carvalho de Melo29a9f662010-02-03 16:52:06 -0200317 pr_err("'.' is the only non valid --field-separator argument\n");
318 return -1;
John Kacurdd68ada2009-09-24 18:02:49 +0200319 }
320
Arnaldo Carvalho de Melod20deb62011-11-25 08:19:45 -0200321 return __cmd_annotate(&annotate);
Ingo Molnar8035e422009-06-06 15:19:13 +0200322}