]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - gdb/win32-nat.c
import gdb-1999-07-07 post reformat
[thirdparty/binutils-gdb.git] / gdb / win32-nat.c
index 68c26496635a17e339b88f6244b770b84598888b..71d8efd580932161e0dea165b05d6c6a1d17b7f1 100644 (file)
@@ -1,8 +1,7 @@
 /* Target-vector operations for controlling win32 child processes, for GDB.
-   Copyright 1995
-   Free Software Foundation, Inc.
-
+   Copyright 1995, 1996, 1997, 1998 Free Software Foundation, Inc.
    Contributed by Cygnus Support.
+
    This file is part of GDB.
 
    This program is free software; you can redistribute it and/or modify
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
-   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   Foundation, Inc., 59 Temple Place - Suite 330,
+   Boston, MA 02111-1307, USA.
+ */
 
 /* by Steve Chamberlain, sac@cygnus.com */
 
+/* We assume we're being built with and will be used for cygwin.  */
+
 #include "defs.h"
 #include "frame.h"             /* required by inferior.h */
 #include "inferior.h"
 #include <signal.h>
 #include <sys/types.h>
 #include <fcntl.h>
+#include <stdlib.h>
+
+#ifdef _MSC_VER
+#include "windefs.h"
+#else /* other WIN32 compiler */
 #include <windows.h>
+#endif
+
 #include "buildsym.h"
+#include "symfile.h"
+#include "objfiles.h"
 #include "gdb_string.h"
-#include "thread.h"
+#include "gdbthread.h"
 #include "gdbcmd.h"
-
-#define CHECK(x) check (x, __FILE__,__LINE__)
-#define DEBUG(x) if (remote_debug) printf x
-
+#include <sys/param.h>
+#include <unistd.h>
+
+/* The ui's event loop. */
+extern int (*ui_loop_hook) PARAMS ((int signo));
+
+/* If we're not using the old Cygwin header file set, define the
+   following which never should have been in the generic Win32 API
+   headers in the first place since they were our own invention... */
+#ifndef _GNU_H_WINDOWS_H
+#define FLAG_TRACE_BIT 0x100
+#define CONTEXT_DEBUGGER (CONTEXT_FULL | CONTEXT_FLOATING_POINT)
+#endif
+
+/* The string sent by cygwin when it processes a signal.
+   FIXME: This should be in a cygwin include file. */
+#define CYGWIN_SIGNAL_STRING "cygwin: signal"
+
+#define CHECK(x)       check (x, __FILE__,__LINE__)
+#define DEBUG_EXEC(x)  if (debug_exec)         printf x
+#define DEBUG_EVENTS(x)        if (debug_events)       printf x
+#define DEBUG_MEM(x)   if (debug_memory)       printf x
+#define DEBUG_EXCEPT(x)        if (debug_exceptions)   printf x
 
 /* Forward declaration */
 extern struct target_ops child_ops;
 
-/* The most recently read context. Inspect ContextFlags to see what 
-   bits are valid. */
+static void child_stop PARAMS ((void));
+static int win32_child_thread_alive PARAMS ((int));
+void child_kill_inferior PARAMS ((void));
+
+static int last_sig = 0;       /* Set if a signal was received from the
+                                  debugged process */
 
-static CONTEXT context;
+/* Thread information structure used to track information that is
+   not available in gdb's thread structure. */
+typedef struct thread_info_struct
+  {
+    struct thread_info_struct *next;
+    DWORD id;
+    HANDLE h;
+    char *name;
+    int suspend_count;
+    CONTEXT context;
+  }
+thread_info;
+
+static thread_info thread_head =
+{NULL};
 
 /* The process and thread handles for the above context. */
 
-static HANDLE current_process;
-static HANDLE current_thread;
-static int current_process_id;
-static int current_thread_id;
+static DEBUG_EVENT current_event;      /* The current debug event from
+                                          WaitForDebugEvent */
+static HANDLE current_process_handle;  /* Currently executing process */
+static thread_info *current_thread;    /* Info on currently selected thread */
+static DWORD main_thread_id;   /* Thread ID of the main thread */
 
 /* Counts of things. */
 static int exception_count = 0;
@@ -63,11 +113,15 @@ static int event_count = 0;
 /* User options. */
 static int new_console = 0;
 static int new_group = 0;
+static int debug_exec = 0;     /* show execution */
+static int debug_events = 0;   /* show events from kernel */
+static int debug_memory = 0;   /* show target memory accesses */
+static int debug_exceptions = 0;       /* show target exceptions */
 
 /* This vector maps GDB's idea of a register's number into an address
-   in the win32 exception context vector. 
+   in the win32 exception context vector.
 
-   It also contains the bit mask needed to load the register in question.  
+   It also contains the bit mask needed to load the register in question.
 
    One day we could read a reg, we could inspect the context we
    already have loaded, if it doesn't have the bit set that we need,
@@ -77,42 +131,35 @@ static int new_group = 0;
    the other regs of the group, and then we copy the info in and set
    out bit. */
 
