From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17221 invoked by alias); 13 Jan 2004 21:32:12 -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 17172 invoked from network); 13 Jan 2004 21:32:09 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sources.redhat.com with SMTP; 13 Jan 2004 21:32:09 -0000 Received: from drow by nevyn.them.org with local (Exim 4.30 #1 (Debian)) id 1AgW8v-0004pT-43 for ; Tue, 13 Jan 2004 16:32:09 -0500 Date: Tue, 13 Jan 2004 21:32:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sources.redhat.com Subject: Re: RFA/doc: Remove set follow-fork-mode ask, update docs Message-ID: <20040113213209.GA18542@nevyn.them.org> Mail-Followup-To: gdb-patches@sources.redhat.com References: <20040112154331.GA25559@nevyn.them.org> <2719-Mon12Jan2004210011+0200-eliz@elta.co.il> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2719-Mon12Jan2004210011+0200-eliz@elta.co.il> User-Agent: Mutt/1.5.1i X-SW-Source: 2004-01/txt/msg00352.txt.bz2 On Mon, Jan 12, 2004 at 09:00:11PM +0200, Eli Zaretskii wrote: > > Date: Mon, 12 Jan 2004 10:43:31 -0500 > > From: Daniel Jacobowitz > > > > Eli, texinfo look OK? > > Yes, thanks. Thanks, I checked this in. There was one missed copyright update. -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer 2004-01-13 Daniel Jacobowitz * infrun.c (follow_fork_mode_ask): Remove. (follow_fork_mode_kind_names): Remove follow_fork_mode_ask. (follow_fork): Simplify and remove internal error for follow_fork_mode_ask. (_initialize_infrun): Update "set follow-fork-mode" help text. 2004-01-13 Daniel Jacobowitz * gdb.texinfo: Update copyright year. Mention that set follow-fork-mode is supported on GNU/Linux. Remove documentation of "set follow-fork-mode ask". Index: infrun.c =================================================================== RCS file: /cvs/src/src/gdb/infrun.c,v retrieving revision 1.126 diff -u -p -r1.126 infrun.c --- infrun.c 3 Jan 2004 15:22:21 -0000 1.126 +++ infrun.c 13 Jan 2004 21:29:26 -0000 @@ -2,7 +2,7 @@ process. Copyright 1986, 1987, 1988, 1989, 1990, 1991, 1992, 1993, 1994, - 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003 Free Software + 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation, Inc. This file is part of GDB. @@ -340,12 +340,10 @@ static struct } pending_follow; -static const char follow_fork_mode_ask[] = "ask"; static const char follow_fork_mode_child[] = "child"; static const char follow_fork_mode_parent[] = "parent"; static const char *follow_fork_mode_kind_names[] = { - follow_fork_mode_ask, follow_fork_mode_child, follow_fork_mode_parent, NULL @@ -357,16 +355,7 @@ static const char *follow_fork_mode_stri static int follow_fork (void) { - const char *follow_mode = follow_fork_mode_string; - int follow_child = (follow_mode == follow_fork_mode_child); - - /* Or, did the user not know, and want us to ask? */ - if (follow_fork_mode_string == follow_fork_mode_ask) - { - internal_error (__FILE__, __LINE__, - "follow_inferior_fork: \"ask\" mode not implemented"); - /* follow_mode = follow_fork_mode_...; */ - } + int follow_child = (follow_fork_mode_string == follow_fork_mode_child); return target_follow_fork (follow_child); } @@ -4075,31 +4064,12 @@ to the user would be loading/unloading o c = add_set_enum_cmd ("follow-fork-mode", class_run, follow_fork_mode_kind_names, &follow_fork_mode_string, -/* ??rehrauer: The "both" option is broken, by what may be a 10.20 - kernel problem. It's also not terribly useful without a GUI to - help the user drive two debuggers. So for now, I'm disabling - the "both" option. */ -/* "Set debugger response to a program call of fork \ - or vfork.\n\ - A fork or vfork creates a new process. follow-fork-mode can be:\n\ - parent - the original process is debugged after a fork\n\ - child - the new process is debugged after a fork\n\ - both - both the parent and child are debugged after a fork\n\ - ask - the debugger will ask for one of the above choices\n\ - For \"both\", another copy of the debugger will be started to follow\n\ - the new child process. The original debugger will continue to follow\n\ - the original parent process. To distinguish their prompts, the\n\ - debugger copy's prompt will be changed.\n\ - For \"parent\" or \"child\", the unfollowed process will run free.\n\ - By default, the debugger will follow the parent process.", - */ "Set debugger response to a program call of fork \ or vfork.\n\ A fork or vfork creates a new process. follow-fork-mode can be:\n\ parent - the original process is debugged after a fork\n\ child - the new process is debugged after a fork\n\ - ask - the debugger will ask for one of the above choices\n\ -For \"parent\" or \"child\", the unfollowed process will run free.\n\ +The unfollowed process will continue to run.\n\ By default, the debugger will follow the parent process.", &setlist); add_show_from_set (c, &showlist); Index: doc/gdb.texinfo =================================================================== RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v retrieving revision 1.186 diff -u -p -r1.186 gdb.texinfo --- doc/gdb.texinfo 8 Jan 2004 22:47:00 -0000 1.186 +++ doc/gdb.texinfo 13 Jan 2004 21:29:30 -0000 @@ -84,7 +84,7 @@ development.'' @vskip 0pt plus 1filll Copyright @copyright{} 1988, 1989, 1990, 1991, 1992, 1993, 1994, 1995, -1996, 1998, 1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc. +1996, 1998, 1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation, Inc. @sp 2 Published by the Free Software Foundation @* 59 Temple Place - Suite 330, @* @@ -2289,9 +2289,10 @@ get its process ID. Then tell @value{GD the child process (@pxref{Attach}). From that point on you can debug the child process just like any other process which you attached to. -On HP-UX (11.x and later only?), @value{GDBN} provides support for -debugging programs that create additional processes using the -@code{fork} or @code{vfork} function. +On some systems, @value{GDBN} provides support for debugging programs that +create additional processes using the @code{fork} or @code{vfork} functions. +Currently, the only platforms with this feature are HP-UX (11.x and later +only?) and GNU/Linux (kernel version 2.5.60 and later). By default, when a program forks, @value{GDBN} will continue to debug the parent process and the child process will run unimpeded. @@ -2315,8 +2316,6 @@ unimpeded. This is the default. The new process is debugged after a fork. The parent process runs unimpeded. -@item ask -The debugger will ask for one of the above choices. @end table @item show follow-fork-mode