From f5911ea11965af3dc626c1c6278e45e214fc640c Mon Sep 17 00:00:00 2001 From: Hafiz Abid Qadeer Date: Fri, 15 Feb 2013 09:47:50 +0000 Subject: [PATCH] 2013-02-15 Pedro Alves Hafiz Abid Qadeer gdb/ * NEWS: Mention new field "trace-file". * tracepoint.c (trace_status_mi): Output "trace-file" field. (tfile_open): Record the trace file's filename in the trace status. (tfile_files_info): Mention the name of the trace file. Check the "filename" field explicitely. (trace_status_command): Explicitely check "filename" field. (trace_find_command): Ditto. (trace_find_pc_command): Ditto. (trace_find_tracepoint_command): Ditto. (trace_find_line_command): Ditto. (trace_find_range_command): Ditto. (trace_find_outside_command): Ditto. * tracepoint.h (struct trace_status) : Rename it to "filename" and make it hold the trace file's filename instead of a boolean. * remote.c (remote_get_trace_status): Initialize "filename" field with NULL instead of 0. gdb/doc/ * gdb.texinfo (GDB/MI Tracepoint Commands) <-trace-status>: Document the "trace-file" field. gdb/testsuite/ * gdb.trace/tfile.exp: Add test for -trace-status command. --- gdb/ChangeLog | 22 ++++++++++++++++++++ gdb/NEWS | 3 +++ gdb/doc/ChangeLog | 6 ++++++ gdb/doc/gdb.texinfo | 4 ++++ gdb/remote.c | 2 +- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.trace/tfile.exp | 4 ++++ gdb/tracepoint.c | 34 +++++++++++++++++++------------ gdb/tracepoint.h | 6 +++--- 9 files changed, 69 insertions(+), 17 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index cc579088b60..2bdab56de17 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,25 @@ +2013-02-15 Pedro Alves + Hafiz Abid Qadeer + + * NEWS: Mention new field "trace-file". + * tracepoint.c (trace_status_mi): Output "trace-file" field. + (tfile_open): Record the trace file's filename in the trace + status. + (tfile_files_info): Mention the name of the trace file. + Check the "filename" field explicitely. + (trace_status_command): Explicitely check "filename" field. + (trace_find_command): Ditto. + (trace_find_pc_command): Ditto. + (trace_find_tracepoint_command): Ditto. + (trace_find_line_command): Ditto. + (trace_find_range_command): Ditto. + (trace_find_outside_command): Ditto. + * tracepoint.h (struct trace_status) : Rename it + to "filename" and make it hold the trace file's filename + instead of a boolean. + * remote.c (remote_get_trace_status): Initialize "filename" + field with NULL instead of 0. + 2013-02-15 Yao Qi * remote.c: Fix a typo. diff --git a/gdb/NEWS b/gdb/NEWS index 38b36c2fb4d..0877aa23a24 100644 --- a/gdb/NEWS +++ b/gdb/NEWS @@ -150,6 +150,9 @@ show filename-display ** The response to breakpoint commands and breakpoint async records includes an "installed" field containing a boolean state about each non-pending tracepoint location is whether installed on target or not. + ** Output of the "-trace-status" command includes a "trace-file" field + containing the name of the trace file being examined. This field is + optional, and only present when examining a trace file. * GDB now supports the "mini debuginfo" section, .gnu_debugdata. You must have the LZMA library available when configuring GDB for this diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index 7c163116e58..6a16958c04f 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,3 +1,9 @@ +2013-02-15 Pedro Alves + Hafiz Abid Qadeer + + * gdb.texinfo (GDB/MI Tracepoint Commands) <-trace-status>: + Document the "trace-file" field. + 2013-02-07 Doug Evans * gdb.texinfo (Machine Code): Clarify argument to disassemble diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index e3f336ea7e1..e8ac8c5987b 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -32094,6 +32094,10 @@ The value of the disconnected tracing flag. @code{1} means that tracing will continue after @value{GDBN} disconnects, @code{0} means that the trace run will stop. +@item trace-file +The filename of the trace file being examined. This field is +optional, and only present when examining a trace file. + @end table @subsubheading @value{GDBN} Command diff --git a/gdb/remote.c b/gdb/remote.c index 608f6be85bc..88a57c864c3 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -10701,7 +10701,7 @@ remote_get_trace_status (struct trace_status *ts) return -1; /* We're working with a live target. */ - ts->from_file = 0; + ts->filename = NULL; if (*p++ != 'T') error (_("Bogus trace status reply from target: %s"), target_buf); diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index f67736cd3b6..77573b0d548 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2013-02-15 Pedro Alves + Hafiz Abid Qadeer + + * gdb.trace/tfile.exp: Add test for -trace-status command. + 2013-02-14 Jan Kratochvil * gdb.threads/pthread_cond_wait.c (main): Remove variable ts. Replace diff --git a/gdb/testsuite/gdb.trace/tfile.exp b/gdb/testsuite/gdb.trace/tfile.exp index b07be75bc6b..2bdde372faf 100644 --- a/gdb/testsuite/gdb.trace/tfile.exp +++ b/gdb/testsuite/gdb.trace/tfile.exp @@ -106,3 +106,7 @@ Not looking at any trace frame.*" \ gdb_test \ "interpreter-exec mi \"-target-select tfile basic.tf\"" \ "\\^connected.*" + +gdb_test "interpreter-exec mi \"-trace-status\"" \ + "\\^done,supported=\"file\",trace-file=\".*basic.tf\",running=\"0\",stop-reason=\"request\",frames=\"${decimal}\",frames-created=\"${decimal}\",buffer-size=\"${decimal}\",buffer-free=\"${decimal}\",disconnected=\".*\",circular=\".*\",user-name=\"\",notes=\"\",start-time=\".*\",stop-time=\".*\"" \ + "-trace-status" diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c index 2e50164a1ca..ca104aa7b21 100644 --- a/gdb/tracepoint.c +++ b/gdb/tracepoint.c @@ -1940,7 +1940,7 @@ trace_status_command (char *args, int from_tty) if (status == -1) { - if (ts->from_file) + if (ts->filename != NULL) printf_filtered (_("Using a trace file.\n")); else { @@ -2100,17 +2100,20 @@ trace_status_mi (int on_stop) status = target_get_trace_status (ts); - if (status == -1 && !ts->from_file) + if (status == -1 && ts->filename == NULL) { ui_out_field_string (uiout, "supported", "0"); return; } - if (ts->from_file) + if (ts->filename != NULL) ui_out_field_string (uiout, "supported", "file"); else if (!on_stop) ui_out_field_string (uiout, "supported", "1"); + if (ts->filename != NULL) + ui_out_field_string (uiout, "trace-file", ts->filename); + gdb_assert (ts->running_known); if (ts->running) @@ -2399,7 +2402,8 @@ trace_find_command (char *args, int from_tty) { /* This should only be called with a numeric argument. */ int frameno = -1; - if (current_trace_status ()->running && !current_trace_status ()->from_file) + if (current_trace_status ()->running + && current_trace_status ()->filename == NULL) error (_("May not look at trace frames while trace is running.")); if (args == 0 || *args == 0) @@ -2450,7 +2454,8 @@ trace_find_pc_command (char *args, int from_tty) { CORE_ADDR pc; - if (current_trace_status ()->running && !current_trace_status ()->from_file) + if (current_trace_status ()->running + && current_trace_status ()->filename == NULL) error (_("May not look at trace frames while trace is running.")); if (args == 0 || *args == 0) @@ -2468,7 +2473,8 @@ trace_find_tracepoint_command (char *args, int from_tty) int tdp; struct tracepoint *tp; - if (current_trace_status ()->running && !current_trace_status ()->from_file) + if (current_trace_status ()->running + && current_trace_status ()->filename == NULL) error (_("May not look at trace frames while trace is running.")); if (args == 0 || *args == 0) @@ -2507,7 +2513,8 @@ trace_find_line_command (char *args, int from_tty) struct symtab_and_line sal; struct cleanup *old_chain; - if (current_trace_status ()->running && !current_trace_status ()->from_file) + if (current_trace_status ()->running + && current_trace_status ()->filename == NULL) error (_("May not look at trace frames while trace is running.")); if (args == 0 || *args == 0) @@ -2572,7 +2579,8 @@ trace_find_range_command (char *args, int from_tty) static CORE_ADDR start, stop; char *tmp; - if (current_trace_status ()->running && !current_trace_status ()->from_file) + if (current_trace_status ()->running + && current_trace_status ()->filename == NULL) error (_("May not look at trace frames while trace is running.")); if (args == 0 || *args == 0) @@ -2605,7 +2613,8 @@ trace_find_outside_command (char *args, int from_tty) CORE_ADDR start, stop; char *tmp; - if (current_trace_status ()->running && !current_trace_status ()->from_file) + if (current_trace_status ()->running + && current_trace_status ()->filename == NULL) error (_("May not look at trace frames while trace is running.")); if (args == 0 || *args == 0) @@ -3762,8 +3771,8 @@ tfile_open (char *filename, int from_tty) trace_regblock_size = 0; ts = current_trace_status (); - /* We know we're working with a file. */ - ts->from_file = 1; + /* We know we're working with a file. Record its name. */ + ts->filename = trace_filename; /* Set defaults in case there is no status line. */ ts->running_known = 0; ts->stop_reason = trace_stop_reason_unknown; @@ -4213,8 +4222,7 @@ tfile_close (int quitting) static void tfile_files_info (struct target_ops *t) { - /* (it would be useful to mention the name of the file). */ - printf_filtered ("Looking at a trace file.\n"); + printf_filtered ("\t`%s'\n", trace_filename); } /* The trace status for a file is that tracing can never be run. */ diff --git a/gdb/tracepoint.h b/gdb/tracepoint.h index 4a3df39e308..b2b9d7c9ce7 100644 --- a/gdb/tracepoint.h +++ b/gdb/tracepoint.h @@ -72,9 +72,9 @@ enum trace_stop_reason struct trace_status { - /* This is true if the status is coming from a file rather - than a live target. */ - int from_file; + /* If the status is coming from a file rather than a live target, + this points at the file's filename. Otherwise, this is NULL. */ + const char *filename; /* This is true if the value of the running field is known. */ int running_known; -- 2.30.2