Introduce interpreter factories
[external/binutils.git] / gdb / mi / mi-interp.c
index 52a3a62..f8c007c 100644 (file)
@@ -1,6 +1,6 @@
 /* MI Interpreter Definitions and Commands for GDB, the GNU debugger.
 
-   Copyright (C) 2002-2014 Free Software Foundation, Inc.
+   Copyright (C) 2002-2016 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -18,7 +18,6 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
 
 #include "defs.h"
-#include <string.h>
 #include "interps.h"
 #include "event-top.h"
 #include "event-loop.h"
@@ -26,7 +25,6 @@
 #include "infrun.h"
 #include "ui-out.h"
 #include "top.h"
-#include "exceptions.h"
 #include "mi-main.h"
 #include "mi-cmds.h"
 #include "mi-out.h"
@@ -39,6 +37,7 @@
 #include "objfiles.h"
 #include "tracepoint.h"
 #include "cli-out.h"
+#include "thread-fsm.h"
 
 /* These are the interpreter setup, etc. functions for the MI
    interpreter.  */
@@ -66,7 +65,8 @@ static void mi_on_no_history (void);
 
 static void mi_new_thread (struct thread_info *t);
 static void mi_thread_exit (struct thread_info *t, int silent);
-static void mi_record_changed (struct inferior*, int);
+static void mi_record_changed (struct inferior*, int, const char *,
+                              const char *);
 static void mi_inferior_added (struct inferior *inf);
 static void mi_inferior_appeared (struct inferior *inf);
 static void mi_inferior_exit (struct inferior *inf);
@@ -174,7 +174,8 @@ mi_interpreter_init (struct interp *interp, int top_level)
 static int
 mi_interpreter_resume (void *data)
 {
-  struct mi_interp *mi = data;
+  struct mi_interp *mi = (struct mi_interp *) data;
+  struct ui *ui = current_ui;
 
   /* As per hack note in mi_interpreter_init, swap in the output
      channels... */
@@ -182,8 +183,8 @@ mi_interpreter_resume (void *data)
 
   /* These overwrite some of the initialization done in
      _intialize_event_loop.  */
-  call_readline = gdb_readline2;
-  input_handler = mi_execute_command_input_handler;
+  ui->call_readline = gdb_readline_no_editing_callback;
+  ui->input_handler = mi_execute_command_input_handler;
   async_command_editing_p = 0;
   /* FIXME: This is a total hack for now.  PB's use of the MI
      implicitly relies on a bug in the async support which allows
@@ -237,7 +238,7 @@ mi_cmd_interpreter_exec (char *command, char **argv, int argc)
     error (_("-interpreter-exec: "
             "Usage: -interpreter-exec interp command"));
 
-  interp_to_use = interp_lookup (argv[0]);
+  interp_to_use = interp_lookup (current_ui, argv[0]);
   if (interp_to_use == NULL)
     error (_("-interpreter-exec: could not find interpreter \"%s\""),
           argv[0]);
@@ -311,16 +312,8 @@ mi_execute_command_wrapper (const char *cmd)
 static void
 mi_on_sync_execution_done (void)
 {
-  /* MI generally prints a prompt after a command, indicating it's
-     ready for further input.  However, due to an historical wart, if
-     MI async, and a (CLI) synchronous command was issued, then we
-     will print the prompt right after printing "^running", even if we
-     cannot actually accept any input until the target stops.  See
-     mi_on_resume.  However, if the target is async but MI is sync,
-     then we need to output the MI prompt now, to replicate gdb's
-     behavior when neither the target nor MI are async.  (Note this
-     observer is only called by the asynchronous target event handling
-     code.)  */
+  /* If MI is sync, then output the MI prompt now, indicating we're
+     ready for further input.  */
   if (!mi_async_p ())
     {
       fputs_unfiltered ("(gdb) \n", raw_stdout);
@@ -335,20 +328,12 @@ mi_execute_command_input_handler (char *cmd)
 {
   mi_execute_command_wrapper (cmd);
 
-  /* MI generally prints a prompt after a command, indicating it's
-     ready for further input.  However, due to an historical wart, if
-     MI is async, and a synchronous command was issued, then we will
-     print the prompt right after printing "^running", even if we
-     cannot actually accept any input until the target stops.  See
-     mi_on_resume.
-
-     If MI is not async, then we print the prompt when the command
-     finishes.  If the target is sync, that means output the prompt
-     now, as in that case executing a command doesn't return until the
-     command is done.  However, if the target is async, we go back to
-     the event loop and output the prompt in the
-     'synchronous_command_done' observer.  */
-  if (!target_is_async_p () || !sync_execution)
+  /* Print a prompt, indicating we're ready for further input, unless
+     we just started a synchronous command.  In that case, we're about
+     to go back to the event loop and will output the prompt in the
+     'synchronous_command_done' observer when the target next
+     stops.  */
+  if (!sync_execution)
     {
       fputs_unfiltered ("(gdb) \n", raw_stdout);
       gdb_flush (raw_stdout);
@@ -372,15 +357,21 @@ mi_command_loop (void *data)
 static void
 mi_new_thread (struct thread_info *t)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
-  struct inferior *inf = find_inferior_pid (ptid_get_pid (t->ptid));
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct inferior *inf = find_inferior_ptid (t->ptid);
+  struct cleanup *old_chain;
 
   gdb_assert (inf);
 
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
+
   fprintf_unfiltered (mi->event_channel, 
                      "thread-created,id=\"%d\",group-id=\"i%d\"",
-                     t->num, inf->num);
+                     t->global_num, inf->num);
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static void
@@ -388,63 +379,109 @@ mi_thread_exit (struct thread_info *t, int silent)
 {
   struct mi_interp *mi;
   struct inferior *inf;
+  struct cleanup *old_chain;
 
   if (silent)
     return;
 
-  inf = find_inferior_pid (ptid_get_pid (t->ptid));
+  inf = find_inferior_ptid (t->ptid);
+
+  mi = (struct mi_interp *) top_level_interpreter_data ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
-  mi = top_level_interpreter_data ();
-  target_terminal_ours ();
   fprintf_unfiltered (mi->event_channel, 
                      "thread-exited,id=\"%d\",group-id=\"i%d\"",
-                     t->num, inf->num);
+                     t->global_num, inf->num);
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification on changing the state of record.  */
 
 static void
-mi_record_changed (struct inferior *inferior, int started)
+mi_record_changed (struct inferior *inferior, int started, const char *method,
+                  const char *format)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
+
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
+
+  if (started)
+    {
+      if (format != NULL)
+       {
+         fprintf_unfiltered (
+           mi->event_channel,
+           "record-started,thread-group=\"i%d\",method=\"%s\",format=\"%s\"",
+           inferior->num, method, format);
+       }
+      else
+       {
+         fprintf_unfiltered (
+           mi->event_channel,
+           "record-started,thread-group=\"i%d\",method=\"%s\"",
+           inferior->num, method);
+       }
+    }
+  else
+    {
+      fprintf_unfiltered (mi->event_channel,
+                         "record-stopped,thread-group=\"i%d\"", inferior->num);
+    }
 
-  fprintf_unfiltered (mi->event_channel,  "record-%s,thread-group=\"i%d\"",
-                     started ? "started" : "stopped", inferior->num);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static void
 mi_inferior_added (struct inferior *inf)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
+
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
-  target_terminal_ours ();
   fprintf_unfiltered (mi->event_channel,
                      "thread-group-added,id=\"i%d\"",
                      inf->num);
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static void
 mi_inferior_appeared (struct inferior *inf)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
+
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
-  target_terminal_ours ();
   fprintf_unfiltered (mi->event_channel,
                      "thread-group-started,id=\"i%d\",pid=\"%d\"",
                      inf->num, inf->pid);
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static void
 mi_inferior_exit (struct inferior *inf)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
+
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
-  target_terminal_ours ();
   if (inf->has_exit_code)
     fprintf_unfiltered (mi->event_channel,
                        "thread-group-exited,id=\"i%d\",exit-code=\"%s\"",
@@ -452,30 +489,26 @@ mi_inferior_exit (struct inferior *inf)
   else
     fprintf_unfiltered (mi->event_channel,
                        "thread-group-exited,id=\"i%d\"", inf->num);
+  gdb_flush (mi->event_channel);
 
-  gdb_flush (mi->event_channel);  
+  do_cleanups (old_chain);
 }
 
 static void
 mi_inferior_removed (struct inferior *inf)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
+
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
-  target_terminal_ours ();
   fprintf_unfiltered (mi->event_channel,
                      "thread-group-removed,id=\"i%d\"",
                      inf->num);
   gdb_flush (mi->event_channel);
-}
-
-/* Cleanup that restores a previous current uiout.  */
-
-static void
-restore_current_uiout_cleanup (void *arg)
-{
-  struct ui_out *saved_uiout = arg;
 
-  current_uiout = saved_uiout;
+  do_cleanups (old_chain);
 }
 
 /* Return the MI interpreter, if it is active -- either because it's
@@ -507,7 +540,7 @@ mi_interp_data (void)
   struct interp *interp = find_mi_interpreter ();
 
   if (interp != NULL)
-    return interp_data (interp);
+    return (struct mi_interp *) interp_data (interp);
   return NULL;
 }
 
@@ -595,84 +628,59 @@ mi_on_normal_stop (struct bpstats *bs, int print_frame)
 
   if (print_frame)
     {
+      struct thread_info *tp;
       int core;
 
-      if (current_uiout != mi_uiout)
-       {
-         /* The normal_stop function has printed frame information
-            into CLI uiout, or some other non-MI uiout.  There's no
-            way we can extract proper fields from random uiout
-            object, so we print the frame again.  In practice, this
-            can only happen when running a CLI command in MI.  */
-         struct ui_out *saved_uiout = current_uiout;
-         struct target_waitstatus last;
-         ptid_t last_ptid;
-
-         current_uiout = mi_uiout;
+      tp = inferior_thread ();
 
-         get_last_target_status (&last_ptid, &last);
-         print_stop_event (&last);
+      if (tp->thread_fsm != NULL
+         && thread_fsm_finished_p (tp->thread_fsm))
+       {
+         enum async_reply_reason reason;
 
-         current_uiout = saved_uiout;
+         reason = thread_fsm_async_reply_reason (tp->thread_fsm);
+         ui_out_field_string (mi_uiout, "reason",
+                              async_reason_lookup (reason));
        }
-      /* Otherwise, frame information has already been printed by
-        normal_stop.  */
-      else
+      print_stop_event (mi_uiout);
+
+      /* Breakpoint hits should always be mirrored to the console.
+        Deciding what to mirror to the console wrt to breakpoints and
+        random stops gets messy real fast.  E.g., say "s" trips on a
+        breakpoint.  We'd clearly want to mirror the event to the
+        console in this case.  But what about more complicated cases
+        like "s&; thread n; s&", and one of those steps spawning a
+        new thread, and that thread hitting a breakpoint?  It's
+        impossible in general to track whether the thread had any
+        relation to the commands that had been executed.  So we just
+        simplify and always mirror breakpoints and random events to
+        the console.
+
+        OTOH, we should print the source line to the console when
+        stepping or other similar commands, iff the step was started
+        by a console command, but not if it was started with
+        -exec-step or similar.  */
+      if ((bpstat_what (tp->control.stop_bpstat).main_action
+          == BPSTAT_WHAT_STOP_NOISY)
+         || !(tp->thread_fsm != NULL
+              && thread_fsm_finished_p (tp->thread_fsm))
+         || (tp->control.command_interp != NULL
+             && tp->control.command_interp != top_level_interpreter ()))
        {
-         /* Breakpoint hits should always be mirrored to the console.
-            Deciding what to mirror to the console wrt to breakpoints
-            and random stops gets messy real fast.  E.g., say "s"
-            trips on a breakpoint.  We'd clearly want to mirror the
-            event to the console in this case.  But what about more
-            complicated cases like "s&; thread n; s&", and one of
-            those steps spawning a new thread, and that thread
-            hitting a breakpoint?  It's impossible in general to
-            track whether the thread had any relation to the commands
-            that had been executed.  So we just simplify and always
-            mirror breakpoints and random events to the console.
-
-            Also, CLI execution commands (-interpreter-exec console
-            "next", for example) in async mode have the opposite
-            issue as described in the "then" branch above --
-            normal_stop has already printed frame information to MI
-            uiout, but nothing has printed the same information to
-            the CLI channel.  We should print the source line to the
-            console when stepping or other similar commands, iff the
-            step was started by a console command (but not if it was
-            started with -exec-step or similar).  */
-         struct thread_info *tp = inferior_thread ();
-
-         if ((!tp->control.stop_step
-                 && !tp->control.proceed_to_finish)
-             || (tp->control.command_interp != NULL
-                 && tp->control.command_interp != top_level_interpreter ()))
-           {
-             struct mi_interp *mi = top_level_interpreter_data ();
-             struct target_waitstatus last;
-             ptid_t last_ptid;
-             struct cleanup *old_chain;
-
-             /* Set the current uiout to CLI uiout temporarily.  */
-             old_chain = make_cleanup (restore_current_uiout_cleanup,
-                                       current_uiout);
-             current_uiout = mi->cli_uiout;
-
-             get_last_target_status (&last_ptid, &last);
-             print_stop_event (&last);
-
-             do_cleanups (old_chain);
-           }
+         struct mi_interp *mi
+           = (struct mi_interp *) top_level_interpreter_data ();
+
+         print_stop_event (mi->cli_uiout);
        }
 
-      ui_out_field_int (mi_uiout, "thread-id",
-                       pid_to_thread_id (inferior_ptid));
+      tp = inferior_thread ();
+      ui_out_field_int (mi_uiout, "thread-id", tp->global_num);
       if (non_stop)
        {
          struct cleanup *back_to = make_cleanup_ui_out_list_begin_end 
            (mi_uiout, "stopped-threads");
 
-         ui_out_field_int (mi_uiout, NULL,
-                           pid_to_thread_id (inferior_ptid));
+         ui_out_field_int (mi_uiout, NULL, tp->global_num);
          do_cleanups (back_to);
        }
       else
@@ -722,12 +730,14 @@ struct mi_suppress_notification mi_suppress_notification =
 static void
 mi_traceframe_changed (int tfnum, int tpnum)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
 
   if (mi_suppress_notification.traceframe)
     return;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   if (tfnum >= 0)
     fprintf_unfiltered (mi->event_channel, "traceframe-changed,"
@@ -737,6 +747,8 @@ mi_traceframe_changed (int tfnum, int tpnum)
     fprintf_unfiltered (mi->event_channel, "traceframe-changed,end");
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification on creating a trace state variable.  */
@@ -744,15 +756,19 @@ mi_traceframe_changed (int tfnum, int tpnum)
 static void
 mi_tsv_created (const struct trace_state_variable *tsv)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   fprintf_unfiltered (mi->event_channel, "tsv-created,"
                      "name=\"%s\",initial=\"%s\"\n",
                      tsv->name, plongest (tsv->initial_value));
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification on deleting a trace state variable.  */
@@ -760,9 +776,11 @@ mi_tsv_created (const struct trace_state_variable *tsv)
 static void
 mi_tsv_deleted (const struct trace_state_variable *tsv)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   if (tsv != NULL)
     fprintf_unfiltered (mi->event_channel, "tsv-deleted,"
@@ -771,6 +789,8 @@ mi_tsv_deleted (const struct trace_state_variable *tsv)
     fprintf_unfiltered (mi->event_channel, "tsv-deleted\n");
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification on modifying a trace state variable.  */
@@ -778,10 +798,12 @@ mi_tsv_deleted (const struct trace_state_variable *tsv)
 static void
 mi_tsv_modified (const struct trace_state_variable *tsv)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
   struct ui_out *mi_uiout = interp_ui_out (top_level_interpreter ());
+  struct cleanup *old_chain;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   fprintf_unfiltered (mi->event_channel,
                      "tsv-modified");
@@ -797,6 +819,8 @@ mi_tsv_modified (const struct trace_state_variable *tsv)
   ui_out_redirect (mi_uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification about a created breakpoint.  */
@@ -804,9 +828,9 @@ mi_tsv_modified (const struct trace_state_variable *tsv)
 static void
 mi_breakpoint_created (struct breakpoint *b)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
   struct ui_out *mi_uiout = interp_ui_out (top_level_interpreter ());
-  volatile struct gdb_exception e;
+  struct cleanup *old_chain;
 
   if (mi_suppress_notification.breakpoint)
     return;
@@ -814,7 +838,9 @@ mi_breakpoint_created (struct breakpoint *b)
   if (b->number <= 0)
     return;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
+
   fprintf_unfiltered (mi->event_channel,
                      "breakpoint-created");
   /* We want the output from gdb_breakpoint_query to go to
@@ -825,11 +851,20 @@ mi_breakpoint_created (struct breakpoint *b)
      breakpoint_created notifications.  So, we use
      ui_out_redirect.  */
   ui_out_redirect (mi_uiout, mi->event_channel);
-  TRY_CATCH (e, RETURN_MASK_ERROR)
-    gdb_breakpoint_query (mi_uiout, b->number, NULL);
+  TRY
+    {
+      gdb_breakpoint_query (mi_uiout, b->number, NULL);
+    }
+  CATCH (e, RETURN_MASK_ERROR)
+    {
+    }
+  END_CATCH
+
   ui_out_redirect (mi_uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification about deleted breakpoint.  */
@@ -837,7 +872,8 @@ mi_breakpoint_created (struct breakpoint *b)
 static void
 mi_breakpoint_deleted (struct breakpoint *b)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct cleanup *old_chain;
 
   if (mi_suppress_notification.breakpoint)
     return;
@@ -845,12 +881,15 @@ mi_breakpoint_deleted (struct breakpoint *b)
   if (b->number <= 0)
     return;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   fprintf_unfiltered (mi->event_channel, "breakpoint-deleted,id=\"%d\"",
                      b->number);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification about modified breakpoint.  */
@@ -858,9 +897,9 @@ mi_breakpoint_deleted (struct breakpoint *b)
 static void
 mi_breakpoint_modified (struct breakpoint *b)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
   struct ui_out *mi_uiout = interp_ui_out (top_level_interpreter ());
-  volatile struct gdb_exception e;
+  struct cleanup *old_chain;
 
   if (mi_suppress_notification.breakpoint)
     return;
@@ -868,7 +907,9 @@ mi_breakpoint_modified (struct breakpoint *b)
   if (b->number <= 0)
     return;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
+
   fprintf_unfiltered (mi->event_channel,
                      "breakpoint-modified");
   /* We want the output from gdb_breakpoint_query to go to
@@ -879,22 +920,31 @@ mi_breakpoint_modified (struct breakpoint *b)
      breakpoint_created notifications.  So, we use
      ui_out_redirect.  */
   ui_out_redirect (mi_uiout, mi->event_channel);
-  TRY_CATCH (e, RETURN_MASK_ERROR)
-    gdb_breakpoint_query (mi_uiout, b->number, NULL);
+  TRY
+    {
+      gdb_breakpoint_query (mi_uiout, b->number, NULL);
+    }
+  CATCH (e, RETURN_MASK_ERROR)
+    {
+    }
+  END_CATCH
+
   ui_out_redirect (mi_uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static int
 mi_output_running_pid (struct thread_info *info, void *arg)
 {
-  ptid_t *ptid = arg;
+  ptid_t *ptid = (ptid_t *) arg;
 
   if (ptid_get_pid (*ptid) == ptid_get_pid (info->ptid))
     fprintf_unfiltered (raw_stdout,
                        "*running,thread-id=\"%d\"\n",
-                       info->num);
+                       info->global_num);
 
   return 0;
 }
@@ -904,7 +954,7 @@ mi_inferior_count (struct inferior *inf, void *arg)
 {
   if (inf->pid != 0)
     {
-      int *count_p = arg;
+      int *count_p = (int *) arg;
       (*count_p)++;
     }
 
@@ -960,7 +1010,8 @@ mi_on_resume (ptid_t ptid)
       struct thread_info *ti = find_thread_ptid (ptid);
 
       gdb_assert (ti);
-      fprintf_unfiltered (raw_stdout, "*running,thread-id=\"%d\"\n", ti->num);
+      fprintf_unfiltered (raw_stdout, "*running,thread-id=\"%d\"\n",
+                         ti->global_num);
     }
 
   if (!running_result_record_printed && mi_proceeded)
@@ -971,7 +1022,7 @@ mi_on_resume (ptid_t ptid)
         for MI3, and may be removed even earlier.  SYNC_EXECUTION is
         checked here because we only need to emit a prompt if a
         synchronous command was issued when the target is async.  */
-      if (!target_is_async_p () || sync_execution)
+      if (!target_can_async_p () || sync_execution)
        fputs_unfiltered ("(gdb) \n", raw_stdout);
     }
   gdb_flush (raw_stdout);
@@ -980,47 +1031,62 @@ mi_on_resume (ptid_t ptid)
 static void
 mi_solib_loaded (struct so_list *solib)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct ui_out *uiout = interp_ui_out (top_level_interpreter ());
+  struct cleanup *old_chain;
 
-  target_terminal_ours ();
-  if (gdbarch_has_global_solist (target_gdbarch ()))
-    fprintf_unfiltered (mi->event_channel,
-                       "library-loaded,id=\"%s\",target-name=\"%s\","
-                       "host-name=\"%s\",symbols-loaded=\"%d\"",
-                       solib->so_original_name, solib->so_original_name,
-                       solib->so_name, solib->symbols_loaded);
-  else
-    fprintf_unfiltered (mi->event_channel,
-                       "library-loaded,id=\"%s\",target-name=\"%s\","
-                       "host-name=\"%s\",symbols-loaded=\"%d\","
-                       "thread-group=\"i%d\"",
-                       solib->so_original_name, solib->so_original_name,
-                       solib->so_name, solib->symbols_loaded,
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
+
+  fprintf_unfiltered (mi->event_channel, "library-loaded");
+
+  ui_out_redirect (uiout, mi->event_channel);
+
+  ui_out_field_string (uiout, "id", solib->so_original_name);
+  ui_out_field_string (uiout, "target-name", solib->so_original_name);
+  ui_out_field_string (uiout, "host-name", solib->so_name);
+  ui_out_field_int (uiout, "symbols-loaded", solib->symbols_loaded);
+  if (!gdbarch_has_global_solist (target_gdbarch ()))
+    {
+      ui_out_field_fmt (uiout, "thread-group", "i%d",
                        current_inferior ()->num);
+    }
+
+  ui_out_redirect (uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static void
 mi_solib_unloaded (struct so_list *solib)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
+  struct ui_out *uiout = interp_ui_out (top_level_interpreter ());
+  struct cleanup *old_chain;
 
-  target_terminal_ours ();
-  if (gdbarch_has_global_solist (target_gdbarch ()))
-    fprintf_unfiltered (mi->event_channel,
-                       "library-unloaded,id=\"%s\",target-name=\"%s\","
-                       "host-name=\"%s\"",
-                       solib->so_original_name, solib->so_original_name,
-                       solib->so_name);
-  else
-    fprintf_unfiltered (mi->event_channel,
-                       "library-unloaded,id=\"%s\",target-name=\"%s\","
-                       "host-name=\"%s\",thread-group=\"i%d\"",
-                       solib->so_original_name, solib->so_original_name,
-                       solib->so_name, current_inferior ()->num);
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
+
+  fprintf_unfiltered (mi->event_channel, "library-unloaded");
+
+  ui_out_redirect (uiout, mi->event_channel);
+
+  ui_out_field_string (uiout, "id", solib->so_original_name);
+  ui_out_field_string (uiout, "target-name", solib->so_original_name);
+  ui_out_field_string (uiout, "host-name", solib->so_name);
+  if (!gdbarch_has_global_solist (target_gdbarch ()))
+    {
+      ui_out_field_fmt (uiout, "thread-group", "i%d",
+                       current_inferior ()->num);
+    }
+
+  ui_out_redirect (uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification about the command parameter change.  */
@@ -1028,13 +1094,15 @@ mi_solib_unloaded (struct so_list *solib)
 static void
 mi_command_param_changed (const char *param, const char *value)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
   struct ui_out *mi_uiout = interp_ui_out (top_level_interpreter ());
+  struct cleanup *old_chain;
 
   if (mi_suppress_notification.cmd_param_changed)
     return;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   fprintf_unfiltered (mi->event_channel,
                      "cmd-param-changed");
@@ -1047,6 +1115,8 @@ mi_command_param_changed (const char *param, const char *value)
   ui_out_redirect (mi_uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 /* Emit notification about the target memory change.  */
@@ -1055,14 +1125,16 @@ static void
 mi_memory_changed (struct inferior *inferior, CORE_ADDR memaddr,
                   ssize_t len, const bfd_byte *myaddr)
 {
-  struct mi_interp *mi = top_level_interpreter_data ();
+  struct mi_interp *mi = (struct mi_interp *) top_level_interpreter_data ();
   struct ui_out *mi_uiout = interp_ui_out (top_level_interpreter ());
   struct obj_section *sec;
+  struct cleanup *old_chain;
 
   if (mi_suppress_notification.memory)
     return;
 
-  target_terminal_ours ();
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
   fprintf_unfiltered (mi->event_channel,
                      "memory-changed");
@@ -1088,6 +1160,8 @@ mi_memory_changed (struct inferior *inferior, CORE_ADDR memaddr,
   ui_out_redirect (mi_uiout, NULL);
 
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
 }
 
 static int
@@ -1097,20 +1171,25 @@ report_initial_inferior (struct inferior *inf, void *closure)
      mi_inferior_added assumes that inferior is fully initialized
      and top_level_interpreter_data is set, we cannot call
      it here.  */
-  struct mi_interp *mi = closure;
+  struct mi_interp *mi = (struct mi_interp *) closure;
+  struct cleanup *old_chain;
+
+  old_chain = make_cleanup_restore_target_terminal ();
+  target_terminal_ours_for_output ();
 
-  target_terminal_ours ();
   fprintf_unfiltered (mi->event_channel,
                      "thread-group-added,id=\"i%d\"",
                      inf->num);
   gdb_flush (mi->event_channel);
+
+  do_cleanups (old_chain);
   return 0;
 }
 
 static struct ui_out *
 mi_ui_out (struct interp *interp)
 {
-  struct mi_interp *mi = interp_data (interp);
+  struct mi_interp *mi = (struct mi_interp *) interp_data (interp);
 
   return mi->mi_uiout;
 }
@@ -1127,7 +1206,7 @@ static int
 mi_set_logging (struct interp *interp, int start_log,
                struct ui_file *out, struct ui_file *logfile)
 {
-  struct mi_interp *mi = interp_data (interp);
+  struct mi_interp *mi = (struct mi_interp *) interp_data (interp);
 
   if (!mi)
     return 0;
@@ -1163,25 +1242,35 @@ mi_set_logging (struct interp *interp, int start_log,
   return 1;
 }
 
+/* The MI interpreter's vtable.  */
+
+static const struct interp_procs mi_interp_procs =
+{
+  mi_interpreter_init,         /* init_proc */
+  mi_interpreter_resume,       /* resume_proc */
+  mi_interpreter_suspend,      /* suspend_proc */
+  mi_interpreter_exec,         /* exec_proc */
+  mi_ui_out,                   /* ui_out_proc */
+  mi_set_logging,              /* set_logging_proc */
+  mi_command_loop              /* command_loop_proc */
+};
+
+/* Factory for MI interpreters.  */
+
+static struct interp *
+mi_interp_factory (const char *name)
+{
+  return interp_new (name, &mi_interp_procs, NULL);
+}
+
 extern initialize_file_ftype _initialize_mi_interp; /* -Wmissing-prototypes */
 
 void
 _initialize_mi_interp (void)
 {
-  static const struct interp_procs procs =
-    {
-      mi_interpreter_init,     /* init_proc */
-      mi_interpreter_resume,   /* resume_proc */
-      mi_interpreter_suspend,  /* suspend_proc */
-      mi_interpreter_exec,     /* exec_proc */
-      mi_ui_out,               /* ui_out_proc */
-      mi_set_logging,          /* set_logging_proc */
-      mi_command_loop          /* command_loop_proc */
-    };
-
   /* The various interpreter levels.  */
-  interp_add (interp_new (INTERP_MI1, &procs));
-  interp_add (interp_new (INTERP_MI2, &procs));
-  interp_add (interp_new (INTERP_MI3, &procs));
-  interp_add (interp_new (INTERP_MI, &procs));
+  interp_factory_register (INTERP_MI1, mi_interp_factory);
+  interp_factory_register (INTERP_MI2, mi_interp_factory);
+  interp_factory_register (INTERP_MI3, mi_interp_factory);
+  interp_factory_register (INTERP_MI, mi_interp_factory);
 }