From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11124 invoked by alias); 15 Dec 2010 12:55:38 -0000 Received: (qmail 11116 invoked by uid 22791); 15 Dec 2010 12:55:37 -0000 X-SWARE-Spam-Status: No, hits=-2.6 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from mail-wy0-f169.google.com (HELO mail-wy0-f169.google.com) (74.125.82.169) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 15 Dec 2010 12:55:32 +0000 Received: by wyj26 with SMTP id 26so1498152wyj.0 for ; Wed, 15 Dec 2010 04:55:30 -0800 (PST) Received: by 10.216.59.143 with SMTP id s15mr780965wec.49.1292417729641; Wed, 15 Dec 2010 04:55:29 -0800 (PST) MIME-Version: 1.0 Received: by 10.216.3.10 with HTTP; Wed, 15 Dec 2010 04:55:09 -0800 (PST) In-Reply-To: <4D08AC60.9060307@codesourcery.com> References: <1291886957-12003-1-git-send-email-vapier@gentoo.org> <201012141157.49676.vapier@gentoo.org> <4D07A65D.2090000@codesourcery.com> <201012141225.34826.vapier@gentoo.org> <4D08AC60.9060307@codesourcery.com> From: Mike Frysinger Date: Wed, 15 Dec 2010 12:55:00 -0000 Message-ID: Subject: Re: [toolchain-devel] [PATCH v2] gdbserver: bfin: new port To: Andrew Stubbs Cc: Daniel Jacobowitz , toolchain-devel@blackfin.uclinux.org, gdb-patches@sourceware.org, Pedro Alves Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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/msg00295.txt.bz2 On Wed, Dec 15, 2010 at 06:54, Andrew Stubbs wrote: > On 14/12/10 17:25, Mike Frysinger wrote: >>> I did start implementing something vaguely target independent (included >>> > =C2=A0in CodeSourcery SG++), but the state of the SH-2A kernel debug = support >>> > =C2=A0put that project on the back burner, so it remains the bare min= imum. >> >> i'm not asking for a finished project. =C2=A0i would like to simply revi= ew the >> code >> you said you have already. =C2=A0so where might one find your SuperH/FDP= IC >> gdb/gdbserver code ? > > The latest CodeSourcery SH-2A toolchain binaries and source can be > downloaded from here: > http://www.codesourcery.com/sgpp/lite/superh/portal/release1602 > > I've attached my original patch. I didn't do the forward port when we > updated GDB myself, so it may be out-of-date in a few places. code looks good ... but there's one slight problem. it is binding to the format gdbserver itself is being compiled as. in other words, this code requires gdbserver itself to be compiled as FDPIC in order to debug FDPIC applications. the current Blackfin code does not have that restriction ... a FLAT gdbserver can debug a FDPIC app just fine. but it seems this limitation is superfluous ... how about we change the __FDPIC__ ifdef to: #if defined(PTRACE_GETFDPIC_EXEC) && defined(PTRACE_GETFDPIC_INTERP) ... #endif do you have an idea of when you'll submit just the qXfer:fdpic support ? seems that these pieces can stand on their own just fine. -mike