gdb: Revert FAKE_PROCESS id code to prepare for gdb 7.6.1
authorJohn Marino <draco@marino.st>
Thu, 3 Oct 2013 19:18:26 +0000 (21:18 +0200)
committerJohn Marino <draco@marino.st>
Sat, 5 Oct 2013 22:51:42 +0000 (00:51 +0200)
This set of local modifications to support the swapper as an inferior
for kgdb is being reverted to ease the merge of the new gdb 7.6.1 code.
However, it appears that gdb 7.6.1 has implemented the concept of a
"fake" process ID independently, so it may not be necessary to add
these modifications back later.

contrib/gdb-7/README.DRAGONFLY
contrib/gdb-7/gdb/corelow.c
contrib/gdb-7/gdb/inf-ptrace.c
contrib/gdb-7/gdb/inferior.c
contrib/gdb-7/gdb/inferior.h
contrib/gdb-7/gdb/infrun.c
contrib/gdb-7/gdb/mi/mi-interp.c
contrib/gdb-7/gdb/mi/mi-main.c
contrib/gdb-7/gdb/top.c

index 9ed60fa..9ccba16 100644 (file)
@@ -21,20 +21,12 @@ Local modifications applied to following files:
        gdb/amd64-tdep.h
        gdb/configure.host
        gdb/configure.tgt
-       gdb/corelow.c
        gdb/defs.h
        gdb/exec.c
        gdb/i386-tdep.h
        gdb/i386bsd-nat.c
        gdb/i386fbsd-tdep.c
-       gdb/inf-ptrace.c
-       gdb/inferior.c
-       gdb/inferior.h
-       gdb/infrun.c
        gdb/main.c
-       gdb/mi/mi-interp.c
-       gdb/mi/mi-main.c
        gdb/osabi.c
-       gdb/top.c
        include/elf/common.h
        include/libiberty.h
index c20abb7..bbfb8ee 100644 (file)
@@ -264,7 +264,7 @@ add_to_thread_list (bfd *abfd, asection *asect, void *reg_sect_arg)
 
   lwpid = core_tid;
 
-  if (current_inferior ()->pid == FAKE_PROCESS_ID)
+  if (current_inferior ()->pid == 0)
     inferior_appeared (current_inferior (), pid);
 
   ptid = ptid_build (pid, lwpid, 0);
index b368784..aa6dcf4 100644 (file)
@@ -314,7 +314,7 @@ inf_ptrace_kill (struct target_ops *ops)
   pid_t pid = ptid_get_pid (inferior_ptid);
   int status;
 
-  if (pid == FAKE_PROCESS_ID)
+  if (pid == 0)
     return;
 
   ptrace (PT_KILL, pid, (PTRACE_TYPE_ARG3)0, 0);
index e019e08..65948c4 100644 (file)
@@ -139,7 +139,7 @@ add_inferior_silent (int pid)
 
   observer_notify_inferior_added (inf);
 
-  if (pid != FAKE_PROCESS_ID)
+  if (pid != 0)
     inferior_appeared (inf, pid);
 
   return inf;
@@ -275,7 +275,7 @@ exit_inferior_1 (struct inferior *inftoex, int silent)
      so that the observers have a chance to look it up.  */
   observer_notify_inferior_exit (inf);
 
-  inf->pid = FAKE_PROCESS_ID;
+  inf->pid = 0;
   if (inf->vfork_parent != NULL)
     {
       inf->vfork_parent->vfork_child = NULL;
@@ -339,7 +339,7 @@ discard_all_inferiors (void)
 
   for (inf = inferior_list; inf; inf = inf->next)
     {
-      if (inf->pid != FAKE_PROCESS_ID)
+      if (inf->pid != 0)
        exit_inferior_silent (inf->pid);
     }
 }
