Lint fixes from Paul Eggert (eggert@twinsun.com):
[deliverable/binutils-gdb.git] / gdb / remote-vx.c
index a8f382d19ac10ed652e25c43dc4f83fc466f9af7..91d2e87f6e0c770fd3bbb05e443584fcc29968f2 100644 (file)
@@ -26,32 +26,33 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include "gdbcore.h"
 #include "command.h"
 #include "symtab.h"
-#include "symfile.h"           /* for struct complaint */
+#include "complaints.h"
 
 #include <string.h>
 #include <errno.h>
 #include <signal.h>
 #include <fcntl.h>
 #include <sys/types.h>
-#include <sys/time.h>
 #include <sys/socket.h>
+#define malloc bogon_malloc    /* Sun claims "char *malloc()" not void * */
 #define free bogon_free                /* Sun claims "int free()" not void */
+#define realloc bogon_realloc  /* Sun claims "char *realloc()", not void * */
 #include <rpc/rpc.h>
+#undef malloc
 #undef free
+#undef realloc
 #include <sys/time.h>          /* UTek's <rpc/rpc.h> doesn't #incl this */
 #include <netdb.h>
-#include <ptrace.h>
-#include "xdr_ptrace.h"
-#include "xdr_ld.h"
-#include "xdr_rdb.h"
-#include "dbgRpcLib.h"
+#include "vx-share/ptrace.h"
+#include "vx-share/xdr_ptrace.h"
+#include "vx-share/xdr_ld.h"
+#include "vx-share/xdr_rdb.h"
+#include "vx-share/dbgRpcLib.h"
 
 #include <symtab.h>
 
 extern void symbol_file_command ();
 extern int stop_soon_quietly;          /* for wait_for_inferior */
-extern void host_convert_to_virtual ();
-extern void host_convert_from_virtual ();
 
 static int net_ptrace_clnt_call ();    /* Forward decl */
 static enum clnt_stat net_clnt_call ();        /* Forward decl */
@@ -85,7 +86,7 @@ static char *find_white_space ();
  
 /* Tell the VxWorks target system to download a file.
    The load addresses of the text, data, and bss segments are
-   stored in pTextAddr, pDataAddr, and *pBssAddr (respectively).
+   stored in *pTextAddr, *pDataAddr, and *pBssAddr (respectively).
    Returns 0 for success, -1 for failure.  */
 
 static int
@@ -106,7 +107,7 @@ net_load (filename, pTextAddr, pDataAddr, pBssAddr)
        The load on the target side can take quite a while, easily
        more than 10 seconds.  The user can kill this call by typing
        CTRL-C if there really is a problem with the load.  
-       
+
        Do not change the tv_sec value without checking -- select() imposes
        a limit of 10**8 on it for no good reason that I can see...  */
 
