This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 0/2] GDB process record and reverse debugging improvements for arm*-linux*
- From: Pedro Alves <palves at redhat dot com>
- To: Omair Javaid <omair dot javaid at linaro dot org>
- Cc: oza Pawandeep <oza dot pawandeep at gmail dot com>, Yao Qi <yao at codesourcery dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, patches <patches at linaro dot org>, Eli Zaretskii <eliz at gnu dot org>
- Date: Fri, 20 Dec 2013 12:38:47 +0000
- Subject: Re: [PATCH 0/2] GDB process record and reverse debugging improvements for arm*-linux*
- Authentication-results: sourceware.org; auth=none
- References: <CANW4E-3+-9FTB2C9tYO42CAV7mZeUP8ayPR+3sH1F5bweJM_7Q at mail dot gmail dot com> <52687B1A dot 2050003 at codesourcery dot com> <527C580B dot 5020100 at linaro dot org> <83k3gj5p9t dot fsf at gnu dot org> <CAK1A=4wzJUDvTEX6SxL_dEHaDvx7H0MaLShfEcN8e-RTSOHWmw at mail dot gmail dot com> <52809426 dot 40405 at linaro dot org> <52929058 dot 1050202 at linaro dot org>
On 11/24/2013 11:48 PM, Omair Javaid wrote:
> After incorporating all suggestions I am posting a final version of all
> three patches posted earlier. Looking for a go ahead for commit.
All three emails in the series have the same subject apart from
the numbers, which make it confusing. It'd be good if you made
sure the subjects were actually different, and more specific,
in the git commit log.
Hmm, I'm also surprised to now find that this "0/2" email
actually contains a patch. That makes it actually 3 patches
in the series. This is not the standard de facto way to
send a series. The best way is for 0 to be the series intro,
with no patch, and then have the patches be replies to the
intro mail. (I'd suggest addressing the comments I just
sent to patches #1 and #2, and resend the whole series with
git send-mail, which does that for you automatically.)
>
> == PATCH 0 ==
>
> This patch adds a news entry about improved process record-replay on
> arm*-linux* targets.
> It also adds linux-record.o to gdb_target_obs in configure.tgt and
> enables gdb.reverse testsuite for arm*-linux* targets.
These could be two separate patches, with NEWS actually being the
last patch in the whole series.
> gdb/testsuite:
>
> 2013-10-24 Omair Javaid <omair.javaid@linaro.org>
>
> * lib/gdb.exp: Enable gdb.reverse testsuite for arm*-linux* targets.
Write:
* lib/gdb.exp (supports_process_record): Return true for arm*-linux*.
Otherwise looks good.
--
Pedro Alves