From: "Maciej W. Rozycki" <macro@codesourcery.com>
To: Matthew Fortune <Matthew.Fortune@imgtec.com>
Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>,
Joel Brobecker <brobecker@adacore.com>,
Richard Sandiford <rdsandiford@googlemail.com>
Subject: RE: [committed] MIPS: Don't infer IRIX OS ABI from generic section names
Date: Wed, 10 Sep 2014 11:51:00 -0000 [thread overview]
Message-ID: <alpine.DEB.1.10.1409101231350.27075@tp.orcam.me.uk> (raw)
In-Reply-To: <6D39441BF12EF246A7ABCE6654B0235320F07EC6@LEMAIL01.le.imgtec.org>
On Wed, 10 Sep 2014, Matthew Fortune wrote:
> > The problem here is we've got assembly source that does not have the
> > usual ABI tag GDB normally infers the OS ABI from.
>
> Could you explain what 'usual ABI tag' refers to in this context?
> I have a feeling there's another layer of ABI annotation that I
> haven't seen yet.
The thing that you get from glibc's csu/abi-note.S.
> > I have verified this change to work correctly in mips-linux-gnu
> > regression testing and committed now.
>
> Thanks for tracking this down it must have been quite time consuming.
Not quite so, the hard part was realising that `vCont;s...' is not
supposed to happen for a software-stepping target, the rest was easy.
Actually it shows a deficiency in the way GDB works, that I believe has
already been discussed, in that the switch between software and hardware
stepping shouldn't really be arbitrary as it is now. Instead it should be
the stub or the native backend that should report to the core of GDB
whether hardware stepping is supported.
Also right now the MIPS backend of `gdbserver' seems to know that
hardware stepping is not supported, but it only uses that knowledge in
some contexts and issues this `ptrace(PTRACE_SINGLESTEP, ...)' request
anyway.
> I'll try to sort out running some GDB tests for any further ABI
> related work. I certainly wouldn't have guessed that .MIPS.abiflags
> would break GDB.
Well, the thing is in a sense it didn't break GDB, just revealed existing
breakage, a place to be careful about and the lack of coverage at least in
my testing. So I think it was good after all.
Maciej
prev parent reply other threads:[~2014-09-10 11:51 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-09 23:04 Maciej W. Rozycki
2014-09-10 7:19 ` Matthew Fortune
2014-09-10 11:51 ` Maciej W. Rozycki [this message]
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=alpine.DEB.1.10.1409101231350.27075@tp.orcam.me.uk \
--to=macro@codesourcery.com \
--cc=Matthew.Fortune@imgtec.com \
--cc=brobecker@adacore.com \
--cc=gdb-patches@sourceware.org \
--cc=rdsandiford@googlemail.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