From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13745 invoked by alias); 22 Mar 2004 19:37:46 -0000 Mailing-List: contact sid-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: sid-owner@sources.redhat.com Received: (qmail 13737 invoked from network); 22 Mar 2004 19:37:45 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 22 Mar 2004 19:37:45 -0000 Received: from int-mx2.corp.redhat.com (nat-pool-rdu-dmz.redhat.com [172.16.52.200] (may be forged)) by mx1.redhat.com (8.12.10/8.12.10) with ESMTP id i2MJbiWA011114 for ; Mon, 22 Mar 2004 14:37:44 -0500 Received: from potter.sfbay.redhat.com (potter.sfbay.redhat.com [172.16.27.15]) by int-mx2.corp.redhat.com (8.11.6/8.11.6) with ESMTP id i2MJbeM29820 for ; Mon, 22 Mar 2004 14:37:40 -0500 Received: from redhat.com (IDENT:YRAIdeW4sOeZgu7X+NaBfVS4aKVKAW6f@vpn50-12.rdu.redhat.com [172.16.50.12]) by potter.sfbay.redhat.com (8.11.6/8.11.6) with ESMTP id i2MJbbR07013 for ; Mon, 22 Mar 2004 11:37:37 -0800 Message-ID: <405F4067.5020302@redhat.com> Date: Mon, 22 Mar 2004 19:37:00 -0000 From: Dave Brolley User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.2) Gecko/20021216 MIME-Version: 1.0 To: sid@sources.redhat.com Subject: [patch][rfa] Multi core debugging with GDB Content-Type: multipart/mixed; boundary="------------000704020302020600030207" X-RedHat-Spam-Score: -4.9 X-SW-Source: 2004-q1/txt/msg00045.txt.bz2 This is a multi-part message in MIME format. --------------000704020302020600030207 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Content-length: 1289 Hi, Debugging multiple cores with multiple instances of GDB within a simulation is currently possible by specifying a separate --gdb option after each --board and specifying a different port for each GDB instance. The simulation ends (as it does without GDB) when the first core exits. However, the other GDB instances all report "Watchdog expired. Target disconnected". This was because of three problems: 1) The "enabled?", "enable-threshold", "yield-host-time?" and "yield-host-time-threshold" of the host scheduler were not being set. This caused the host scheduler to yield host time after the first GDB stub exited. 2) The other GDB stubs were not properly notifying their GDB clients that they were exiting 3) Even after proper notfication and yielding were implemented, the packets never reached their GDB clients because their socket components were never polled again which was because the host scheduler was never advanced again which was because the whole shut down process was initiated when the first GDB stub drove main's "stop!" pin. This problem is corrected by having the sockets do one final transmit when their "fini" pins are driven by shutdown-sequence. The result is an orderly shutdown of the GDB stubs and their clients. OK to commit? Dave --------------000704020302020600030207 Content-Type: text/plain; name="sid-gdb.patch.txt" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="sid-gdb.patch.txt" Content-length: 6825 Index: sid/component/consoles/socketio.cxx =================================================================== RCS file: /cvs/src/src/sid/component/consoles/socketio.cxx,v retrieving revision 1.5 diff -c -p -r1.5 socketio.cxx *** sid/component/consoles/socketio.cxx 4 Mar 2002 20:38:56 -0000 1.5 --- sid/component/consoles/socketio.cxx 22 Mar 2004 18:48:32 -0000 *************** *** 1,7 **** // socketio.cxx - A console that uses a socket to do its I/O. // -*- C++ -*- ! // Copyright (C) 1999, 2000, 2002 Red Hat. // This file is part of SID and is licensed under the GPL. // See the file COPYING.SID for conditions for redistribution. --- 1,7 ---- // socketio.cxx - A console that uses a socket to do its I/O. // -*- C++ -*- ! // Copyright (C) 1999-2002, 2004 Red Hat. // This file is part of SID and is licensed under the GPL. // See the file COPYING.SID for conditions for redistribution. *************** void *** 297,302 **** --- 297,306 ---- socketio::fini_handler (host_int_4) { this->poll_control.cancel (); + + // Flush out any remaining data + if (this->connected_p) + this->poll_transmit (); if (this->connected_p) { Index: sid/component/gdb/gdb.cxx =================================================================== RCS file: /cvs/src/src/sid/component/gdb/gdb.cxx,v retrieving revision 1.10 diff -c -p -r1.10 gdb.cxx *** sid/component/gdb/gdb.cxx 8 May 2002 19:15:30 -0000 1.10 --- sid/component/gdb/gdb.cxx 22 Mar 2004 18:48:32 -0000 *************** *** 1,6 **** // gdb.cxx - GDB stub implementation. -*- C++ -*- ! // Copyright (C) 1999-2002 Red Hat. // This file is part of SID and is licensed under the GPL. // See the file COPYING.SID for conditions for redistribution. --- 1,6 ---- // gdb.cxx - GDB stub implementation. -*- C++ -*- ! // Copyright (C) 1999-2002, 2004 Red Hat. // This file is part of SID and is licensed under the GPL. // See the file COPYING.SID for conditions for redistribution. *************** gdb::deinit_handler (host_int_4) *** 1550,1556 **** { // disconnect if needed if (this->connected_p) ! this->remote_rx_eof_handler (); } --- 1550,1562 ---- { // disconnect if needed if (this->connected_p) ! { ! // shut down target ! target_power (false); ! // signal gdb ! gdbserv_fromtarget_exit (gdbserv, 0); ! this->remote_rx_eof_handler (); ! } } Index: sid/main/dynamic/commonCfg.cxx =================================================================== RCS file: /cvs/src/src/sid/main/dynamic/commonCfg.cxx,v retrieving revision 1.6 diff -c -p -r1.6 commonCfg.cxx *** sid/main/dynamic/commonCfg.cxx 27 Oct 2003 18:55:34 -0000 1.6 --- sid/main/dynamic/commonCfg.cxx 22 Mar 2004 18:48:34 -0000 *************** SessionCfg::SessionCfg (const string nam *** 530,536 **** tcl_bridge (NULL), loader (NULL), verbose (false), ! use_stdio (true) { add_child (host_sched); add_child (sim_sched); --- 530,538 ---- tcl_bridge (NULL), loader (NULL), verbose (false), ! use_stdio (true), ! board_count (0), ! gdb_count (0) { add_child (host_sched); add_child (sim_sched); *************** void SessionCfg::use_tcl_bridge () *** 657,662 **** --- 659,684 ---- init_seq->add_output (7, tcl_bridge, "!event"); } + void SessionCfg::write_config (Writer &w) + { + AggregateCfg::write_config (w); + + // Stop the host scheduler if all of the GDB stubs are stopped + if (gdb_count) + { + assert (host_sched); + Setting (host_sched, "yield-host-time-threshold", + sidutil::make_attribute (gdb_count)).write_to (w); + Setting (host_sched, "yield-host-time?", "0").write_to (w); + } + + // Stop the sim scheduler if any of the GDB stubs are stopped + assert (sim_sched); + Setting (sim_sched, "enable-threshold", + sidutil::make_attribute (board_count)).write_to (w); + Setting (sim_sched, "enabled?", + sidutil::make_attribute (board_count)).write_to (w); + } // LoaderCfg LoaderCfg::~LoaderCfg () {} *************** void BoardCfg::set_gdb (const string por *** 920,925 **** --- 942,948 ---- gdb = new GdbCfg ("gdb", port, cpu, this, sess); add_child (gdb); sess->use_no_stdio (); + sess->add_gdb (); } Index: sid/main/dynamic/commonCfg.h =================================================================== RCS file: /cvs/src/src/sid/main/dynamic/commonCfg.h,v retrieving revision 1.3 diff -c -p -r1.3 commonCfg.h *** sid/main/dynamic/commonCfg.h 21 Oct 2003 21:39:04 -0000 1.3 --- sid/main/dynamic/commonCfg.h 22 Mar 2004 18:48:34 -0000 *************** struct UlogCfg *** 182,187 **** --- 182,188 ---- // you should really only make one of these, with an empty name, // unless you want some crazy multi-session support. + class BoardCfg; class LoaderCfg; struct SessionCfg : *************** struct SessionCfg : *** 214,219 **** --- 215,226 ---- bool use_stdio; void add_ulog_file (const string filename); map ulog_map; + void add_gdb () { ++gdb_count; } + void add_board (ComponentCfg *b) { ++board_count; add_child (b); } + virtual void write_config (Writer &w); + private: + sid::host_int_4 board_count; + sid::host_int_4 gdb_count; }; class CpuCfg : *************** public: *** 277,283 **** virtual ~GprofCfg (); }; - class BoardCfg; class GdbCfg : virtual public AggregateCfg { --- 284,289 ---- Index: sid/main/dynamic/mainDynamic.cxx =================================================================== RCS file: /cvs/src/src/sid/main/dynamic/mainDynamic.cxx,v retrieving revision 1.5 diff -c -p -r1.5 mainDynamic.cxx *** sid/main/dynamic/mainDynamic.cxx 21 Oct 2003 21:39:04 -0000 1.5 --- sid/main/dynamic/mainDynamic.cxx 22 Mar 2004 18:48:34 -0000 *************** main(int argc, char* argv[]) *** 617,623 **** { need_sess (sess); if (curr_board) ! sess->add_child (curr_board); curr_board = NULL; string new_board_type = optstring(); string new_board_name (new_board_type + "-" + --- 617,623 ---- { need_sess (sess); if (curr_board) ! sess->add_board (curr_board); curr_board = NULL; string new_board_type = optstring(); string new_board_name (new_board_type + "-" + *************** main(int argc, char* argv[]) *** 878,884 **** } if (sess && curr_board) ! sess->add_child (curr_board); if (persistent_p) config_items.push_back (make_pair (false, string("set main persistent? true"))); --- 878,884 ---- } if (sess && curr_board) ! sess->add_board (curr_board); if (persistent_p) config_items.push_back (make_pair (false, string("set main persistent? true"))); --------------000704020302020600030207--