From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25867 invoked by alias); 9 Apr 2010 19:42:17 -0000 Received: (qmail 25855 invoked by uid 22791); 9 Apr 2010 19:42:16 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=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; Fri, 09 Apr 2010 19:42:09 +0000 Received: (qmail 32204 invoked from network); 9 Apr 2010 19:42:07 -0000 Received: from unknown (HELO orlando.localnet) (pedro@127.0.0.2) by mail.codesourcery.com with ESMTPA; 9 Apr 2010 19:42:07 -0000 From: Pedro Alves To: gdb-patches@sourceware.org Subject: Re: [RFC] ARI fixes: Remove NAT_FILE for solaris Date: Fri, 09 Apr 2010 19:42:00 -0000 User-Agent: KMail/1.12.2 (Linux/2.6.31-20-generic; KDE/4.3.2; x86_64; ; ) Cc: "Pierre Muller" References: <001501cad7ff$3950cf30$abf26d90$@muller@ics-cnrs.unistra.fr> <201004091713.19282.pedro@codesourcery.com> <000001cad814$9ae9a770$d0bcf650$@muller@ics-cnrs.unistra.fr> In-Reply-To: <000001cad814$9ae9a770$d0bcf650$@muller@ics-cnrs.unistra.fr> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Message-Id: <201004092042.05112.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-04/txt/msg00278.txt.bz2 On Friday 09 April 2010 19:43:53, Pierre Muller wrote: > > -----Message d'origine----- > > De : gdb-patches-owner@sourceware.org [mailto:gdb-patches- > > owner@sourceware.org] De la part de Pedro Alves > > Envoy=E9 : Friday, April 09, 2010 6:13 PM > > =C0 : gdb-patches@sourceware.org > > Cc : Pierre Muller > > Objet : Re: [RFC] ARI fixes: Remove NAT_FILE for solaris > >=20 > > On Friday 09 April 2010 17:10:50, Pierre Muller wrote: > > > This patch simply removes i386/nm-i386sol2.h > > > > > > The only remaining code was > > > #ifdef NEW_PROC_API > > > #define CANNOT_STEP_HW_WATCHPOINTS > > > #endif > > > > > > I moved that into gdb/configure.ac. > > > The most difficult part was to get all the tools > > > (in the correct versions) to be able to regenerate > > > config.in and configure on a solaris machine. > >=20 > > This fixes nothing. There's a reason we want to get rid > > of macros defined in nm files and used in common code. As is, > > a cross debugger hosted on solaris behaves differently from > > a cross debugger hosted on all other hosts. >=20 > I agree, but I thought that doing this step by step would be easier... In principle that's fine. But, the only effect of your patch is pacifying the ARI; it's a step backwards. ARI is tool with a purpose --- remind us of things we need to fix, not to have clean results by hiding bugs! > My big patches take ages to get in, so I prefer to do it bitwise! Yeah, sorry, I still have one of those in my queue. >=20 > How could we change this so that this > would only trigger if we are debugging native and not some other=20 > target? First off, we get to decide if the special casing on solaris is still needed in the first place. It may not be needed anymore. What does it affect? What do the comments around it suggest? For what versions of solaris was it relevant. Etc.. Second, if we still need the workaround, can it be done all within the target side? > Could we change the macro > into a test that the current target is native solaris? If we can't do the fix in the target side, then I don't think that would be correct. What if we're cross debugging solaris from a non-solaris host (recently someone mentioned they had a solaris gdbserver port)? That would=20 suggest a gdbarch flag instead (target_gdbarch). But I'd first try to check if the workaround is still relevant at all. > I don't know enough on gdbarch and tdep structures > to have an idea how to write such a test. >=20 > Advices welcome. --=20 Pedro Alves