From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32439 invoked by alias); 13 Dec 2010 18:44:09 -0000 Received: (qmail 32413 invoked by uid 22791); 13 Dec 2010 18:44:08 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 13 Dec 2010 18:44:03 +0000 Received: (qmail 9127 invoked from network); 13 Dec 2010 18:44:01 -0000 Received: from unknown (HELO orlando.localnet) (pedro@127.0.0.2) by mail.codesourcery.com with ESMTPA; 13 Dec 2010 18:44:01 -0000 From: Pedro Alves To: gdb-patches@sourceware.org, Andrew Stubbs Subject: Re: [PATCH v2] gdbserver: bfin: new port Date: Mon, 13 Dec 2010 18:44:00 -0000 User-Agent: KMail/1.13.5 (Linux/2.6.33-29-realtime; KDE/4.4.5; x86_64; ; ) Cc: Mike Frysinger , toolchain-devel@blackfin.uclinux.org, Daniel Jacobowitz , Jie Zhang References: <1291886957-12003-1-git-send-email-vapier@gentoo.org> <1291887065-12093-1-git-send-email-vapier@gentoo.org> In-Reply-To: <1291887065-12093-1-git-send-email-vapier@gentoo.org> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Message-Id: <201012131843.59666.pedro@codesourcery.com> X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2010-12/txt/msg00193.txt.bz2 > /* Under uClinux, programs are loaded at non-zero offsets, which we need > diff --git a/gdb/regformats/reg-bfin.dat b/gdb/regformats/reg-bfin.dat > new file mode 100644 > index 0000000..38dca22 > --- /dev/null > +++ b/gdb/regformats/reg-bfin.dat > @@ -0,0 +1,63 @@ > +name:bfin > +expedite:pc,sp,fp > +32:r0 > +32:r1 > +32:r2 > +32:r3 > +32:r4 > +32:r5 > +32:r6 > +32:r7 > +32:p0 > +32:p1 > +32:p2 > +32:p3 > +32:p4 > +32:p5 > +32:sp > +32:fp > +32:i0 > +32:i1 > +32:i2 > +32:i3 > +32:m0 > +32:m1 > +32:m2 > +32:m3 > +32:b0 > +32:b1 > +32:b2 > +32:b3 > +32:l0 > +32:l1 > +32:l2 > +32:l3 > +32:a0x > +32:a0w > +32:a1x > +32:a1w > +32:astat > +32:rets > +32:lc0 > +32:lt0 > +32:lb0 > +32:lc1 > +32:lt1 > +32:lb1 > +32:cycles > +32:cycles2 > +32:usp > +32:seqstat > +32:syscfg > +32:reti > +32:retx > +32:retn > +32:rete > +32:pc > +32:cc > +32:text_addr > +32:text_end_addr > +32:data_addr > +32:fdpic_exec > +32:fdpic_interp > +32:ipend Hmmm. These last 6 look to be some kind of pseudo registers, not part of the ISA -- I'm quite surprised to see these here, as part of the bfin core register set. I understand these to be fdpic related; ISTR some discussion about making these be reported with a new qXfer object? That'd be better, IMO. I haven't looked at any bfin docs, but if any of the other registers aren't core registers (only available in some variants), you should consider implementing xml target register descriptions. -- Pedro Alves