From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16314 invoked by alias); 9 Jul 2004 12:12:29 -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 16300 invoked from network); 9 Jul 2004 12:12:27 -0000 Received: from unknown (HELO sohm.kpit.com) (203.129.230.82) by sourceware.org with SMTP; 9 Jul 2004 12:12:27 -0000 content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Subject: RE: [Patch H8/300] : Fix for crashing of gdb with" info f" command Date: Fri, 09 Jul 2004 12:12:00 -0000 Message-ID: <4A1BE23A7B777442B60F4B4916AE0F130218AAA5@sohm.kpit.com> X-MS-Has-Attach: X-MS-TNEF-Correlator: From: "Sherry Samuel" To: Cc: X-SW-Source: 2004-07/txt/msg00086.txt.bz2 Hi Reference:http://sources.redhat.com/ml/gdb-patches/2004-06/msg00428.html In the latest GDB(gdb+dejagnu-20040630) snap shot, the "info f" problem des= cribed in the above link is fixed. Referring through the links you have sug= gested, it is evident that h8300 target is not in the list of the new GDB f= rame structure. Is it possible to know what should be done from our side to= update H8300 targets to the new frame work? Can you suggest how to go abou= t it? Thanks in advance. Sherry Samuel, KPIT Cummins InfoSystems Ltd. Pune, India ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Free download of GNU based tool-chains for Renesas' SH and H8 Series. The following site also offers free technical support to its users.=20 Visit http://www.kpitgnutools.com for details.=20 Latest versions of KPIT GNU tools are released on June 1, 2004. ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~=20 -----Original Message----- From: Andrew Cagney [mailto:cagney@gnu.org] Sent: Wednesday, June 23, 2004 1:39 AM To: Sherry Samuel Cc: gdb-patches@sources.redhat.com Subject: Re: [Patch H8/300] : Fix for crashing of gdb with" info f" command > ******************CHANGELOG TEXT*****************************************= ************************** > 2004-06-18 Sherry Samuel >=20 > * h8300-tdep.c (h8300_frame_init_saved_regs) : Check whether frame base= =20 > is null.=09 Are you sure this patch is against 6.1 or a current GDB? The function=20 h8300_frame_init_saved_regs has a void return type. Also, you might want to read over: Post GDB 6.2, require new frame code http://sources.redhat.com/ml/gdb/2004-06/msg00199.html As this will affect the h8/300. Andrew >=20 > --- gdb-6.1/gdb/h8300-tdep.c.old 2004-06-17 15:57:18.000000000 +0530 > +++ gdb-6.1/gdb/h8300-tdep.c 2004-06-17 17:08:54.000000000 +0530 > @@ -500,8 +500,8 @@ h8300_frame_init_saved_regs (struct fram > CORE_ADDR limit =3D (sal.end && sal.end < get_frame_pc (fi)) > ? sal.end : get_frame_pc (fi); > /* This will fill in fields in fi. */ > - > - > + if(!get_frame_base (fi))//check whether frame is null. > + return null_frame_id;//if null return >=20=20 > h8300_examine_prologue (func_addr, limit, get_frame_base (fi), > deprecated_get_frame_saved_regs (fi), fi);