From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5889 invoked by alias); 18 Aug 2003 17:42:45 -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 5882 invoked from network); 18 Aug 2003 17:42:45 -0000 Received: from unknown (HELO localhost.redhat.com) (66.30.197.194) by sources.redhat.com with SMTP; 18 Aug 2003 17:42:45 -0000 Received: from redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id D14912B7F; Mon, 18 Aug 2003 13:42:39 -0400 (EDT) Message-ID: <3F41100F.1080706@redhat.com> Date: Mon, 18 Aug 2003 17:42:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.2) Gecko/20030223 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Daniel Jacobowitz Cc: gdb-patches@sources.redhat.com, cagney@redhat.com Subject: Re: [patch rfc, 6.0?] only allow raw raw registers; Was: [patch rfc] Add NUM_REGS pseudo regs to MIPS References: <200307271855.h6RItIho043770@oberon.asicdesigners.com> <3F2442B3.1000506@redhat.com> <3F254328.2030407@redhat.com> <20030728163857.GA28595@nevyn.them.org> <20030818170933.GA26128@nevyn.them.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-08/txt/msg00303.txt.bz2 > On Mon, Jul 28, 2003 at 12:38:57PM -0400, Daniel Jacobowitz wrote: > >> On Mon, Jul 28, 2003 at 11:37:12AM -0400, Andrew Cagney wrote: > >> > I've checked this tweak into 6.0 and mainline: >> > > >> > >2003-07-27 Andrew Cagney >> > > >> > > * mips-tdep.c (print_gp_register_row): Print the GPR's register >> > > MOD NUM_REGS. >> > > > >> > >> > and this change into just the mainline (6.0 will follow in a few days if >> > no one notices a problem): >> > > >> > >2003-07-27 Andrew Cagney >> > > >> > > * regcache.c (struct regcache_descr): Update comments on >> > > nr_raw_registers. >> > > (init_legacy_regcache_descr): Don't set nr_raw_registers or >> > > sizeof_raw_register_valid_p. >> > > (init_regcache_descr): Set nr_raw_registers and >> > > sizeof_raw_register_valid_p before calling >> > > init_legacy_regcache_descr. >> > > > >> >> On a related note, before this change (2003-07-07 actually) >> mipsel-linux was completely broken. Info registers, in addition to >> printing R90, also claimed that all registers were zero. Backtraces >> broke for the same reason. >> >> I haven't tried newer mainline yet; I will today or tomorrow. > > > Sorry, Andrew - we're almost there but MIPS register handling is still > messed up. Take a look at "info all-registers": What does "maint print [raw-]registers" look like? > (gdb) i all-registers > zero at v0 v1 a0 a1 a2 a3 > R0 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 > t0 t1 t2 t3 t4 t5 t6 t7 > R8 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 > s0 s1 s2 s3 s4 s5 s6 s7 > R16 7fff7eaf 7fff7dcc 7fff7dd4 00000003 004059cc 10012808 00000000 00000000 > t8 t9 k0 k1 gp sp s8 ra > R24 00000000 00000000 00000000 00000000 00000000 7fff7df0 00000000 00000000 > zero at v0 v1 a0 a1 > 00000000 0067194b 00000004 80279720 00000020 2aac0ac0 > a2: 0xffffffff flt: nan dbl: nan > a3: 0xffffffff flt: nan > t0: 0xffffffff flt: nan dbl: nan > t1: 0xffffffff flt: nan > t2: 0xffffffff flt: nan dbl: nan > t3: 0xffffffff flt: nan > > > Obviously the registers aren't where it expects them to be. > > -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer