This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v3 01/17] Fix and test "checkpoint" in non-stop mode
- From: Pedro Alves <palves at redhat dot com>
- To: Doug Evans <dje at google dot com>
- Cc: gdb-patches <gdb-patches at sourceware dot org>
- Date: Tue, 19 May 2015 19:07:56 +0100
- Subject: Re: [PATCH v3 01/17] Fix and test "checkpoint" in non-stop mode
- Authentication-results: sourceware.org; auth=none
- References: <1429267521-21047-1-git-send-email-palves at redhat dot com> <1429267521-21047-2-git-send-email-palves at redhat dot com> <yjt2bnii6wdu dot fsf at ruffy dot mtv dot corp dot google dot com> <5537DEDD dot 5000103 at redhat dot com> <21823 dot 45051 dot 20019 dot 326313 at ruffy2 dot mtv dot corp dot google dot com> <CADPb22QhTB6-KaaXSJ4ABp5Rq1SbG=VVvUyJcBJ3uvyYEZyJng at mail dot gmail dot com>
On 04/28/2015 10:50 PM, Doug Evans wrote:
> Another thought was that if this is just a one-off and you're manually
> splitting up non-stop from all-stop to maintain parallelizability of the tests
> then I might not mind the new file.
I don't have plans to do this in any other test, so for now at least,
this is really just a one-off. It's special because "checkpoint"
only really works with single-threaded programs (because it uses
fork), while non-stop/all-stop mostly matters about threaded programs.
>
> IOW, tests that want to handle both all-stop and non-stop in one
> "make check" can do so in one file. Any loss in parallelization
> is probably minimal (modulo really big tests).
Yes, and we already do that in some tests.
>
> Btw, checkpoint.exp completes in 9 seconds on my system.
> I suspect doubling that and doing both all-stop and non-stop in
> the one file won't increase "make check-parallel" noticeably.
It makes a larger difference for me:
$ time make check RUNTESTFLAGS="checkpoint.exp"
real 0m24.716s
user 0m22.676s
sys 0m2.168s
$ time make check RUNTESTFLAGS="checkpoint.exp checkpoint-ns.exp"
real 0m49.241s
user 0m45.183s
sys 0m4.306s
$ time make check TESTS="gdb.base/checkpoint.exp gdb.base/checkpoint-ns.exp" -j
real 0m28.606s
user 0m49.684s
sys 0m5.590s
Thanks,
Pedro Alves