This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [testsuite patch] Regression for foll-vfork.exp [Re: [PATCH] Remove HP-UX reference in foll-vfork.exp]
- From: Simon Marchi <simon dot marchi at ericsson dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: Pedro Alves <palves at redhat dot com>, <gdb-patches at sourceware dot org>
- Date: Fri, 8 Jan 2016 16:04:13 -0500
- Subject: Re: [testsuite patch] Regression for foll-vfork.exp [Re: [PATCH] Remove HP-UX reference in foll-vfork.exp]
- Authentication-results: sourceware.org; auth=none
- References: <1450728591-7224-1-git-send-email-simon dot marchi at ericsson dot com> <56786B62 dot 2070303 at redhat dot com> <567971F3 dot 1070505 at ericsson dot com> <20160108205202 dot GB24397 at host1 dot jankratochvil dot net>
On 16-01-08 03:52 PM, Jan Kratochvil wrote:
> On Tue, 22 Dec 2015 16:53:23 +0100, Simon Marchi wrote:
>> Thanks, pushed.
>
> fe33faff35a8ee19db823149e764e3373e603bb9 is the first bad commit
> commit fe33faff35a8ee19db823149e764e3373e603bb9
> Author: Simon Marchi <simon.marchi@ericsson.com>
> Date: Tue Dec 22 10:52:31 2015 -0500
> Remove HP-UX reference in foll-vfork.exp
>
> FAIL: gdb.base/foll-vfork.exp: exec: vfork parent follow, finish after tcatch vfork: continue to vfork
> FAIL: gdb.base/foll-vfork.exp: exec: vfork child follow, finish after tcatch vfork: continue to vfork
> FAIL: gdb.base/foll-vfork.exp: exit: vfork parent follow, finish after tcatch vfork: continue to vfork
> FAIL: gdb.base/foll-vfork.exp: exit: vfork child follow, finish after tcatch vfork: continue to vfork
>
> It happens for plain gdb.base/foll-vfork.exp runtest on Fedora 23 x86_64.
>
> -Temporary catchpoint 2 (vforked process 24562), vfork () at ../sysdeps/unix/sysv/linux/x86_64/vfork.S:52^M
> +Temporary catchpoint 2 (vforked process 25345), vfork () at ../sysdeps/unix/sysv/linux/x86_64/vfork.S:52^M
> 52 pushq %rdi^M
> Current language: auto^M
> The current source language is "auto; currently asm".^M
> -(gdb) PASS: gdb.base/foll-vfork.exp: exec: vfork parent follow, finish after tcatch vfork: continue to vfork
> +(gdb) FAIL: gdb.base/foll-vfork.exp: exec: vfork parent follow, finish after tcatch vfork: continue to vfork
>
> -Temporary catchpoint 2 (vforked process 24629), vfork () at ../sysdeps/unix/sysv/linux/x86_64/vfork.S:52^M
> +Temporary catchpoint 2 (vforked process 25411), vfork () at ../sysdeps/unix/sysv/linux/x86_64/vfork.S:52^M
> 52 pushq %rdi^M
> Current language: auto^M
> The current source language is "auto; currently asm".^M
> -(gdb) PASS: gdb.base/foll-vfork.exp: exec: vfork child follow, finish after tcatch vfork: continue to vfork
> +(gdb) FAIL: gdb.base/foll-vfork.exp: exec: vfork child follow, finish after tcatch vfork: continue to vfork
>
>
> So I have reverted it and just simplified the comment, OK for check-in?
>
> The third case is not necessary during testing but I have changed back all the
> 3 cases.
>
>
> Jan
I don't see this on Ubuntu 14.04.3 LTS, gdb master, gcc 4.8.4. But if it's still
a valid test outcome, then yes it should be re-added. Sorry for breaking it.
-