From: "Kris Warkentin" <kewarken@qnx.com>
To: <gdb-patches@sources.redhat.com>
Subject: Re: [RFC] QNX Neutrino/i386 support
Date: Sun, 09 Mar 2003 14:54:00 -0000 [thread overview]
Message-ID: <008701c2e64c$6342cb60$2a00a8c0@dash> (raw)
> Sorry but the native procfs stuff is beyond my maintainership, and I
> don't feel confident enough to approve that stuff based on my global
> maintainership.
Okay. I guess I'll have to wait for someone else to pipe up.
> The i386-nto-tdep.c (and the nto-tdep.c for that matter) should only
> contain ISA/ABI related code. You'll probably need some register
> mapping functions in there (for interpreting core files), but I fail
> to see how the code that's in your proposed i386-nto-tdep.c file is
> related to the rest of the stuff. There just seem to be too many
> interdependencies between all the different files.
Okay, I can see that. One of the things I've been working on is untying all
our register stuff (in our own tree) because it bounces back and forth
between files so I can relate to how nasty that gets. All there is in
i386-nto-tdep.c however is the link map stuff and the register fetch/store
functions which many tdep files have. I just happen to do a little extra
calculating to report back regset numbers and sizes, etc. that are handy for
nto-procfs and remote-nto to know.
My plan was to have all the generic stuff in nto-tdep.c and then have hooks
for register store/fetch in the <arch>-nto-tdep.c files. This way
nto-procfs.c and remote-nto.c get to take advantage of the same files.
Forgive my bad ascii art but we have:
nto-procfs.c__ _i386-nto-tdep.c
\_nto-tdep.[ch]__/_sh-nto-tdep.c
remote-nto.c__/ \_mips-nto-tdep.c, arm, ppc, ...
Where the nto-tdep.h is the interface to the arch specific back ends.
> That's the main reason why I
> (and Andrew before me) have asked you to seperate the target-dependent
> bits from the native and remote support and ask approval for those
> bits seperately.
Alright. I had understood that you wanted me to separate the native from
the remote but perhaps I misunderstood. If you would like to do it this
way, it's fine by me.
> We have seen in the past that such cleanups hardly ever happen. Would
> it be acceptable for you to check the new files in on a branch and
> merge it in bit by bit from there?
I hope that I will eventually earn your trust but I can see your point.
Tell you what, how about you add the file as you presented it to me and then
I will start submitting additions to you. Trust is a two way street so I
will trust you to do the right thing and help me get our support in and
working.
cheers,
Kris
next reply other threads:[~2003-03-09 14:54 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-03-09 14:54 Kris Warkentin [this message]
-- strict thread matches above, loose matches on Subject: below --
2003-03-08 13:45 Mark Kettenis
2003-03-08 15:02 ` Kris Warkentin
2003-03-08 23:49 ` Mark Kettenis
2003-03-09 0:14 ` Mark Kettenis
2003-03-09 4:34 ` Kris Warkentin
2003-03-09 17:51 ` Kris Warkentin
2003-03-19 18:19 ` Kris Warkentin
2003-03-19 18:27 ` Mark Kettenis
2003-03-08 15:19 ` Kris Warkentin
2003-03-08 15:22 ` Kris Warkentin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='008701c2e64c$6342cb60$2a00a8c0@dash' \
--to=kewarken@qnx.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox