From patchwork Thu Nov 17 14:02:09 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Uros Bizjak X-Patchwork-Id: 126226 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id CD233B721E for ; Fri, 18 Nov 2011 01:02:31 +1100 (EST) Received: (qmail 11318 invoked by alias); 17 Nov 2011 14:02:27 -0000 Received: (qmail 11308 invoked by uid 22791); 17 Nov 2011 14:02:25 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_LOW, TW_ZJ X-Spam-Check-By: sourceware.org Received: from mail-gx0-f175.google.com (HELO mail-gx0-f175.google.com) (209.85.161.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 17 Nov 2011 14:02:10 +0000 Received: by ggnh4 with SMTP id h4so1182775ggn.20 for ; Thu, 17 Nov 2011 06:02:10 -0800 (PST) MIME-Version: 1.0 Received: by 10.236.193.68 with SMTP id j44mr9000761yhn.97.1321538529792; Thu, 17 Nov 2011 06:02:09 -0800 (PST) Received: by 10.146.137.4 with HTTP; Thu, 17 Nov 2011 06:02:09 -0800 (PST) In-Reply-To: <8EDD1E84-FF74-4F58-8762-98EE148A6BC3@sandoe-acoustics.co.uk> References: <20111117104937.GY14325@bubble.grove.modra.org> <8EDD1E84-FF74-4F58-8762-98EE148A6BC3@sandoe-acoustics.co.uk> Date: Thu, 17 Nov 2011 15:02:09 +0100 Message-ID: Subject: Re: [PATCH, testsuite]: Fix timeout in simulate-thread.exp From: Uros Bizjak To: Iain Sandoe Cc: GCC Patches , Alan Modra , Andrew MacLeod , Richard Henderson Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org On Thu, Nov 17, 2011 at 2:22 PM, Iain Sandoe wrote: > > On 17 Nov 2011, at 10:49, Alan Modra wrote: > >> On Thu, Nov 17, 2011 at 10:54:23AM +0100, Uros Bizjak wrote: >>> >>> The solution for this problem is to search for "simulate_thread_done" >>> in gdb log. >> >> Thanks!  I like this for another reason too:  I've seen these tests >> fail due to gdb never reporting simulate_thread_main on any >> instruction in that function. > > ... works for me on powerpc-darwin9 ... > > with a local experimental 7.3.1 port to powerpc-darwin9, some more tests > pass, but not all ... > ... what base version/patch is needed to get all the tests to pass for you? It looks to me that you will also need following rs6000 gdb patch, otherwise relative addresses of a conditional branch will be calculated against opening lwarx/ldarx instruction: Uros. --- rs6000-tdep.c 2011-03-18 19:52:31.000000000 +0100 +++ rs6000-tdep.c.ub 2011-11-17 14:58:46.000000000 +0100 @@ -1126,7 +1126,7 @@ ppc_deal_with_atomic_sequence (struct fr if (absolute) breaks[1] = immediate; else - breaks[1] = pc + immediate; + breaks[1] = loc + immediate; bc_insn_count++; last_breakpoint++;