From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8579 invoked by alias); 2 Oct 2014 18:00:05 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 8569 invoked by uid 89); 2 Oct 2014 18:00:04 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.7 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Thu, 02 Oct 2014 18:00:02 +0000 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s92Hxua0006543 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 2 Oct 2014 13:59:56 -0400 Received: from tucnak.zalov.cz (ovpn-116-116.ams2.redhat.com [10.36.116.116]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id s92HxsR0011587 (version=TLSv1/SSLv3 cipher=AES128-GCM-SHA256 bits=128 verify=NO); Thu, 2 Oct 2014 13:59:55 -0400 Received: from tucnak.zalov.cz (localhost [127.0.0.1]) by tucnak.zalov.cz (8.14.9/8.14.9) with ESMTP id s92HxqLS028787; Thu, 2 Oct 2014 19:59:52 +0200 Received: (from jakub@localhost) by tucnak.zalov.cz (8.14.9/8.14.9/Submit) id s92Hxoxj028490; Thu, 2 Oct 2014 19:59:50 +0200 Date: Thu, 02 Oct 2014 18:00:00 -0000 From: Jakub Jelinek To: Segher Boessenkool , Andrew MacLeod , Bernd Schmidt , gcc-patches , richard.sandiford@arm.com, rdsandiford@googlemail.com Subject: Re: parallel check output changes? Message-ID: <20141002175950.GJ1986@tucnak.redhat.com> Reply-To: Jakub Jelinek References: <541AD880.7080703@redhat.com> <541AF451.3070406@redhat.com> <541B1710.8060809@codesourcery.com> <20140918173609.GM17454@tucnak.redhat.com> <20140918184455.GB28595@gate.crashing.org> <20140919093723.GA26414@gate.crashing.org> <87iokel5c0.fsf@e105548-lin.cambridge.arm.com> <5422DB41.1090800@redhat.com> <20141002164739.GA25260@gate.crashing.org> <871tqqnz4k.fsf@googlemail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <871tqqnz4k.fsf@googlemail.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-IsSubscribed: yes X-SW-Source: 2014-10/txt/msg00206.txt.bz2 On Thu, Oct 02, 2014 at 06:46:19PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > > On Wed, Sep 24, 2014 at 10:54:57AM -0400, Andrew MacLeod wrote: > >> Is this suppose to be resolved now? I'm still seeing some issues with a > >> branch cut from mainline from yesterday. > > > > Confirmed. The following patch works for me, and Andrew has tested it > > as well. The comment it removes isn't valid before the patch either. > > I get the impression from a short dismissal like that that this script > is pretty hated :-(. Remember that originally the script was trying to No, it is certainly appreciated that it speeded up the processing. > make the result of combining separate .sum files the same as the .sum > file you'd get for -j1. As Jakub said upthread, that's a lost cause > with the new approach to parallel testing, but I think the comment was > valid while matching -j1 was still a goal. I'm sorry for invalidating those assumptions. Indeed, before my recent changes, all tests for the same testcase name were run serially by the same job. If we wanted to preserve that property, we could e.g. store the results of gcc_parallel_test_run_p in some tcl array with testcase as the key, and after the if { $gcc_runtest_parallelize_enable == 0 } { return 1 } test add a test if we've been asked about a particular testcase already, just return what we've returned before. Perhaps accompany that with lowering the granularity (e.g. from 10 to 5). Jakub