From: Yao Qi <qiyaoltc@gmail.com>
To: Antoine Tremblay <antoine.tremblay@ericsson.com>
Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [PATCH V2 0/5] Support tracepoints for ARM linux in GDBServer
Date: Mon, 07 Nov 2016 09:25:00 -0000 [thread overview]
Message-ID: <CAH=s-PMocQH1gJQZoGoX3PCguUm8JETVH9+8Sh8Sc-GnawqJaA@mail.gmail.com> (raw)
In-Reply-To: <20161103143300.24934-1-antoine.tremblay@ericsson.com>
On Thu, Nov 3, 2016 at 2:32 PM, Antoine Tremblay
<antoine.tremblay@ericsson.com> wrote:
>
> Since all the prerequisites for this series have been addressed,
> this is a V2 of https://sourceware.org/ml/gdb-patches/2016-01/msg00111.html
All "hard" prerequisites are addressed, but we still want to "teach
unwinders to terminate gracefully in an arch-independent way".
https://sourceware.org/ml/gdb-patches/2016-05/msg00060.html
I didn't follow it up closely. I hope we can make progress on this...
"Progress" here means either "it is completely wrong, let us handle
unavailable data in each arch unwinder one by one" or "it is
correct, let us remove these redundant code in each arch".
I am still testing arm-linux gdbserver without and with software
single step. I still see some intermittent regressions _with_
software single step,
+FAIL: gdb.threads/non-stop-fair-events.exp: signal_thread=8: thread 1
broke out of loop (timeout)
+FAIL: gdb.threads/schedlock.exp: schedlock=off: cmd=step: step to
increment (1) (timeout)
This reveals something wrong in software single step in GDBserver.
I don't think we should bring tracepoint in until these regressions are
fixed. I won't work on these regressions until next pre-release. If
you can reproduce them and help to fix them, that will be great.
--
Yao (齐尧)
next prev parent reply other threads:[~2016-11-07 9:25 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-03 14:33 Antoine Tremblay
2016-11-03 14:33 ` [PATCH V2 4/5] Use FAST_TRACEPOINT_LABEL in range-stepping.exp Antoine Tremblay
2016-11-03 14:33 ` [PATCH V2 2/5] Enable tracing of pseudo-registers on ARM Antoine Tremblay
2016-11-03 14:33 ` [PATCH V2 5/5] Support tracepoints for ARM linux in GDBServer Antoine Tremblay
2016-11-03 17:51 ` Eli Zaretskii
2016-11-03 18:12 ` Antoine Tremblay
2016-11-10 14:01 ` Yao Qi
2016-11-15 14:42 ` Antoine Tremblay
2016-11-16 20:49 ` Yao Qi
2016-11-03 14:33 ` [PATCH V2 1/5] Teach arm unwinders to terminate gracefully Antoine Tremblay
2016-11-03 14:33 ` [PATCH V2 3/5] Improve tests to allow for targets that support trace but not ftrace Antoine Tremblay
2016-11-07 9:25 ` Yao Qi [this message]
[not found] ` <wwoka8dasqta.fsf@ericsson.com>
2016-11-08 15:34 ` [PATCH V2 0/5] Support tracepoints for ARM linux in GDBServer Antoine Tremblay
2016-11-09 16:39 ` Yao Qi
2016-11-09 17:58 ` Antoine Tremblay
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='CAH=s-PMocQH1gJQZoGoX3PCguUm8JETVH9+8Sh8Sc-GnawqJaA@mail.gmail.com' \
--to=qiyaoltc@gmail.com \
--cc=antoine.tremblay@ericsson.com \
--cc=gdb-patches@sourceware.org \
/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