@@ -118,7 +119,7 @@ net_load (filename, pTextAddr, pDataAddr, pBssAddr)
 
     if (status == RPC_SUCCESS)
       {
-        if (*ldstruct.name == NULL)    /* load failed on VxWorks side */
+        if (*ldstruct.name == 0)       /* load failed on VxWorks side */
           return -1;
        *pTextAddr = ldstruct.txt_addr;
        *pDataAddr = ldstruct.data_addr;
@@ -384,7 +385,11 @@ vx_read_register (regno)
   
   ptrace_in.pid = inferior_pid;
   ptrace_out.info.more_data = (caddr_t) &out_data;
+#ifndef I80960
   out_data.len   = 18 * REGISTER_RAW_SIZE (0);         /* FIXME 68k hack */
+#else
+  out_data.len = (16 + 16 + 3) * REGISTER_RAW_SIZE (0);
+#endif
   out_data.bytes = (caddr_t) registers;
   
   status = net_ptrace_clnt_call (PTRACE_GETREGS, &ptrace_in, &ptrace_out);
@@ -398,49 +403,40 @@ vx_read_register (regno)
   
 #ifdef I80960
 
-      bcopy ((char *) inferior_registers.r_lreg,
-            &registers[REGISTER_BYTE (R0_REGNUM)], 16 * sizeof (int));
-      bcopy ((char *) inferior_registers.r_greg,
-            &registers[REGISTER_BYTE (G0_REGNUM)], 16 * sizeof (int));
-
-      /* Don't assume that a location in registers[] is properly aligned.  */
-
-      bcopy ((char *) &inferior_registers.r_pcw,
-            &registers[REGISTER_BYTE (PCW_REGNUM)], sizeof (int));
-      bcopy ((char *) &inferior_registers.r_acw,
-            &registers[REGISTER_BYTE (ACW_REGNUM)], sizeof (int));
-      bcopy ((char *) &inferior_registers.r_lreg[2],   /* r2 (RIP) -> IP */
-            &registers[REGISTER_BYTE (IP_REGNUM)], sizeof (int));
-      bcopy ((char *) &inferior_registers.r_tcw,
-            &registers[REGISTER_BYTE (TCW_REGNUM)], sizeof (int));
-
-      /* If the target has floating point registers, fetch them.
-        Otherwise, zero the floating point register values in
-        registers[] for good measure, even though we might not
-        need to.  */
-
-      if (target_has_fp)
-       {
-         ptrace_in.pid = inferior_pid;
-         ptrace_out.info.more_data = (caddr_t) &inferior_fp_registers;
-         status = net_ptrace_clnt_call (PTRACE_GETFPREGS, &ptrace_in, &ptrace_out);
-         if (status)
-           error (rpcerr);
-         if (ptrace_out.status == -1)
-           {
-             errno = ptrace_out.errno;
-             perror_with_name ("net_ptrace_clnt_call(PTRACE_GETFPREGS)");
-           }
-         
-         bcopy (&inferior_fp_registers, &registers[REGISTER_BYTE (FP0_REGNUM)],
-                REGISTER_RAW_SIZE (FP0_REGNUM) * 4);
-       }
-      else
-       {
-         bzero ((char *) &registers[REGISTER_BYTE (FP0_REGNUM)],
-                REGISTER_RAW_SIZE (FP0_REGNUM) * 4);
-       }
-
+  {
+    /* If the target has floating point registers, fetch them.
+       Otherwise, zero the floating point register values in
+       registers[] for good measure, even though we might not
+       need to.  */
+    /* @@ Can't use this -- the rdb library for the 960 target
+       doesn't support setting or retrieving FP regs.  KR  */
+#if 0
+    struct fp_status inferior_fp_registers;
+
+    if (target_has_fp)
+      {
+       ptrace_in.pid = inferior_pid;
+       ptrace_out.info.more_data = (caddr_t) &inferior_fp_registers;
+       status = net_ptrace_clnt_call (PTRACE_GETFPREGS,
+                                      &ptrace_in, &ptrace_out);
+       if (status)
+         error (rpcerr);
+       if (ptrace_out.status == -1)
+         {
+           errno = ptrace_out.errno;
+           perror_with_name ("net_ptrace_clnt_call(PTRACE_GETFPREGS)");
+         }
+
+       bcopy (&inferior_fp_registers, &registers[REGISTER_BYTE (FP0_REGNUM)],
+              REGISTER_RAW_SIZE (FP0_REGNUM) * 4);
+      }
+    else
+      {
+       bzero ((char *) &registers[REGISTER_BYTE (FP0_REGNUM)],
+              REGISTER_RAW_SIZE (FP0_REGNUM) * 4);
+      }
+#endif
+  }
 #else  /* not 960, thus must be 68000:  FIXME!  */
 
   if (target_has_fp)
@@ -476,7 +472,8 @@ vx_read_register (regno)
 static void
 vx_prepare_to_store ()
 {
-  vx_read_register (-1);
+  /* Fetch all registers, if any of them are not yet fetched.  */
+  read_register_bytes (0, NULL, REGISTER_BYTES);
 }
 
 
@@ -507,20 +504,7 @@ vx_write_register (regno)
 
 #ifdef I80960
 
-  /* FIXME */
-  bcopy (&registers[REGISTER_BYTE (R0_REGNUM)],
-        (char *) inferior_registers.r_lreg, 16 * sizeof (int));
-  bcopy (&registers[REGISTER_BYTE (G0_REGNUM)],
-        (char *) inferior_registers.r_greg, 16 * sizeof (int));
-
-  /* Don't assume that a location in registers[] is properly aligned.  */
-
-  bcopy (&registers[REGISTER_BYTE (PCW_REGNUM)],
-        (char *) &inferior_registers.r_pcw, sizeof (int));
-  bcopy (&registers[REGISTER_BYTE (ACW_REGNUM)],
-        (char *) &inferior_registers.r_acw, sizeof (int));
-  bcopy (&registers[REGISTER_BYTE (TCW_REGNUM)],
-        (char *) &inferior_registers.r_tcw, sizeof (int));
+  in_data.len = (16 + 16 + 3) * sizeof (REGISTER_TYPE);
 
 #else  /* not 960 -- assume 68k -- FIXME */
 
@@ -548,10 +532,10 @@ vx_write_register (regno)
 
 
 #ifdef I80960
-
-      bcopy (&registers[REGISTER_BYTE (FP0_REGNUM)], &inferior_fp_registers,
-            sizeof inferior_fp_registers.fps_regs);
-
+#if 0 /* @@ Not supported by target.  */
+      in_data.bytes = &registers[REGISTER_BYTE (FP0_REGNUM)];
+      in_data.len = 4 * REGISTER_RAW_SIZE (FP0_REGNUM);
+#endif
 #else  /* not 960 -- assume 68k -- FIXME */
 
       in_data.bytes = &registers[REGISTER_BYTE (FP0_REGNUM)];
@@ -712,7 +696,7 @@ vx_load_command (arg_string, from_tty)
   immediate_quit--;
 
   /* FIXME, for now we ignore data_addr and bss_addr.  */
-  (void) symbol_file_add (arg_string, from_tty, text_addr, 0, 0);
+  symbol_file_add (arg_string, from_tty, text_addr, 0, 0, 0);
 }
 
 #ifdef FIXME  /* Not ready for prime time */
