From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 22393 invoked by alias); 18 Jun 2003 23:16:18 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 22355 invoked from network); 18 Jun 2003 23:16:17 -0000 Received: from unknown (HELO crack.them.org) (146.82.138.56) by sources.redhat.com with SMTP; 18 Jun 2003 23:16:17 -0000 Received: from dsl093-172-017.pit1.dsl.speakeasy.net ([66.93.172.17] helo=nevyn.them.org ident=mail) by crack.them.org with asmtp (Exim 3.12 #1 (Debian)) id 19SmAt-0000AD-00; Wed, 18 Jun 2003 18:17:07 -0500 Received: from drow by nevyn.them.org with local (Exim 3.36 #1 (Debian)) id 19Sm9x-0000Ai-00; Wed, 18 Jun 2003 19:16:09 -0400 Date: Wed, 18 Jun 2003 23:16:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sources.redhat.com Cc: msnyder@redhat.com Subject: RFA: Collect unexplained stopped threads in lin-lwp Message-ID: <20030618231609.GA394@nevyn.them.org> Mail-Followup-To: gdb-patches@sources.redhat.com, msnyder@redhat.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.1i X-SW-Source: 2003-06/txt/msg00606.txt.bz2 [Michael, you more or less approved this patch in December, but it's seen a few changes - linux_record_stopped_pid isn't a dummy any more.] This patch just accepts processes we aren't currently debugging which report a SIGSTOP, and throws them onto a list. Not very useful by itself, but my next patch will both cause this to happen (by enabling fork events) and empty the list when it receives fork events. I'm only submitting it separately, because it was the last meaningful piece I could break out. Is this OK? -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer 2003-06-18 Daniel Jacobowitz * config/nm-linux.h (linux_record_stopped_pid): New prototype. * lin-lwp.c (child_wait): Call linux_record_stopped_pid. (lin_lwp_wait): Likewise. Update comments. * linux-nat.c (struct simple_pid_list, add_to_pid_list) (pull_pid_from_list, linux_record_stopped_pid): New. diff -x '*.gmo' -Nurp src/gdb/config/nm-linux.h src-one/gdb/config/nm-linux.h --- src/gdb/config/nm-linux.h 2003-06-15 16:56:47.000000000 -0400 +++ src-one/gdb/config/nm-linux.h 2003-06-18 15:19:11.000000000 -0400 @@ -78,6 +78,8 @@ extern int linux_proc_xfer_memory (CORE_ int write, struct mem_attrib *attrib, struct target_ops *target); +extern void linux_record_stopped_pid (int pid); + #define CHILD_INSERT_FORK_CATCHPOINT #define CHILD_INSERT_VFORK_CATCHPOINT #define CHILD_INSERT_EXEC_CATCHPOINT diff -x '*.gmo' -Nurp src/gdb/lin-lwp.c src-one/gdb/lin-lwp.c --- src/gdb/lin-lwp.c 2003-06-04 16:51:28.000000000 -0400 +++ src-one/gdb/lin-lwp.c 2003-06-18 15:11:32.000000000 -0400 @@ -1109,6 +1109,23 @@ child_wait (ptid_t ptid, struct target_w save_errno = EINTR; } + /* Check for stop events reported by a process we didn't already + know about - in this case, anything other than inferior_ptid. + + If we're expecting to receive stopped processes after fork, + vfork, and clone events, then we'll just add the new one to + our list and go back to waiting for the event to be reported + - the stopped process might be returned from waitpid before + or after the event is. If we want to handle debugging of + CLONE_PTRACE processes we need to do more here, i.e. switch + to multi-threaded mode. */ + if (pid != -1 && WIFSTOPPED (status) && WSTOPSIG (status) == SIGSTOP + && pid != GET_PID (inferior_ptid)) + { + pid = -1; + save_errno = EINTR; + } + clear_sigio_trap (); clear_sigint_trap (); } @@ -1272,6 +1289,22 @@ retry: lp = find_lwp_pid (pid_to_ptid (lwpid)); + /* Check for stop events reported by a process we didn't + already know about - anything not already in our LWP + list. + + If we're expecting to receive stopped processes after + fork, vfork, and clone events, then we'll just add the + new one to our list and go back to waiting for the event + to be reported - the stopped process might be returned + from waitpid before or after the event is. */ + if (WIFSTOPPED (status) && !lp) + { + linux_record_stopped_pid (lwpid); + status = 0; + continue; + } + /* Make sure we don't report an event for the exit of an LWP not in our list, i.e. not part of the current process. This can happen if we detach from a program we original forked and then it @@ -1282,6 +1315,13 @@ retry: continue; } + /* NOTE drow/2003-06-17: This code seems to be meant for debugging + CLONE_PTRACE processes which do not use the thread library - + otherwise we wouldn't find the new LWP this way. That doesn't + currently work, and the following code is currently unreachable + due to the two blocks above. If it's fixed some day, this code + should be broken out into a function so that we can also pick up + LWPs from the new interface. */ if (!lp) { lp = add_lwp (BUILD_LWP (lwpid, GET_PID (inferior_ptid))); diff -x '*.gmo' -Nurp src/gdb/linux-nat.c src-one/gdb/linux-nat.c --- src/gdb/linux-nat.c 2003-06-15 16:56:47.000000000 -0400 +++ src-one/gdb/linux-nat.c 2003-06-18 16:47:55.000000000 -0400 @@ -54,12 +54,53 @@ #define __WALL 0x40000000 /* Wait for any child. */ #endif +struct simple_pid_list +{ + int pid; + struct simple_pid_list *next; +}; +struct simple_pid_list *stopped_pids; + /* This variable is a tri-state flag: -1 for unknown, 0 if PTRACE_O_TRACEFORK can not be used, 1 if it can. */ static int linux_supports_tracefork_flag = -1; +/* Trivial list manipulation functions to keep track of a list of + new stopped processes. */ +static void +add_to_pid_list (struct simple_pid_list **listp, int pid) +{ + struct simple_pid_list *new_pid = xmalloc (sizeof (struct simple_pid_list)); + new_pid->pid = pid; + new_pid->next = *listp; + *listp = new_pid; +} + +static int +pull_pid_from_list (struct simple_pid_list **listp, int pid) +{ + struct simple_pid_list **p; + + for (p = listp; *p != NULL; p = &(*p)->next) + if ((*p)->pid == pid) + { + struct simple_pid_list *next = (*p)->next; + xfree (*p); + *p = next; + return 1; + } + return 0; +} + +void +linux_record_stopped_pid (int pid) +{ + add_to_pid_list (&stopped_pids, pid); +} + + /* A helper function for linux_test_for_tracefork, called after fork (). */ static void