From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3610 invoked by alias); 26 Apr 2002 23:49:43 -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 3569 invoked from network); 26 Apr 2002 23:49:39 -0000 Received: from unknown (HELO fred.ninemoons.com) (68.14.214.217) by sources.redhat.com with SMTP; 26 Apr 2002 23:49:39 -0000 Received: (from fnf@localhost) by fred.ninemoons.com (8.11.6/8.11.6) id g3QNo3s23978; Fri, 26 Apr 2002 16:50:03 -0700 From: Fred Fish Message-Id: <200204262350.g3QNo3s23978@fred.ninemoons.com> Subject: [RFA] Remove dependency on load address of environ To: gdb-patches@sources.redhat.com Date: Fri, 26 Apr 2002 16:49:00 -0000 Cc: fnf@redhat.com Reply-To: fnf@redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-04/txt/msg01097.txt.bz2 To implement the "maint space" command, gdb depends upon "environ" being located near where sbrk() starts allocating space. This patch removes that assumption by explicitly saving the base address at the entry to main() and also simplifies the code. -Fred 2002-04-26 Fred Fish * main.c (display_space_base): Add global variable. (main): Initialize display_space_base by calling sbrk. * main.c (captured_main): Add display_space_base decl, remove environ decl and reference, remove lim variable, use display_space_base. * top.c (command_loop): Ditto. * event-top.c (command_handler): Ditto. (command_line_handler_continuation): Ditto. Index: top.c =================================================================== RCS file: /cvs/src/src/gdb/top.c,v retrieving revision 1.62 diff -c -p -r1.62 top.c *** top.c 23 Apr 2002 03:00:57 -0000 1.62 --- top.c 26 Apr 2002 23:45:34 -0000 *************** command_loop (void) *** 759,764 **** --- 759,765 ---- #endif extern int display_time; extern int display_space; + extern char *display_space_base; while (instream && !feof (instream)) { *************** command_loop (void) *** 782,791 **** if (display_space) { #ifdef HAVE_SBRK ! extern char **environ; ! char *lim = (char *) sbrk (0); ! ! space_at_cmd_start = (long) (lim - (char *) &environ); #endif } --- 783,789 ---- if (display_space) { #ifdef HAVE_SBRK ! space_at_cmd_start = (char *) sbrk (0) - display_space_base; #endif } *************** command_loop (void) *** 805,813 **** if (display_space) { #ifdef HAVE_SBRK ! extern char **environ; ! char *lim = (char *) sbrk (0); ! long space_now = lim - (char *) &environ; long space_diff = space_now - space_at_cmd_start; printf_unfiltered ("Space used: %ld (%c%ld for this command)\n", --- 803,809 ---- if (display_space) { #ifdef HAVE_SBRK ! long space_now = (char *) sbrk (0) - display_space_base; long space_diff = space_now - space_at_cmd_start; printf_unfiltered ("Space used: %ld (%c%ld for this command)\n", Index: event-top.c =================================================================== RCS file: /cvs/src/src/gdb/event-top.c,v retrieving revision 1.20 diff -c -p -r1.20 event-top.c *** event-top.c 27 Mar 2002 21:20:15 -0000 1.20 --- event-top.c 26 Apr 2002 23:45:35 -0000 *************** command_handler (char *command) *** 475,480 **** --- 475,481 ---- #endif extern int display_time; extern int display_space; + extern char *display_space_base; quit_flag = 0; if (instream == stdin && stdin_is_tty) *************** command_handler (char *command) *** 494,503 **** if (display_space) { #ifdef HAVE_SBRK ! extern char **environ; ! char *lim = (char *) sbrk (0); ! ! space_at_cmd_start = (long) (lim - (char *) &environ); #endif } --- 495,501 ---- if (display_space) { #ifdef HAVE_SBRK ! space_at_cmd_start = (char *) sbrk (0) - display_space_base; #endif } *************** command_handler (char *command) *** 538,546 **** if (display_space) { #ifdef HAVE_SBRK ! extern char **environ; ! char *lim = (char *) sbrk (0); ! long space_now = lim - (char *) &environ; long space_diff = space_now - space_at_cmd_start; printf_unfiltered ("Space used: %ld (%c%ld for this command)\n", --- 536,542 ---- if (display_space) { #ifdef HAVE_SBRK ! long space_now = (char *) sbrk (0) - display_space_base; long space_diff = space_now - space_at_cmd_start; printf_unfiltered ("Space used: %ld (%c%ld for this command)\n", *************** command_line_handler_continuation (struc *** 560,565 **** --- 556,562 ---- { extern int display_time; extern int display_space; + extern char *display_space_base; long time_at_cmd_start = arg->data.longint; long space_at_cmd_start = arg->next->data.longint; *************** command_line_handler_continuation (struc *** 577,585 **** if (display_space) { #ifdef HAVE_SBRK ! extern char **environ; ! char *lim = (char *) sbrk (0); ! long space_now = lim - (char *) &environ; long space_diff = space_now - space_at_cmd_start; printf_unfiltered ("Space used: %ld (%c%ld for this command)\n", --- 574,580 ---- if (display_space) { #ifdef HAVE_SBRK ! long space_now = (char *) sbrk (0) - display_space_base; long space_diff = space_now - space_at_cmd_start; printf_unfiltered ("Space used: %ld (%c%ld for this command)\n", Index: main.c =================================================================== RCS file: /cvs/src/src/gdb/main.c,v retrieving revision 1.16 diff -c -p -r1.16 main.c *** main.c 17 Jan 2002 22:15:17 -0000 1.16 --- main.c 26 Apr 2002 23:45:36 -0000 *************** int display_time; *** 45,50 **** --- 45,54 ---- int display_space; + /* Remember the lowest sbrk limit when display_space is nonzero. */ + + char *display_space_base; + /* Whether this is the async version or not. The async version is invoked on the command line with the -nw --async options. In this version, the usual command_loop is substituted by and event loop which *************** extern int gdbtk_test (char *); *** 679,689 **** if (display_space) { #ifdef HAVE_SBRK ! extern char **environ; ! char *lim = (char *) sbrk (0); ! ! printf_unfiltered ("Startup size: data size %ld\n", ! (long) (lim - (char *) &environ)); #endif } --- 683,690 ---- if (display_space) { #ifdef HAVE_SBRK ! long space_now = (char *) sbrk (0) - display_space_base; ! printf_unfiltered ("Startup size: data size %ld\n", space_now); #endif } *************** main (int argc, char **argv) *** 731,736 **** --- 732,738 ---- struct captured_main_args args; args.argc = argc; args.argv = argv; + display_space_base = sbrk (0); catch_errors (captured_main, &args, "", RETURN_MASK_ALL); return 0; }