@@ -825,7 +809,7 @@ vx_lookup_symbol (name, pAddr)
   status = net_clnt_call (VX_SYMBOL_INQ, xdr_wrapstring, &name,
                          xdr_SYMBOL_ADDR, &symbolAddr);
   if (status != RPC_SUCCESS) {
-      complain (&cant_contact_target, 0);
+      complain (&cant_contact_target);
       return -1;
   }
 
@@ -1036,7 +1020,7 @@ add_symbol_stub (arg)
   struct ldfile *pLoadFile = (struct ldfile *)arg;
 
   printf("\t%s: ", pLoadFile->name);
-  (void) symbol_file_add (pLoadFile->name, 0, pLoadFile->txt_addr, 0, 0);
+  symbol_file_add (pLoadFile->name, 0, pLoadFile->txt_addr, 0, 0, 0);
   printf ("ok\n");
   return 1;
 }
@@ -1140,8 +1124,7 @@ vx_open (args, from_tty)
   clnt_freeres (pClient, xdr_ldtabl, &loadTable);
 }
 \f
-/* attach_command --
-   takes a task started up outside of gdb and ``attaches'' to it.
+/* Takes a task started up outside of gdb and ``attaches'' to it.
    This stops it cold in its tracks and allows us to start tracing it.  */
 
 static void
@@ -1155,8 +1138,6 @@ vx_attach (args, from_tty)
   Ptrace_return ptrace_out;
   int status;
 
-  dont_repeat();
-
   if (!args)
     error_no_arg ("process-id to attach");
 
@@ -1184,23 +1165,6 @@ vx_attach (args, from_tty)
   push_target (&vx_run_ops);
   inferior_pid = pid;
   vx_running = 0;