-struct regmappings
-  {
-    char *incontext;
-    int mask;
-  };
-
-static const struct regmappings
-  mappings[] =
-{
-  {(char *) &context.Eax, CONTEXT_INTEGER},
-  {(char *) &context.Ecx, CONTEXT_INTEGER},
-  {(char *) &context.Edx, CONTEXT_INTEGER},
-  {(char *) &context.Ebx, CONTEXT_INTEGER},
-  {(char *) &context.Esp, CONTEXT_CONTROL},
-  {(char *) &context.Ebp, CONTEXT_CONTROL},
-  {(char *) &context.Esi, CONTEXT_INTEGER},
-  {(char *) &context.Edi, CONTEXT_INTEGER},
-  {(char *) &context.Eip, CONTEXT_CONTROL},
-  {(char *) &context.EFlags, CONTEXT_CONTROL},
-  {(char *) &context.SegCs, CONTEXT_SEGMENTS},
-  {(char *) &context.SegSs, CONTEXT_SEGMENTS},
-  {(char *) &context.SegDs, CONTEXT_SEGMENTS},
-  {(char *) &context.SegEs, CONTEXT_SEGMENTS},
-  {(char *) &context.SegFs, CONTEXT_SEGMENTS},
-  {(char *) &context.SegGs, CONTEXT_SEGMENTS},
-  {&context.FloatSave.RegisterArea[0 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[1 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[2 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[3 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[4 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[5 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[6 * 10], CONTEXT_FLOATING_POINT},
-  {&context.FloatSave.RegisterArea[7 * 10], CONTEXT_FLOATING_POINT},
+#define context_offset(x) ((int)&(((CONTEXT *)NULL)->x))
+static const int mappings[] =
+{
+  context_offset (Eax),
+  context_offset (Ecx),
+  context_offset (Edx),
+  context_offset (Ebx),
+  context_offset (Esp),
+  context_offset (Ebp),
+  context_offset (Esi),
+  context_offset (Edi),
+  context_offset (Eip),
+  context_offset (EFlags),
+  context_offset (SegCs),
+  context_offset (SegSs),
+  context_offset (SegDs),
+  context_offset (SegEs),
+  context_offset (SegFs),
+  context_offset (SegGs),
+  context_offset (FloatSave.RegisterArea[0 * 10]),
+  context_offset (FloatSave.RegisterArea[1 * 10]),
+  context_offset (FloatSave.RegisterArea[2 * 10]),
+  context_offset (FloatSave.RegisterArea[3 * 10]),
+  context_offset (FloatSave.RegisterArea[4 * 10]),
+  context_offset (FloatSave.RegisterArea[5 * 10]),
+  context_offset (FloatSave.RegisterArea[6 * 10]),
+  context_offset (FloatSave.RegisterArea[7 * 10]),
 };
 
-
 /* This vector maps the target's idea of an exception (extracted
    from the DEBUG_EVENT structure) to GDB's idea. */
 
@@ -122,7 +169,6 @@ struct xlate_exception
     enum target_signal us;
   };
 
-
 static const struct xlate_exception
   xlate[] =
 {
@@ -133,71 +179,200 @@ static const struct xlate_exception
   {EXCEPTION_SINGLE_STEP, TARGET_SIGNAL_TRAP},
   {-1, -1}};
 
+/* Find a thread record given a thread id.
+   If get_context then also retrieve the context for this
+   thread. */
+static thread_info *
+thread_rec (DWORD id, int get_context)
+{
+  thread_info *th;
+
+  for (th = &thread_head; (th = th->next) != NULL;)
+    if (th->id == id)
+      {
+       if (!th->suspend_count && get_context)
+         {
+           if (get_context > 0)
+             th->suspend_count = SuspendThread (th->h) + 1;
+           else if (get_context < 0)
+             th->suspend_count = -1;
+
+           th->context.ContextFlags = CONTEXT_DEBUGGER;
+           GetThreadContext (th->h, &th->context);
+         }
+       return th;
+      }
+
+  return NULL;
+}
 
-static void 
-check (BOOL ok, const char *file, int line)
+/* Add a thread to the thread list */
+static thread_info *
+child_add_thread (DWORD id, HANDLE h)
 {
-  if (!ok)
-    printf_filtered ("error return %s:%d was %d\n", file, line, GetLastError ());
+  thread_info *th;
+
+  if ((th = thread_rec (id, FALSE)))
+    return th;
+
+  th = (thread_info *) xmalloc (sizeof (*th));
+  memset (th, 0, sizeof (*th));
+  th->id = id;
+  th->h = h;
+  th->next = thread_head.next;
+  thread_head.next = th;
+  add_thread (id);
+  return th;
 }
 
+/* Clear out any old thread list and reintialize it to a
+   pristine state. */
 static void
-child_fetch_inferior_registers (int r)
+child_init_thread_list ()
 {
-  if (r < 0)
+  thread_info *th = &thread_head;
+
+  DEBUG_EVENTS (("gdb: child_init_thread_list\n"));
+  init_thread_list ();
+  while (th->next != NULL)
     {
-      for (r = 0; r < NUM_REGS; r++)
-       child_fetch_inferior_registers (r);
+      thread_info *here = th->next;
+      th->next = here->next;
+      (void) CloseHandle (here->h);
+      free (here);
     }
-  else
+}
+
+/* Delete a thread from the list of threads */
+static void
+child_delete_thread (DWORD id)
+{
+  thread_info *th;
+
+  if (info_verbose)
+    printf_unfiltered ("[Deleting %s]\n", target_pid_to_str (id));
+  delete_thread (id);
+
+  for (th = &thread_head;
+       th->next != NULL && th->next->id != id;
+       th = th->next)
+    continue;
+
+  if (th->next != NULL)
     {
-      supply_register (r, mappings[r].incontext);
+      thread_info *here = th->next;
+      th->next = here->next;
+      CloseHandle (here->h);
+      free (here);
     }
 }
 
 static void
-child_store_inferior_registers (int r)
+check (BOOL ok, const char *file, int line)
+{
+  if (!ok)
+    printf_filtered ("error return %s:%d was %d\n", file, line, GetLastError ());
+}
+
+static void
+do_child_fetch_inferior_registers (int r)
 {
-  if (r < 0)
+  if (r >= 0)
+    supply_register (r, ((char *) &current_thread->context) + mappings[r]);
+  else
     {
       for (r = 0; r < NUM_REGS; r++)
-       child_store_inferior_registers (r);
+       do_child_fetch_inferior_registers (r);
     }
+}
+
+static void
+child_fetch_inferior_registers (int r)
+{
+  current_thread = thread_rec (inferior_pid, TRUE);
+  do_child_fetch_inferior_registers (r);
+}
+
+static void
+do_child_store_inferior_registers (int r)
+{
+  if (r >= 0)
+    read_register_gen (r, ((char *) &current_thread->context) + mappings[r]);
   else
     {
-      read_register_gen (r, mappings[r].incontext);
+      for (r = 0; r < NUM_REGS; r++)
+       do_child_store_inferior_registers (r);
     }
 }
 
+/* Store a new register value into the current thread context */
+static void
+child_store_inferior_registers (int r)
+{
+  current_thread = thread_rec (inferior_pid, TRUE);
+  do_child_store_inferior_registers (r);
+}
 
 /* Wait for child to do something.  Return pid of child, or -1 in case
    of error; store status through argument pointer OURSTATUS.  */
 
-
-static void 
-handle_load_dll (DEBUG_EVENT * event)
+static int
+handle_load_dll (PTR dummy)
 {
+  LOAD_DLL_DEBUG_INFO *event = &current_event.u.LoadDll;
   DWORD dll_name_ptr;
   DWORD done;
+  char dll_buf[MAX_PATH + 1];
+  char *p, *dll_name = NULL, *dll_basename;
+  struct objfile *objfile;
+  MEMORY_BASIC_INFORMATION minfo;
+
+  dll_buf[0] = dll_buf[sizeof (dll_buf) - 1] = '\0';
+
+  /* The following code attempts to find the name of the dll by reading the
+     name from the processes memory.  Unfortunately it doesn't work right.
+     Doing this the "right way" for Windows is very difficult.  FIXME */
+#ifdef DOESNT_WORK
+  memset (&minfo, 0, sizeof minfo);
+  if (VirtualQueryEx (current_process_handle, (LPCVOID) event->lpBaseOfDll,
+                     &minfo, sizeof (minfo)) && minfo.BaseAddress)
+    {
+      DWORD len;
+      IMAGE_DOS_HEADER *hmm0 = (IMAGE_DOS_HEADER *) minfo.BaseAddress;
+      HMODULE hmm = (HMODULE) (((DWORD) hmm0) + hmm0->e_lfanew);
 
-  ReadProcessMemory (current_process,
-                    (DWORD) event->u.LoadDll.lpImageName,
-                    (char *) &dll_name_ptr,
-                    sizeof (dll_name_ptr), &done);
-
-  /* See if we could read the address of a string, and that the 
-     address isn't null. */
+      if ((len = GetModuleFileName (hmm, dll_buf, MAX_PATH)))
+       {
+         dll_name = dll_buf;
+         dll_name[len] = '\0';
+       }
+    }
+#endif
 
-  if (done == sizeof (dll_name_ptr) && dll_name_ptr)
+  /* Attempt to read the name of the dll that was detected.
+     This is documented to work only when actively debugging
+     a program.  It will not work for attached processes. */
+  if (dll_name == NULL || *dll_name == '\0')
     {
-      char *dll_name;
-      int size = event->u.LoadDll.fUnicode ? sizeof (WCHAR) : sizeof (char);
+      int size = event->fUnicode ? sizeof (WCHAR) : sizeof (char);
       int len = 0;
       char b[2];
+
+      ReadProcessMemory (current_process_handle,
+                        (LPCVOID) event->lpImageName,
+                        (char *) &dll_name_ptr,
+                        sizeof (dll_name_ptr), &done);
+
+      /* See if we could read the address of a string, and that the
+         address isn't null. */
+
+      if (done != sizeof (dll_name_ptr) || !dll_name_ptr)
+       return 1;
+
       do
        {
-         ReadProcessMemory (current_process,
-                            dll_name_ptr + len * size,
+         ReadProcessMemory (current_process_handle,
+                            (LPCVOID) (dll_name_ptr + len * size),
                             &b,
                             size,
                             &done);
@@ -205,14 +380,13 @@ handle_load_dll (DEBUG_EVENT * event)
        }
       while ((b[0] != 0 || b[size - 1] != 0) && done == size);
 
-
       dll_name = alloca (len);
 
-      if (event->u.LoadDll.fUnicode)
+      if (event->fUnicode)
        {
          WCHAR *unicode_dll_name = (WCHAR *) alloca (len * sizeof (WCHAR));
-         ReadProcessMemory (current_process,
-                            dll_name_ptr,
+         ReadProcessMemory (current_process_handle,
+                            (LPCVOID) dll_name_ptr,
                             unicode_dll_name,
                             len * sizeof (WCHAR),
                             &done);
@@ -223,55 +397,170 @@ handle_load_dll (DEBUG_EVENT * event)
        }
       else
        {
-         ReadProcessMemory (current_process,
-                            dll_name_ptr,
+         ReadProcessMemory (current_process_handle,
+                            (LPCVOID) dll_name_ptr,
                             dll_name,
                             len,
                             &done);
        }
+    }
 
-      /* FIXME!! It would be nice to define one symbol which pointed to the 
-         front of the dll if we can't find any symbols. */
+  if (!dll_name)
+    return 1;
 
-      context.ContextFlags = CONTEXT_FULL;
-      GetThreadContext (current_thread, &context);
+  while ((p = strchr (dll_name, '\\')))
+    *p = '/';
 
-      symbol_file_add (dll_name, 0, (int) event->u.LoadDll.lpBaseOfDll, 0, 0, 0);
+  /* FIXME!! It would be nice to define one symbol which pointed to the
+     front of the dll if we can't find any symbols. */
 
-      /* We strip off the path of the dll for tidiness. */
-      if (strrchr (dll_name, '\\'))
-       dll_name = strrchr (dll_name, '\\') + 1;
-      printf_unfiltered ("%x:%s\n", event->u.LoadDll.lpBaseOfDll, dll_name);
-    }
+  if (!(dll_basename = strrchr (dll_name, '/')))
+    dll_basename = dll_name;
+  else
+    dll_basename++;
+
+  ALL_OBJFILES (objfile)
+  {
+    char *objfile_basename;
+    objfile_basename = strrchr (objfile->name, '/');
+
+    if (objfile_basename &&
+       strcmp (dll_basename, objfile_basename + 1) == 0)
+      {
+       printf_unfiltered ("%x:%s (symbols previously loaded)\n",
+                          event->lpBaseOfDll, dll_name);
+       goto out;
+      }
+  }
+
+  /* The symbols in a dll are offset by 0x1000, which is the
+     the offset from 0 of the first byte in an image - because
+     of the file header and the section alignment.
+
+     FIXME: Is this the real reason that we need the 0x1000 ? */
+
+  printf_unfiltered ("%x:%s", event->lpBaseOfDll, dll_name);
+  symbol_file_add (dll_name, 0, (int) event->lpBaseOfDll + 0x1000, 0, 0, 0, 0, 1);
+  printf_unfiltered ("\n");
+
+out:
+  return 1;
 }
 
+/* Handle DEBUG_STRING output from child process.
+   Cygwin prepends its messages with a "cygwin:".  Interpret this as
+   a Cygwin signal.  Otherwise just print the string as a warning. */
+static int
+handle_output_debug_string (struct target_waitstatus *ourstatus)
+{
+  char *s;
+  int gotasig = FALSE;
 
-static void
-handle_exception (DEBUG_EVENT * event, struct target_waitstatus *ourstatus)
+  if (!target_read_string
+    ((CORE_ADDR) current_event.u.DebugString.lpDebugStringData, &s, 1024, 0)
+      || !s || !*s)
+    return gotasig;
+
+  if (strncmp (s, CYGWIN_SIGNAL_STRING, sizeof (CYGWIN_SIGNAL_STRING) - 1))
+    {
+      warning (s);
+    }
+  else
+    {
+      char *p;
+      /*last_sig = */ strtol (s + sizeof (CYGWIN_SIGNAL_STRING) - 1, &p, 0);
+      gotasig = target_signal_from_host (last_sig);
+      ourstatus->value.sig = gotasig;
+      if (gotasig)
+       ourstatus->kind = TARGET_WAITKIND_STOPPED;
+    }
+
+  free (s);
+  return gotasig;
+}
+
+static int
+handle_exception (struct target_waitstatus *ourstatus)
 {
   int i;
   int done = 0;
+  thread_info *th;
+
   ourstatus->kind = TARGET_WAITKIND_STOPPED;
 
-  for (i = 0; !done && xlate[i].us > 0; i++)
-    {
-      if (xlate[i].them == event->u.Exception.ExceptionRecord.ExceptionCode)
-       {
-         ourstatus->value.sig = xlate[i].us;
-         done = 1;
-         break;
-       }
-    }
+  /* Record the context of the current thread */
+  th = thread_rec (current_event.dwThreadId, -1);
 
-  if (!done)
+  switch (current_event.u.Exception.ExceptionRecord.ExceptionCode)
     {
-      printf_unfiltered ("Want to know about exception code %08x\n",
-                        event->u.Exception.ExceptionRecord.ExceptionCode);
+    case EXCEPTION_ACCESS_VIOLATION:
+      DEBUG_EXCEPT (("gdb: Target exception ACCESS_VIOLATION at 0x%08x\n",
+              current_event.u.Exception.ExceptionRecord.ExceptionAddress));
+      ourstatus->value.sig = TARGET_SIGNAL_SEGV;
+      break;
+    case STATUS_STACK_OVERFLOW:
+      DEBUG_EXCEPT (("gdb: Target exception STACK_OVERFLOW at 0x%08x\n",
+              current_event.u.Exception.ExceptionRecord.ExceptionAddress));
+      ourstatus->value.sig = TARGET_SIGNAL_SEGV;
+      break;
+    case EXCEPTION_BREAKPOINT:
+      DEBUG_EXCEPT (("gdb: Target exception BREAKPOINT at 0x%08x\n",
+              current_event.u.Exception.ExceptionRecord.ExceptionAddress));
+      ourstatus->value.sig = TARGET_SIGNAL_TRAP;
+      break;
+    case DBG_CONTROL_C:
+      DEBUG_EXCEPT (("gdb: Target exception CONTROL_C at 0x%08x\n",
+              current_event.u.Exception.ExceptionRecord.ExceptionAddress));
+      ourstatus->value.sig = TARGET_SIGNAL_INT;
+      /* User typed CTRL-C.  Continue with this status */
+      last_sig = SIGINT;       /* FIXME - should check pass state */
+      break;
+    case EXCEPTION_SINGLE_STEP:
+      DEBUG_EXCEPT (("gdb: Target exception SINGLE_STEP at 0x%08x\n",
+              current_event.u.Exception.ExceptionRecord.ExceptionAddress));
+      ourstatus->value.sig = TARGET_SIGNAL_TRAP;
+      break;
+    default:
+      /* This may be a structured exception handling exception.  In
+         that case, we want to let the program try to handle it, and
+         only break if we see the exception a second time.  */
+      if (current_event.u.Exception.dwFirstChance)
+       return 0;
+
+      printf_unfiltered ("gdb: unknown target exception 0x%08x at 0x%08x\n",
+                   current_event.u.Exception.ExceptionRecord.ExceptionCode,
+               current_event.u.Exception.ExceptionRecord.ExceptionAddress);
       ourstatus->value.sig = TARGET_SIGNAL_UNKNOWN;
+      break;
     }
-  context.ContextFlags = CONTEXT_FULL;
-  GetThreadContext (current_thread, &context);
   exception_count++;
+  return 1;
+}
+
+/* Resume all artificially suspended threads if we are continuing
+   execution */
+static BOOL
+child_continue (DWORD continue_status, int id)
+{
+  int i;
+  thread_info *th;
+  BOOL res;
+
+  DEBUG_EVENTS (("ContinueDebugEvent (cpid=%d, ctid=%d, DBG_CONTINUE);\n",
+                current_event.dwProcessId, current_event.dwThreadId));
+  res = ContinueDebugEvent (current_event.dwProcessId,
+                           current_event.dwThreadId,
+                           continue_status);
+  if (res)
+    for (th = &thread_head; (th = th->next) != NULL;)
+      if (((id == -1) || (id == th->id)) && th->suspend_count)
+       {
+         for (i = 0; i < th->suspend_count; i++)
+           (void) ResumeThread (th->h);
+         th->suspend_count = 0;
+       }
+
+  return res;
 }
 
 static int
@@ -279,63 +568,121 @@ child_wait (int pid, struct target_waitstatus *ourstatus)
 {
   /* We loop when we get a non-standard exception rather than return
      with a SPURIOUS because resume can try and step or modify things,
-     which needs a current_thread.  But some of these exceptions mark
+     which needs a current_thread->h.  But some of these exceptions mark
      the birth or death of threads, which mean that the current thread
      isn't necessarily what you think it is. */
 
   while (1)
     {
-      DEBUG_EVENT event;
-      BOOL t = WaitForDebugEvent (&event, INFINITE);
+      DWORD continue_status;
+      BOOL debug_event = WaitForDebugEvent (&current_event, 20);
+      char *p;
+      thread_info *th;
+      int sig;
 
-      DEBUG (("%d = WaitForDebugEvent() code=%d pid=%d tid=%d)\n",
-             t,
-             event.dwDebugEventCode,
-             event.dwProcessId,
-             event.dwThreadId));
-
-      event_count++;
+      if (debug_event)
+       {
+         event_count++;
+
+         continue_status = DBG_CONTINUE;
+
+         switch (current_event.dwDebugEventCode)
+           {
+           case CREATE_THREAD_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%x code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "CREATE_THREAD_DEBUG_EVENT"));
+             /* Record the existence of this thread */
+             child_add_thread (current_event.dwThreadId,
+                               current_event.u.CreateThread.hThread);
+             if (info_verbose)
+               printf_unfiltered ("[New %s]\n",
+                             target_pid_to_str (current_event.dwThreadId));
+             break;
+
+           case EXIT_THREAD_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "EXIT_THREAD_DEBUG_EVENT"));
+             child_delete_thread (current_event.dwThreadId);
+             break;
+
+           case CREATE_PROCESS_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "CREATE_PROCESS_DEBUG_EVENT"));
+             current_process_handle = current_event.u.CreateProcessInfo.hProcess;
+
+             main_thread_id = inferior_pid = current_event.dwThreadId;
+             /* Add the main thread */
+             current_thread = child_add_thread (inferior_pid,
+                                current_event.u.CreateProcessInfo.hThread);
+             break;
+
+           case EXIT_PROCESS_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "EXIT_PROCESS_DEBUG_EVENT"));
+             ourstatus->kind = TARGET_WAITKIND_EXITED;
+             ourstatus->value.integer = current_event.u.ExitProcess.dwExitCode;
+             CloseHandle (current_process_handle);
+             return current_event.dwProcessId;
+             break;
+
+           case LOAD_DLL_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "LOAD_DLL_DEBUG_EVENT"));
+             catch_errors (handle_load_dll, NULL, "", RETURN_MASK_ALL);
+             registers_changed ();     /* mark all regs invalid */
+             break;
+
+           case UNLOAD_DLL_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "UNLOAD_DLL_DEBUG_EVENT"));
+             break;            /* FIXME: don't know what to do here */
+
+           case EXCEPTION_DEBUG_EVENT:
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "EXCEPTION_DEBUG_EVENT"));
+             if (handle_exception (ourstatus))
+               return current_event.dwThreadId;
+             continue_status = DBG_EXCEPTION_NOT_HANDLED;
+             break;
+
+           case OUTPUT_DEBUG_STRING_EVENT:     /* message from the kernel */
+             DEBUG_EVENTS (("gdb: kernel event for pid=%d tid=%d code=%s)\n",
+                       current_event.dwProcessId, current_event.dwThreadId,
+                            "OUTPUT_DEBUG_STRING_EVENT"));
+             if (handle_output_debug_string (ourstatus))
+               return main_thread_id;
+             break;
+           default:
+             printf_unfiltered ("gdb: kernel event for pid=%d tid=%d\n",
+                                current_event.dwProcessId,
+                                current_event.dwThreadId);
+             printf_unfiltered ("                 unknown event code %d\n",
+                                current_event.dwDebugEventCode);
+             break;
+           }
+
+         CHECK (child_continue (continue_status, -1));
+       }
+      else
+       {
+         int detach = 0;
 
-      current_thread_id = event.dwThreadId;
-      current_process_id = event.dwProcessId;
+         if (ui_loop_hook != NULL)
+           detach = ui_loop_hook (0);
 
-      switch (event.dwDebugEventCode)
-       {
-       case CREATE_THREAD_DEBUG_EVENT:
-       case EXIT_THREAD_DEBUG_EVENT:
-       case CREATE_PROCESS_DEBUG_EVENT:
-         break;
-
-       case EXIT_PROCESS_DEBUG_EVENT:
-         ourstatus->kind = TARGET_WAITKIND_EXITED;
-         ourstatus->value.integer = event.u.ExitProcess.dwExitCode;
-         CloseHandle (current_process);
-         CloseHandle (current_thread);
-         return current_process_id;
-         break;
-
-       case LOAD_DLL_DEBUG_EVENT:
-         handle_load_dll (&event);
-         break;
-       case EXCEPTION_DEBUG_EVENT:
-         handle_exception (&event, ourstatus);
-         return current_process_id;
-       default:
-         printf_unfiltered ("waitfor it %d %d %d %d\n", t,
-                            event.dwDebugEventCode,
-                            event.dwProcessId,
-                            event.dwThreadId);
-         break;
+         if (detach)
+           child_kill_inferior ();
        }
-      CHECK (ContinueDebugEvent (current_process_id,
-                                current_thread_id,
-                                DBG_CONTINUE));
     }
 }
 
