From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ej1-x636.google.com (mail-ej1-x636.google.com [IPv6:2a00:1450:4864:20::636]) by sourceware.org (Postfix) with ESMTPS id 001C4385840D for ; Thu, 1 Jun 2023 13:21:10 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 001C4385840D Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-ej1-x636.google.com with SMTP id a640c23a62f3a-97448470403so104181966b.1 for ; Thu, 01 Jun 2023 06:21:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1685625669; x=1688217669; h=mime-version:message-id:date:in-reply-to:subject:cc:to:from :user-agent:references:from:to:cc:subject:date:message-id:reply-to; bh=vDVn2Pj5bP8DsDdCKB1TyT/9VWC9naXKQAu5M+EJCIs=; b=viVLP6Z7C3/lM08t9zccuFQ2KBjnTUd1eTbkn+D2wTIVLXbOxHlY1ceZemMtWFY67a HdWYHFWTtZMJP1mbVm4Oqc8jMlATm1dEEDxAkEU8K21l+a+wMkeIzDrkwUF3pi6h11+Z 4NDi/JMbxI52xzIOabwFdxc6IAHIuZQMj88b2tLRpyGQWTZ19iPUR7ZrsxPnW7NVIko0 OZ8cm5Cws7idjyHKXfoiUhPz07ogMmmP/e/GKFO6L3Y1u9jVdaokufTk9pVFHR/dScUt Q/4C0a4GvXe6ixFGex31RAK/snwuxuOl177b//s9Ghfz0Td+jEFwtRZlyM0HG2OKSvXY Z+fg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1685625669; x=1688217669; h=mime-version:message-id:date:in-reply-to:subject:cc:to:from :user-agent:references:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=vDVn2Pj5bP8DsDdCKB1TyT/9VWC9naXKQAu5M+EJCIs=; b=kQrCSEkz1fbFicbw7eJZr7e06opyMTInF/19vjV/pVqy09ACop8L6wc6YcuY4epBio KcJbReyrM5E/TeQNXT9R9E70j6egvJM7ZcKyK3XCtbTcg85Cz//sQF0oV5LONHol2uAO zeBnWmxIPsfXUn6QPCrPxO2ou/SWgTpq07EDW9V+kemVopSeC2TGg6UvOibGyVYeDGvw Ve77+2LhJbtJauRnRXhIN0nBbgvK4xNMDwS3qd/ZkZzU6I4XzwUMYW+vqTTemPqBx+wv iBKHhJhr9uEqdEtIGUAK0qG+PzPtrhjoJn+Onve/wOFYnKFVrOX5ZJ+NqAGuL0BbaeUQ WXbA== X-Gm-Message-State: AC+VfDxCI4ZbLP6Z7IeBnkfTwMGYMaQcoTvszVM59LuORQETmcr3cMTN Wy/NdJfUlhmfVNtOhkUM4FIdD3+Y0D7qlNrllSk= X-Google-Smtp-Source: ACHHUZ4pE5j5sD3/pmedoWoFD/M70RxZZENjxtB8i1HrCP8lQALfBcUA3jvtIGWLk4lypGfx3GUGtg== X-Received: by 2002:a17:907:3f8d:b0:96f:57ba:4b56 with SMTP id hr13-20020a1709073f8d00b0096f57ba4b56mr10052534ejc.20.1685625669626; Thu, 01 Jun 2023 06:21:09 -0700 (PDT) Received: from localhost (p200300eaff33e08878ebf65188eb3c13.dip0.t-ipconnect.de. [2003:ea:ff33:e088:78eb:f651:88eb:3c13]) by smtp.gmail.com with ESMTPSA id re7-20020a170906d8c700b0094f109a5b3asm10571921ejb.135.2023.06.01.06.21.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 01 Jun 2023 06:21:08 -0700 (PDT) References: <20230531200602.165033-1-thiago.bauermann@linaro.org> <87zg5j4jf2.fsf@redhat.com> User-agent: mu4e 1.10.2; emacs 28.2 From: Thiago Jung Bauermann To: Andrew Burgess Cc: gdb-patches@sourceware.org Subject: Re: [PATCH] gdb/testsuite: Avoid infinite loop in gdb.reverse/step-reverse.exp In-reply-to: <87zg5j4jf2.fsf@redhat.com> Date: Thu, 01 Jun 2023 15:21:07 +0200 Message-ID: <87353btj24.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-4.3 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hello Andrew, Andrew Burgess writes: > Thiago Jung Bauermann via Gdb-patches > writes: > >> This testcase sometimes gets stuck in a loop for hours when running in our >> CI. The problem is that due to an issue unrelated to reverse debugging the >> inferior exits early, and because of the overly generic ".*" pattern the >> testcase keeps sending the "next" command without noticing that the >> inferior is gone. >> >> gdb_test_multiple has a pattern to detect that "The program is not being >> run.", but since it is placed after the patterns from the caller it won't >> be triggered. It also has a timeout pattern, but for some reason I don't >> understand it often doesn't trigger. > > The timeout is triggered between successful matches. So each time the > test matches the '-re -wrap ".*"' this counts as a successful match and > the timeout is reset. Thank you for the explanation, it was very helpful. If you don't mind, I will incorporate it in the patch description: > gdb_test_multiple has a pattern to detect that "The program is not being > run.", but since it is placed after the patterns from the caller it won't > be triggered. It also has a timeout pattern but because it is triggered > between successful matches, each time the test matches the '-re -wrap ".*"' > this counts as a successful match and the timeout is reset. -- Thiago