From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 97498 invoked by alias); 8 Jan 2016 21:04:18 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 97487 invoked by uid 89); 8 Jan 2016 21:04:17 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.3 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY autolearn=no version=3.3.2 spammy=outcome, tcatch, foll-vfork.exp, UD:foll-vfork.exp X-HELO: usplmg21.ericsson.net Received: from usplmg21.ericsson.net (HELO usplmg21.ericsson.net) (198.24.6.65) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Fri, 08 Jan 2016 21:04:16 +0000 Received: from EUSAAHC004.ericsson.se (Unknown_Domain [147.117.188.84]) by usplmg21.ericsson.net (Symantec Mail Security) with SMTP id 68.17.32102.14420965; Fri, 8 Jan 2016 22:04:01 +0100 (CET) Received: from [142.133.110.144] (147.117.188.8) by smtp-am.internal.ericsson.com (147.117.188.86) with Microsoft SMTP Server id 14.3.248.2; Fri, 8 Jan 2016 16:04:14 -0500 Subject: Re: [testsuite patch] Regression for foll-vfork.exp [Re: [PATCH] Remove HP-UX reference in foll-vfork.exp] To: Jan Kratochvil References: <1450728591-7224-1-git-send-email-simon.marchi@ericsson.com> <56786B62.2070303@redhat.com> <567971F3.1070505@ericsson.com> <20160108205202.GB24397@host1.jankratochvil.net> CC: Pedro Alves , From: Simon Marchi Message-ID: <5690244D.9080309@ericsson.com> Date: Fri, 08 Jan 2016 21:04:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <20160108205202.GB24397@host1.jankratochvil.net> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2016-01/txt/msg00143.txt.bz2 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 > 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. -