From patchwork Wed Aug 12 20:50:02 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joel Brobecker X-Patchwork-Id: 8167 Received: (qmail 12362 invoked by alias); 12 Aug 2015 20:50:07 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 12316 invoked by uid 89); 12 Aug 2015 20:50:07 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL, BAYES_00, KAM_LAZY_DOMAIN_SECURITY, RCVD_IN_DNSWL_LOW autolearn=no version=3.3.2 X-HELO: rock.gnat.com Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Wed, 12 Aug 2015 20:50:06 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 1E6A1290A7; Wed, 12 Aug 2015 16:50:04 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id Iq9OXvJF8Vu3; Wed, 12 Aug 2015 16:50:04 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id D6381290A0; Wed, 12 Aug 2015 16:50:03 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id 3CB3B46993; Wed, 12 Aug 2015 13:50:02 -0700 (PDT) Date: Wed, 12 Aug 2015 13:50:02 -0700 From: Joel Brobecker To: Luis Machado Cc: Pedro Alves , gdb-patches@sourceware.org, Sergio Durigan Junior Subject: Re: [PATCH v4 00/18] All-stop on top of non-stop Message-ID: <20150812205002.GI22245@adacore.com> References: <1432250354-2721-1-git-send-email-palves@redhat.com> <55C4E3BD.8040801@redhat.com> <20150812183208.GA24901@adacore.com> <55CB9907.9080506@redhat.com> <20150812202600.GA9183@adacore.com> <55CBAD06.4090707@codesourcery.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <55CBAD06.4090707@codesourcery.com> User-Agent: Mutt/1.5.23 (2014-03-12) > My idea of a testcase comment is at the beginning of the testcase file, > explaining what the test does and why it does it. I'd mention the amd64 > example as well, since it is part of why the test was created in the first > place. > > That should give others enough background to pursue an investigation about > why this potentially fails for them. > > My 2 cents anyway. There is this perception that the testcase was created because of the issue on amd64, but that's not true. The testcase was created, albeit in AdaCore's infrastructure only, to test that "next" in that context works as expected. Only later on did it allow us to find another bug which actually has nothing to do with the initial reason for creating the testcase. I hope I'm not looking like I'm splitting hair, but I feel like there is a bit of a misunderstanding somewhere, probably because the testcase appears as new to the GDB community and was combined with an amd64-specific fix. That being said, I propose the attached patch. I confess I'm not super convinced about the comment on amd64, as I think it might become one day irrelevant. But I don't mind it that much; if it is helpful to others... What do you think? gdb/testsuite/ChangeLog: * gdb.base/dso2dso.exp: Improve the testcase's documentation. From bbb505f2f822997b4285c61cede194fe2617b295 Mon Sep 17 00:00:00 2001 From: Joel Brobecker Date: Wed, 12 Aug 2015 13:40:54 -0700 Subject: [PATCH] gdb.base/dso2dso.exp: Improve testcase documentation. gdb/testsuite/ChangeLog: * gdb.base/dso2dso.exp: Improve the testcase's documentation. --- gdb/testsuite/gdb.base/dso2dso.exp | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/gdb/testsuite/gdb.base/dso2dso.exp b/gdb/testsuite/gdb.base/dso2dso.exp index b604012..16eb1f3 100644 --- a/gdb/testsuite/gdb.base/dso2dso.exp +++ b/gdb/testsuite/gdb.base/dso2dso.exp @@ -13,6 +13,16 @@ # You should have received a copy of the GNU General Public License # along with this program. If not, see . +# The purpose of that testcase is to verify that we can "next" over +# a call to a function provided by one shared library made from another +# shared library, and that GDB stops at the expected location. In this +# case, the call is made from sub1 (provided by libdso1) and we are +# calling sub2 (provided by libdso2). +# +# Note that, while this is not the main purpose of this testcase, it +# also happens to exercise an issue with displaced stepping on amd64 +# when libdso1 is mapped at an address greater than 0xffffffff. + if { [skip_shlib_tests] } { return 0 } @@ -53,10 +63,6 @@ if { ![runto_main] } { return -1 } -# Verify that we can "next" over the call to sub2 (provided by -# libdso2) make from sub1 (provided by libdso1), and land at -# the expected location. - set bp_location [gdb_get_line_number "STOP HERE" $srcfile_libdso1] gdb_breakpoint ${srcfile_libdso1}:${bp_location} -- 2.1.4