-
-
-
 /* Attach to process PID, then initialize for debugging it.  */
 
 static void
@@ -348,14 +695,13 @@ child_attach (args, from_tty)
   if (!args)
     error_no_arg ("process-id to attach");
 
-  current_process_id = strtoul (args, 0, 0);
+  current_event.dwProcessId = strtoul (args, 0, 0);
 
-  ok = DebugActiveProcess (current_process_id);
+  ok = DebugActiveProcess (current_event.dwProcessId);
 
   if (!ok)
     error ("Can't attach to process.");
 
-
   exception_count = 0;
   event_count = 0;
 
@@ -365,19 +711,17 @@ child_attach (args, from_tty)
 
       if (exec_file)
        printf_unfiltered ("Attaching to program `%s', %s\n", exec_file,
-                          target_pid_to_str (current_process_id));
+                          target_pid_to_str (current_event.dwProcessId));
       else
        printf_unfiltered ("Attaching to %s\n",
-                          target_pid_to_str (current_process_id));
+                          target_pid_to_str (current_event.dwProcessId));
 
       gdb_flush (gdb_stdout);
     }
 
-  inferior_pid = current_process_id;
   push_target (&child_ops);
 }
 
-
 static void
 child_detach (args, from_tty)
      char *args;
@@ -396,7 +740,6 @@ child_detach (args, from_tty)
   unpush_target (&child_ops);
 }
 