@@ -361,10 +361,10 @@ find_inferior_pid (int pid)
 {
   struct inferior *inf;
 
-  /* Looking for a fake inferior pid is always wrong, and indicative of
-     a bug somewhere else.  There may be more than one with fake pid,
+  /* Looking for inferior pid == 0 is always wrong, and indicative of
+     a bug somewhere else.  There may be more than one with pid == 0,
      for instance.  */
-  gdb_assert (pid != FAKE_PROCESS_ID);
+  gdb_assert (pid != 0);
 
   for (inf = inferior_list; inf; inf = inf->next)
     if (inf->pid == pid)
@@ -457,7 +457,7 @@ have_inferiors (void)
   struct inferior *inf;
 
   for (inf = inferior_list; inf; inf = inf->next)
-    if (inf->pid != FAKE_PROCESS_ID)
+    if (inf->pid != 0)
       return 1;
 
   return 0;
@@ -469,7 +469,7 @@ have_live_inferiors (void)
   struct inferior *inf;
 
   for (inf = inferior_list; inf; inf = inf->next)
-    if (inf->pid != FAKE_PROCESS_ID)
+    if (inf->pid != 0)
       {
        struct thread_info *tp;
        
@@ -496,7 +496,7 @@ prune_inferiors (void)
     {
       if (ss == current
          || !ss->removable
-         || ss->pid != FAKE_PROCESS_ID)
+         || ss->pid != 0)
        {
          ss_link = &ss->next;
          ss = *ss_link;
@@ -578,7 +578,7 @@ print_inferior (struct ui_out *uiout, char *requested_inferiors)
 
       ui_out_field_int (uiout, "number", inf->num);
 
-      if (kernel_debugger || (inf->pid != FAKE_PROCESS_ID))
+      if (inf->pid)
        ui_out_field_string (uiout, "target-id",
                             target_pid_to_str (pid_to_ptid (inf->pid)));
       else
@@ -704,7 +704,7 @@ inferior_command (char *args, int from_tty)
                    ? bfd_get_filename (inf->pspace->ebfd)
                    : _("<noexec>")));
 
-  if (inf->pid != FAKE_PROCESS_ID)
+  if (inf->pid != 0)
     {
       if (inf->pid != ptid_get_pid (inferior_ptid))
        {
@@ -731,9 +731,9 @@ inferior_command (char *args, int from_tty)
       set_current_program_space (inf->pspace);
     }
 
-  if (inf->pid != FAKE_PROCESS_ID && is_running (inferior_ptid))
+  if (inf->pid != 0 && is_running (inferior_ptid))
     ui_out_text (current_uiout, "(running)\n");
-  else if (inf->pid != FAKE_PROCESS_ID)
+  else if (inf->pid != 0)
     {
       ui_out_text (current_uiout, "\n");
       print_stack_frame (get_selected_frame (NULL), 1, SRC_AND_LOC);
@@ -778,7 +778,7 @@ remove_inferior_command (char *args, int from_tty)
          continue;
        }
     
-      if (inf->pid != FAKE_PROCESS_ID)
+      if (inf->pid != 0)
        {
          warning (_("Can not remove active inferior %d."), num);
          continue;
@@ -1071,8 +1071,7 @@ initialize_inferiors (void)
      can only allocate an inferior when all those modules have done
      that.  Do this after initialize_progspace, due to the
      current_program_space reference.  */
-
-  current_inferior_ = add_inferior (FAKE_PROCESS_ID);
+  current_inferior_ = add_inferior (0);
   current_inferior_->pspace = current_program_space;
   current_inferior_->aspace = current_program_space->aspace;
 
index 8683629..f9b3656 100644 (file)
@@ -31,8 +31,6 @@ struct regcache;
 struct ui_out;
 struct terminal_info;
 
-#define FAKE_PROCESS_ID -777
-
 #include "ptid.h"
 
 /* For bpstat.  */
index b8adbf7..a6e1af7 100644 (file)
@@ -1146,7 +1146,7 @@ remove_displaced_stepping_state (int pid)
 {
   struct displaced_step_inferior_state *it, **prev_next_p;
 
-  gdb_assert (pid != FAKE_PROCESS_ID);
+  gdb_assert (pid != 0);
 
   it = displaced_step_inferior_states;
   prev_next_p = &displaced_step_inferior_states;
index 10bb1e1..c9cd24d 100644 (file)
@@ -586,7 +586,7 @@ mi_output_running_pid (struct thread_info *info, void *arg)
 static int
 mi_inferior_count (struct inferior *inf, void *arg)
 {
-  if (inf->pid != FAKE_PROCESS_ID)
+  if (inf->pid != 0)
     {
       int *count_p = arg;
       (*count_p)++;
index 259ee71..b170f01 100644 (file)
@@ -361,7 +361,7 @@ mi_cmd_exec_interrupt (char *command, char **argv, int argc)
 static int
 run_one_inferior (struct inferior *inf, void *arg)
 {
-  if (inf->pid != FAKE_PROCESS_ID)
+  if (inf->pid != 0)
     {
       if (inf->pid != ptid_get_pid (inferior_ptid))
        {
@@ -564,7 +564,7 @@ print_one_inferior (struct inferior *inferior, void *xdata)
 
       ui_out_field_fmt (uiout, "id", "i%d", inferior->num);
       ui_out_field_string (uiout, "type", "process");
-      if (inferior->pid != FAKE_PROCESS_ID)
+      if (inferior->pid != 0)
        ui_out_field_int (uiout, "pid", inferior->pid);
 
       if (inferior->pspace->ebfd)
@@ -574,7 +574,7 @@ print_one_inferior (struct inferior *inferior, void *xdata)
        }
 
       data.cores = 0;
-      if (inferior->pid != FAKE_PROCESS_ID)
+      if (inferior->pid != 0)
        {
          data.pid = inferior->pid;
          iterate_over_threads (collect_cores, &data);
@@ -1796,7 +1796,7 @@ mi_cmd_remove_inferior (char *command, char **argv, int argc)
   if (!inf)
     error (_("the specified thread group does not exist"));
 
-  if (inf->pid != FAKE_PROCESS_ID)
+  if (inf->pid != 0)
     error (_("cannot remove an active inferior"));
 
   if (inf == current_inferior ())
@@ -1809,7 +1809,7 @@ mi_cmd_remove_inferior (char *command, char **argv, int argc)
        error (_("Cannot remove last inferior"));
 
       set_current_inferior (new_inferior);
-      if (new_inferior->pid != FAKE_PROCESS_ID)
+      if (new_inferior->pid != 0)
        tp = any_thread_of_process (new_inferior->pid);
       switch_to_thread (tp ? tp->ptid : null_ptid);
       set_current_program_space (new_inferior->pspace);
@@ -2064,7 +2064,7 @@ mi_cmd_execute (struct mi_parse *parse)
         an inferior with multiple threads, then a random one will be picked.
         This is not a problem -- frontend should always provide --thread if
         it wishes to operate on a specific thread.  */
-      if (inf->pid != FAKE_PROCESS_ID)
+      if (inf->pid != 0)
        tp = any_live_thread_of_process (inf->pid);
       switch_to_thread (tp ? tp->ptid : null_ptid);
       set_current_program_space (inf->pspace);
index e1f8191..903795e 100644 (file)
@@ -1181,7 +1181,7 @@ kill_or_detach (struct inferior *inf, void *args)
   struct qt_args *qt = args;
   struct thread_info *thread;
 
-  if (inf->pid == FAKE_PROCESS_ID)
+  if (inf->pid == 0)
     return 0;
 
   thread = any_thread_of_process (inf->pid);
@@ -1211,7 +1211,7 @@ print_inferior_quit_action (struct inferior *inf, void *arg)
 {
   struct ui_file *stb = arg;
 
-  if (inf->pid == FAKE_PROCESS_ID)
+  if (inf->pid == 0)
     return 0;
 
   if (inf->attach_flag)