This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC/RFA] Target vectors for native Linux targets
- From: Ulrich Weigand <uweigand at de dot ibm dot com>
- To: drow at false dot org (Daniel Jacobowitz)
- Cc: uweigand at de dot ibm dot com (Ulrich Weigand), mark dot kettenis at xs4all dot nl (Mark Kettenis), manjo at austin dot ibm dot com, gdb-patches at sources dot redhat dot com
- Date: Mon, 5 Sep 2005 19:31:59 +0200 (CEST)
- Subject: Re: [RFC/RFA] Target vectors for native Linux targets
Daniel Jacobowitz wrote:
>I'm presenting an alternative patch, stealing bits from mine last
>December and other clever bits from Ulrich's. His patch was much
>prettier than mine - but missed some important things that generated
>ugliness in my first patch (for instance non-FETCH_INFERIOR_REGISTERS
>targets). I've come up with cleaner solutions than I did the first
>time round.
Thanks for looking into this again!
> On Fri, Aug 19, 2005 at 02:51:24AM +0200, Ulrich Weigand wrote:
> > One problem with the conversion is that I wouldn't want to have to
> > convert all the various Linux subtargets at the same time. It's a
> > lot of work, and I'm unable to test most of those platforms. Thus
> > I've thought of a way to stage the conversion:
>
> I bit the bullet and did them all, since I had most of it lying around
> anyway.
Excellent ;-)
> This patch has only been tested on i686-pc-linux-gnu at the moment, but
> it's been mightily proofread, and I have high confidence in it. It
> obsoletes about half your posted patch for S/390 (the watchpoint bits
> are an obvious follow-up). Could you give this a spin on S/390 for me?
Your patch works fine with no regressions on s390-ibm-linux and
s390x-ibm-linux. I'll be happy to do the watchpoint bits once
your patch is committed ...
> + /* FIXME drow/2005-09-04: The hardcoded use of register_addr should go
> + away. This requires disentangling the various definitions of it
> + (particularly alpha-nat.c's). */
> +#ifdef FETCH_INFERIOR_REGISTERS
> + t = inf_ptrace_target ();
> +#else
> + t = inf_ptrace_trad_target (register_addr);
> +#endif
This means FETCH_INFERIOR_REGISTERS needs to be kept around in various
nm-linux.h files for now? I'd have hoped to get rid of at least the
s390 variant ...
Bye,
Ulrich
--
Dr. Ulrich Weigand
Linux on zSeries Development
Ulrich.Weigand@de.ibm.com