-
 /* Print status information about what we're accessing.  */
 
 static void
@@ -416,20 +759,23 @@ child_open (arg, from_tty)
   error ("Use the \"run\" command to start a Unix child process.");
 }
 
-
 /* Start an inferior win32 child process and sets inferior_pid to its pid.
    EXEC_FILE is the file to run.
    ALLARGS is a string containing the arguments to the program.
    ENV is the environment vector to pass.  Errors reported with error().  */
 
-
 static void
 child_create_inferior (exec_file, allargs, env)
      char *exec_file;
      char *allargs;
      char **env;
 {
-  char *real_path;
+  char real_path[MAXPATHLEN];
+  char *winenv;
+  char *temp;
+  int envlen;
+  int i;
+
   STARTUPINFO si;
   PROCESS_INFORMATION pi;
   struct target_waitstatus dummy;
@@ -445,11 +791,9 @@ child_create_inferior (exec_file, allargs, env)
   memset (&si, 0, sizeof (si));
   si.cb = sizeof (si);
 
-  /* A realpath is always the same size, or a bit shorter than a nice path. */
-  real_path = alloca (strlen (exec_file) + 1);
-  path_to_real_path (exec_file, real_path);
+  cygwin32_conv_to_win32_path (exec_file, real_path);
 
-  flags = DEBUG_ONLY_THIS_PROCESS | DEBUG_PROCESS;
+  flags = DEBUG_ONLY_THIS_PROCESS;
 
   if (new_group)
     flags |= CREATE_NEW_PROCESS_GROUP;
@@ -457,35 +801,108 @@ child_create_inferior (exec_file, allargs, env)
   if (new_console)
     flags |= CREATE_NEW_CONSOLE;
 
-  args = alloca (strlen (exec_file) + strlen (allargs) + 2);
+  args = alloca (strlen (real_path) + strlen (allargs) + 2);
+
+  strcpy (args, real_path);
 
-  strcpy (args, exec_file);
   strcat (args, " ");
   strcat (args, allargs);
 
-  ret = CreateProcess (real_path,
-                      args,
+  /* Prepare the environment vars for CreateProcess.  */
+  {
+    /* This code use to assume all env vars were file names and would
+       translate them all to win32 style.  That obviously doesn't work in the
+       general case.  The current rule is that we only translate PATH.
+       We need to handle PATH because we're about to call CreateProcess and
+       it uses PATH to find DLL's.  Fortunately PATH has a well-defined value
+       in both posix and win32 environments.  cygwin.dll will change it back
+       to posix style if necessary.  */
+
+    static const char *conv_path_names[] =
+    {
+      "PATH=",
+      0
+    };
+
+    /* CreateProcess takes the environment list as a null terminated set of
+       strings (i.e. two nulls terminate the list).  */
+
+    /* Get total size for env strings.  */
+    for (envlen = 0, i = 0; env[i] && *env[i]; i++)
+      {
+       int j, len;
+
+       for (j = 0; conv_path_names[j]; j++)
+         {
+           len = strlen (conv_path_names[j]);
+           if (strncmp (conv_path_names[j], env[i], len) == 0)
+             {
+               if (cygwin32_posix_path_list_p (env[i] + len))
+                 envlen += len
+                   + cygwin32_posix_to_win32_path_list_buf_size (env[i] + len);
+               else
+                 envlen += strlen (env[i]) + 1;
+               break;
+             }
+         }
+       if (conv_path_names[j] == NULL)
+         envlen += strlen (env[i]) + 1;
+      }
+
+    winenv = alloca (envlen + 1);
+
+    /* Copy env strings into new buffer.  */
+    for (temp = winenv, i = 0; env[i] && *env[i]; i++)
+      {
+       int j, len;
+
+       for (j = 0; conv_path_names[j]; j++)
+         {
+           len = strlen (conv_path_names[j]);
+           if (strncmp (conv_path_names[j], env[i], len) == 0)
+             {
+               if (cygwin32_posix_path_list_p (env[i] + len))
+                 {
+                   memcpy (temp, env[i], len);
+                   cygwin32_posix_to_win32_path_list (env[i] + len, temp + len);
+                 }
+               else
+                 strcpy (temp, env[i]);
+               break;
+             }
+         }
+       if (conv_path_names[j] == NULL)
+         strcpy (temp, env[i]);
+
+       temp += strlen (temp) + 1;
+      }
+
+    /* Final nil string to terminate new env.  */
+    *temp = 0;
+  }
+
+  ret = CreateProcess (0,
+                      args,    /* command line */
                       NULL,    /* Security */
                       NULL,    /* thread */
                       TRUE,    /* inherit handles */
                       flags,   /* start flags */
-                      env,
+                      winenv,
                       NULL,    /* current directory */
                       &si,
                       &pi);
   if (!ret)
-    error ("Error creating process %s, (error %d)\n", exec_file, GetLastError());
+    error ("Error creating process %s, (error %d)\n", exec_file, GetLastError ());
 
   exception_count = 0;
   event_count = 0;
 
-  inferior_pid = pi.dwProcessId;
-  current_process = pi.hProcess;
-  current_thread = pi.hThread;
-  current_process_id = pi.dwProcessId;
-  current_thread_id = pi.dwThreadId;
+  current_process_handle = pi.hProcess;
+  current_event.dwProcessId = pi.dwProcessId;
+  memset (&current_event, 0, sizeof (current_event));
+  inferior_pid = current_event.dwThreadId = pi.dwThreadId;
   push_target (&child_ops);
-  init_thread_list ();
+  child_init_thread_list ();
   init_wait_for_inferior ();
   clear_proceed_status ();
   target_terminal_init ();
@@ -500,18 +917,20 @@ child_create_inferior (exec_file, allargs, env)
 static void
 child_mourn_inferior ()
 {
+  (void) child_continue (DBG_CONTINUE, -1);
   unpush_target (&child_ops);
   generic_mourn_inferior ();
 }
 
-
 /* Send a SIGINT to the process group.  This acts just like the user typed a
    ^C on the controlling terminal. */
 
-void
+static void
 child_stop ()
 {
+  DEBUG_EVENTS (("gdb: GenerateConsoleCtrlEvent (CTRLC_EVENT, 0)\n"));
   CHECK (GenerateConsoleCtrlEvent (CTRL_C_EVENT, 0));
+  registers_changed ();                /* refresh register state */
 }
 
 int
@@ -521,12 +940,18 @@ child_xfer_memory (CORE_ADDR memaddr, char *our, int len,
   DWORD done;
   if (write)
     {
-      WriteProcessMemory (current_process, memaddr, our, len, &done);
-      FlushInstructionCache (current_process, memaddr, len);
+      DEBUG_MEM (("gdb: write target memory, %d bytes at 0x%08x\n",
+                 len, memaddr));
+      WriteProcessMemory (current_process_handle, (LPVOID) memaddr, our,
+                         len, &done);
+      FlushInstructionCache (current_process_handle, (LPCVOID) memaddr, len);
     }
   else
     {
-      ReadProcessMemory (current_process, memaddr, our, len, &done);
+      DEBUG_MEM (("gdb: read target memory, %d bytes at 0x%08x\n",
+                 len, memaddr));
+      ReadProcessMemory (current_process_handle, (LPCVOID) memaddr, our, len,
+                        &done);
     }
   return done;
 }
@@ -534,37 +959,60 @@ child_xfer_memory (CORE_ADDR memaddr, char *our, int len,
 void
 child_kill_inferior (void)
 {
-  CHECK (TerminateProcess (current_process, 0));
-  CHECK (CloseHandle (current_process));
-  CHECK (CloseHandle (current_thread));
+  CHECK (TerminateProcess (current_process_handle, 0));
+
+  for (;;)
+    {
+      if (!child_continue (DBG_CONTINUE, -1))
+       break;
+      if (!WaitForDebugEvent (&current_event, INFINITE))
+       break;
+      if (current_event.dwDebugEventCode == EXIT_PROCESS_DEBUG_EVENT)
+       break;
+    }
+
+  CHECK (CloseHandle (current_process_handle));
+
+  /* this may fail in an attached process so don't check. */
+  (void) CloseHandle (current_thread->h);
+  target_mourn_inferior ();    /* or just child_mourn_inferior? */
 }
 
 void
-child_resume (int pid, int step, enum target_signal signal)
+child_resume (int pid, int step, enum target_signal sig)
 {
-  DEBUG (("child_resume (%d, %d, %d);\n", pid, step, signal));
+  int i;
+  thread_info *th;
+  DWORD continue_status = last_sig > 0 && last_sig < NSIG ?
+  DBG_EXCEPTION_NOT_HANDLED : DBG_CONTINUE;
+
+  DEBUG_EXEC (("gdb: child_resume (pid=%d, step=%d, sig=%d);\n",
+              pid, step, sig));
 
+  /* Get context for currently selected thread */
+  th = thread_rec (current_event.dwThreadId, FALSE);
   if (step)
     {
+#ifdef i386
       /* Single step by setting t bit */
       child_fetch_inferior_registers (PS_REGNUM);
-      context.EFlags |= FLAG_TRACE_BIT;
+      th->context.EFlags |= FLAG_TRACE_BIT;
+#endif
     }
 
-  if (context.ContextFlags)
+  if (th->context.ContextFlags)
     {
-      CHECK (SetThreadContext (current_thread, &context));
-      context.ContextFlags = 0;
+      CHECK (SetThreadContext (th->h, &th->context));
+      th->context.ContextFlags = 0;
     }
 
-  if (signal)
-    {
-      fprintf_unfiltered (gdb_stderr, "Can't send signals to the child.\n");
-    }
+  /* Allow continuing with the same signal that interrupted us.
+     Otherwise complain. */
+  if (sig && sig != last_sig)
+    fprintf_unfiltered (gdb_stderr, "Can't send signals to the child.  signal %d\n", sig);
 
-  CHECK (ContinueDebugEvent (current_process_id,
-                            current_thread_id,
-                            DBG_CONTINUE));
+  last_sig = 0;
+  child_continue (continue_status, pid);
 }
 
 static void
@@ -582,55 +1030,62 @@ child_can_run ()
 static void
 child_close ()
 {
+  DEBUG_EVENTS (("gdb: child_close, inferior_pid=%d\n", inferior_pid));
+}
+
+struct target_ops child_ops;
 
+static void
+init_child_ops (void)
+{
+  child_ops.to_shortname = "child";
+  child_ops.to_longname = "Win32 child process";
+  child_ops.to_doc = "Win32 child process (started by the \"run\" command).";
+  child_ops.to_open = child_open;
+  child_ops.to_close = child_close;
+  child_ops.to_attach = child_attach;
+  child_ops.to_detach = child_detach;
+  child_ops.to_resume = child_resume;
+  child_ops.to_wait = child_wait;
+  child_ops.to_fetch_registers = child_fetch_inferior_registers;
+  child_ops.to_store_registers = child_store_inferior_registers;
+  child_ops.to_prepare_to_store = child_prepare_to_store;
+  child_ops.to_xfer_memory = child_xfer_memory;
+  child_ops.to_files_info = child_files_info;
+  child_ops.to_insert_breakpoint = memory_insert_breakpoint;
+  child_ops.to_remove_breakpoint = memory_remove_breakpoint;
+  child_ops.to_terminal_init = terminal_init_inferior;
+  child_ops.to_terminal_inferior = terminal_inferior;
+  child_ops.to_terminal_ours_for_output = terminal_ours_for_output;
+  child_ops.to_terminal_ours = terminal_ours;
+  child_ops.to_terminal_info = child_terminal_info;
+  child_ops.to_kill = child_kill_inferior;
+  child_ops.to_load = 0;
+  child_ops.to_lookup_symbol = 0;
+  child_ops.to_create_inferior = child_create_inferior;
+  child_ops.to_mourn_inferior = child_mourn_inferior;
+  child_ops.to_can_run = child_can_run;
+  child_ops.to_notice_signals = 0;
+  child_ops.to_thread_alive = win32_child_thread_alive;
+  child_ops.to_stop = child_stop;
+  child_ops.to_stratum = process_stratum;
+  child_ops.DONT_USE = 0;
+  child_ops.to_has_all_memory = 1;
+  child_ops.to_has_memory = 1;
+  child_ops.to_has_stack = 1;
+  child_ops.to_has_registers = 1;
+  child_ops.to_has_execution = 1;
+  child_ops.to_sections = 0;
+  child_ops.to_sections_end = 0;
+  child_ops.to_magic = OPS_MAGIC;
 }
-struct target_ops child_ops =
-{
-  "child",                     /* to_shortname */
-  "Win32 child process",       /* to_longname */
-  "Win32 child process (started by the \"run\" command).",     /* to_doc */
-  child_open,                  /* to_open */
-  child_close,                 /* to_close */
-  child_attach,                        /* to_attach */
-  child_detach,                        /* to_detach */
-  child_resume,                        /* to_resume */
-  child_wait,                  /* to_wait */
-  child_fetch_inferior_registers,/* to_fetch_registers */
-  child_store_inferior_registers,/* to_store_registers */
-  child_prepare_to_store,      /* to_child_prepare_to_store */
-  child_xfer_memory,           /* to_xfer_memory */
-  child_files_info,            /* to_files_info */
-  memory_insert_breakpoint,    /* to_insert_breakpoint */
-  memory_remove_breakpoint,    /* to_remove_breakpoint */
-  terminal_init_inferior,      /* to_terminal_init */
-  terminal_inferior,           /* to_terminal_inferior */
-  terminal_ours_for_output,    /* to_terminal_ours_for_output */
-  terminal_ours,               /* to_terminal_ours */
-  child_terminal_info,         /* to_terminal_info */
-  child_kill_inferior,         /* to_kill */
-  0,                           /* to_load */
-  0,                           /* to_lookup_symbol */
-  child_create_inferior,       /* to_create_inferior */
-  child_mourn_inferior,                /* to_mourn_inferior */
-  child_can_run,               /* to_can_run */
-  0,                           /* to_notice_signals */
-  0,                           /* to_thread_alive */
-  child_stop,                  /* to_stop */
-  process_stratum,             /* to_stratum */
-  0,                           /* to_next */
-  1,                           /* to_has_all_memory */
-  1,                           /* to_has_memory */
-  1,                           /* to_has_stack */
-  1,                           /* to_has_registers */
-  1,                           /* to_has_execution */
-  0,                           /* to_sections */
-  0,                           /* to_sections_end */
-  OPS_MAGIC                    /* to_magic */
-};
 
 void
 _initialize_inftarg ()
 {
+  struct cmd_list_element *c;
+  init_child_ops ();
+
   add_show_from_set
     (add_set_cmd ("new-console", class_support, var_boolean,
                  (char *) &new_console,
@@ -645,5 +1100,55 @@ _initialize_inftarg ()
                  &setlist),
      &showlist);
 
+  add_show_from_set
+    (add_set_cmd ("debugexec", class_support, var_boolean,
+                 (char *) &debug_exec,
+                 "Set whether to display execution in child process.",
+                 &setlist),
+     &showlist);
+
+  add_show_from_set
+    (add_set_cmd ("debugevents", class_support, var_boolean,
+                 (char *) &debug_events,
+                 "Set whether to display kernel events in child process.",
+                 &setlist),
+     &showlist);
+
+  add_show_from_set
+    (add_set_cmd ("debugmemory", class_support, var_boolean,
+                 (char *) &debug_memory,
+                 "Set whether to display memory accesses in child process.",
+                 &setlist),
+     &showlist);
+
+  add_show_from_set
+    (add_set_cmd ("debugexceptions", class_support, var_boolean,
+                 (char *) &debug_exceptions,
+              "Set whether to display kernel exceptions in child process.",
+                 &setlist),
+     &showlist);
+
   add_target (&child_ops);
 }
+
+/* Determine if the thread referenced by "pid" is alive
+   by "polling" it.  If WaitForSingleObject returns WAIT_OBJECT_0
+   it means that the pid has died.  Otherwise it is assumed to be alive. */
+static int
+win32_child_thread_alive (int pid)
+{
+  return WaitForSingleObject (thread_rec (pid, FALSE)->h, 0) == WAIT_OBJECT_0 ?
+    FALSE : TRUE;
+}
+
+/* Convert pid to printable format. */
+char *
+cygwin_pid_to_str (int pid)
+{
+  static char buf[80];
+  if (pid == current_event.dwProcessId)
+    sprintf (buf, "process %d", pid);
+  else
+    sprintf (buf, "thread %d.0x%x", current_event.dwProcessId, pid);
+  return buf;
+}