-
-  mark_breakpoints_out ();
-
-  /* Set up the "saved terminal modes" of the inferior
-     based on what modes we are starting it with.  */
-  target_terminal_init ();
-
-  /* Install inferior's terminal modes.  */
-  target_terminal_inferior ();
-
-  /* We will get a task spawn event immediately.  */
-  init_wait_for_inferior ();
-  clear_proceed_status ();
-  stop_soon_quietly = 1;
-  wait_for_inferior ();
-  stop_soon_quietly = 0;
-  normal_stop ();
 }
 
 
@@ -1252,19 +1216,13 @@ vx_detach (args, from_tty)
 /* vx_kill -- takes a running task and wipes it out.  */
 
 static void
-vx_kill (args, from_tty)
-     char *args;
-     int from_tty;
+vx_kill ()
 {
   Rptrace ptrace_in;
   Ptrace_return ptrace_out;
   int status;
 
-  if (args)
-    error ("Argument given to VxWorks \"kill\".");
-
-  if (from_tty)
-      printf ("Killing pid %s.\n", local_hex_string(inferior_pid));
+  printf ("Killing pid %s.\n", local_hex_string(inferior_pid));
 
   bzero ((char *)&ptrace_in,  sizeof (ptrace_in));
   bzero ((char *)&ptrace_out, sizeof (ptrace_out));
@@ -1272,14 +1230,15 @@ vx_kill (args, from_tty)
 
   status = net_ptrace_clnt_call (PTRACE_KILL, &ptrace_in, &ptrace_out);
   if (status == -1)
-    error (rpcerr);
-  if (ptrace_out.status == -1)
+    warning (rpcerr);
+  else if (ptrace_out.status == -1)
     {
       errno = ptrace_out.errno;
       perror_with_name ("Killing VxWorks process");
     }
 
-  /* If it gives good status, the process is *gone*, no events remain.  */
+  /* If it gives good status, the process is *gone*, no events remain.
+     If the kill failed, assume the process is gone anyhow.  */
   inferior_pid = 0;
   pop_target ();       /* go back to non-executing VxWorks connection */
 }
@@ -1351,7 +1310,7 @@ Specify the name of the machine to connect to.",
        vx_open, vx_close, vx_attach, 0, /* vx_detach, */
        0, 0, /* resume, wait */
        0, 0, /* read_reg, write_reg */
-       0, host_convert_to_virtual, host_convert_from_virtual,  /* prep_to_store, */
+       0, /* prep_to_store, */
        vx_xfer_memory, vx_files_info,
        0, 0, /* insert_breakpoint, remove_breakpoint */
        0, 0, 0, 0, 0,  /* terminal stuff */
@@ -1359,6 +1318,8 @@ Specify the name of the machine to connect to.",
        vx_load_command,
        vx_lookup_symbol,
        vx_create_inferior, 0,  /* mourn_inferior */
+       0, /* can_run */
+       0, /* notice_signals */
        core_stratum, 0, /* next */
        1, 1, 0, 0, 0,  /* all mem, mem, stack, regs, exec */
        0, 0,                   /* Section pointers */
@@ -1373,7 +1334,7 @@ struct target_ops vx_run_ops = {
        vx_proc_open, vx_proc_close, 0, vx_detach, /* vx_attach */
        vx_resume, vx_wait,
        vx_read_register, vx_write_register,
-       vx_prepare_to_store, host_convert_to_virtual, host_convert_from_virtual,
+       vx_prepare_to_store,
        vx_xfer_memory, vx_run_files_info,
        vx_insert_breakpoint, vx_remove_breakpoint,
        0, 0, 0, 0, 0,  /* terminal stuff */
@@ -1381,6 +1342,8 @@ struct target_ops vx_run_ops = {
        vx_load_command,
        vx_lookup_symbol,
        0, vx_mourn_inferior,
+       0,  /* can_run */
+       0, /* notice_signals */
        process_stratum, 0, /* next */
        0, 1, 1, 1, 1,  /* all mem, mem, stack, regs, exec */
                        /* all_mem is off to avoid spurious msg in "i files" */
This page took 0.027789 seconds and 4 git commands to generate.