From patchwork Mon Jan 13 17:03:18 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luis Machado X-Patchwork-Id: 182804 Delivered-To: patch@linaro.org Received: by 2002:ac9:44c4:0:0:0:0:0 with SMTP id t4csp4543748och; Mon, 13 Jan 2020 09:21:15 -0800 (PST) X-Google-Smtp-Source: APXvYqwkMd+RDyonNMAq/eb4S0y/JPdnoUDHm6R/WmAVmKf4Y9KKkMoMob2weiXUHIGy4JQsLt0y X-Received: by 2002:aca:c786:: with SMTP id x128mr12876102oif.2.1578935025901; Mon, 13 Jan 2020 09:03:45 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1578935025; cv=none; d=google.com; s=arc-20160816; b=SaC42iCPEkWDmkZM8sz57CTphj30rvcMcn1HPFXn6VyhYigjueq4PA0+9r0m2U6ONm 3datUWrLDmChfKB6Gf89g5NC86Sn+zu3JcmhhGqPQ2htUEodcehXBRqR5uejs0ChyvsN efWCOpXizFnFe923PPBwFMMT0YxYt/kTDqZOMehaeLnOrZCBRSDb/We8CVolCrDGjkh1 VAZfAvc5bVjYiCHuXb2NjJA2ny/TW0XccvUNzVoNY4DYErCeCdsDv9aaTRCBCLv84B41 lpA6BvWVDUZDzejge6ZL9LT30giZsE8tR/eTrqYHkNlKdf7tCYNyhW3iNsJiBKrWVdqZ M5vA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature:delivered-to:sender:list-help:list-post:list-archive :list-subscribe:list-unsubscribe:list-id:precedence:mailing-list :dkim-signature:domainkey-signature; bh=WOPh4DRQhh8T7Yhxqxd1FwDs7xn3TNs1RgiDJMiBFls=; b=vqkGKZ9NtM588gXIxaSs/dFiA+q42vQNtUhg3xD9zIQE1l4+HntIK7Xz+ICboX9+bD BFQXb9RJKob7HOK5CqknBuKEDQweNTFtOBUGgKnrdScDrJghjeCo18dnsqjhFvY4oHjK GdUPh1fmk8F/aiIoo0rx3d0LFld6GNM+v2ixkmsJ2liBvf2bdbn76pRMhXIFhanxhSGu PuWCwX61T02BbUDK5Rg4PdoQZ3n8CDyMyxeOno2zm6BXQA2sM36me5gxyn+qkXJ6btnG eelvS2w5qug0ln0W1QkV20Kn1asp4/r0qT5klpJDsr2BjZ77YQ16hFba+VEKK++ssz7u 7zuQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sourceware.org header.s=default header.b=veyfOhzK; dkim=pass header.i=@linaro.org header.s=google header.b="vBlj/jFP"; spf=pass (google.com: domain of gdb-patches-return-163017-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom="gdb-patches-return-163017-patch=linaro.org@sourceware.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id y22si6936893oti.269.2020.01.13.09.03.45 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 13 Jan 2020 09:03:45 -0800 (PST) Received-SPF: pass (google.com: domain of gdb-patches-return-163017-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; Authentication-Results: mx.google.com; dkim=pass header.i=@sourceware.org header.s=default header.b=veyfOhzK; dkim=pass header.i=@linaro.org header.s=google header.b="vBlj/jFP"; spf=pass (google.com: domain of gdb-patches-return-163017-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom="gdb-patches-return-163017-patch=linaro.org@sourceware.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:cc:subject:date:message-id:in-reply-to :references; q=dns; s=default; b=fnqsqH1+u+kl5ly7VshKf2ww0Y8asgb NwSls3HUkhRHTwgHIzq2uqRbqSpxyeEdOKzUTsoZPIW/r2/++fCVxc8eT/xoTWBQ QxFeRETDt4lgUnUUBMmG5ozSAkfNLa3JlBBqbahTNRqrjDCmpebQcMkD5TwoXgU6 wxEg0ejb60cA= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:cc:subject:date:message-id:in-reply-to :references; s=default; bh=g4NngKnGkpzgpzddPVJlwQYiXdg=; b=veyfO hzKzl5nOgqjgNz6wE0YSjN1eE/bTA5WQAVKcrkGpnVHcvCGl90t42jbuwLVETq1x YCa5caTMqvKYV5YgpdNrp7BpA4DEAVHkGYJvNTBWN2FlwsvyVANaTf4VC6uT0Emu nuak+3swyX+NrKtTMJNWa/m61mAK3GgjnbUeJU= Received: (qmail 10151 invoked by alias); 13 Jan 2020 17:03:35 -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 10108 invoked by uid 89); 13 Jan 2020 17:03:35 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.1 spammy=Trace, exercises X-HELO: mail-qk1-f179.google.com Received: from mail-qk1-f179.google.com (HELO mail-qk1-f179.google.com) (209.85.222.179) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 13 Jan 2020 17:03:33 +0000 Received: by mail-qk1-f179.google.com with SMTP id r14so9116957qke.13 for ; Mon, 13 Jan 2020 09:03:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=WOPh4DRQhh8T7Yhxqxd1FwDs7xn3TNs1RgiDJMiBFls=; b=vBlj/jFPY17+SyZ2dbCuEV2SpZS0/2jaKNp92DeLn5K6p4EvxIhuN8L7PAiB2SiN7z x7shqlOvyFdRbnnMBg2IZDb9NgvernikjEaOMXYCowcuCYsSn8r3ythYZpRkcTl0yNGw 6cSDBJUb483gdx55Hb0cWdTFODl9p6s+dWrW2tsmYIv/O4gm9ewy6AqIJ+vVffcDcZC/ o6ODiQFiEr6YuIAc31UDYbSLfUscad95hzQcsXFGdtD8YY/fkN8/AUduLK2imf7DNyFW 9JhB5qKWH02i+QSPPDXD0K3xn8JqcNSSzDS1Qven3T8Uk40PYhSA41j7Fkdu7N+3rZTK MVsg== Return-Path: Received: from localhost.localdomain ([179.183.9.215]) by smtp.gmail.com with ESMTPSA id g18sm5252703qki.13.2020.01.13.09.03.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Jan 2020 09:03:30 -0800 (PST) From: Luis Machado To: gdb-patches@sourceware.org Cc: alan.hayward@arm.com, tankut.baris.aktemur@intel.com Subject: [PATCH 2/2] [AArch64] Test handling of additional brk instruction patterns Date: Mon, 13 Jan 2020 14:03:18 -0300 Message-Id: <20200113170318.6335-2-luis.machado@linaro.org> In-Reply-To: <20200113170318.6335-1-luis.machado@linaro.org> References: <20191223173432.16955-1-luis.machado@linaro.org> <20200113170318.6335-1-luis.machado@linaro.org> X-IsSubscribed: yes This test exercises the previous patch's code and makes sure GDB can properly get a SIGTRAP from various brk instruction patterns. GDB needs to be able to see the program exiting normally. If GDB doesn't support the additional brk instructions, we will see timeouts. We bail out with the first timeout since we won't be able to step through the program breakpoint anyway, so it is no use carrying on. gdb/testsuite/ChangeLog: 2020-01-13 Luis Machado * gdb.arch/aarch64-brk-patterns.c: New source file. * gdb.arch/aarch64-brk-patterns.exp: New test. --- gdb/testsuite/gdb.arch/aarch64-brk-patterns.c | 30 +++++++++ .../gdb.arch/aarch64-brk-patterns.exp | 67 +++++++++++++++++++ 2 files changed, 97 insertions(+) create mode 100644 gdb/testsuite/gdb.arch/aarch64-brk-patterns.c create mode 100644 gdb/testsuite/gdb.arch/aarch64-brk-patterns.exp -- 2.17.1 diff --git a/gdb/testsuite/gdb.arch/aarch64-brk-patterns.c b/gdb/testsuite/gdb.arch/aarch64-brk-patterns.c new file mode 100644 index 0000000000..ccf9a35a94 --- /dev/null +++ b/gdb/testsuite/gdb.arch/aarch64-brk-patterns.c @@ -0,0 +1,30 @@ +/* This file is part of GDB, the GNU debugger. + + Copyright 2020 Free Software Foundation, Inc. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +int main(void) +{ + /* Dummy instruction just so GDB doesn't stop at the first breakpoint + instruction. */ + __asm __volatile ("nop\n\t"); + + /* Multiple BRK instruction patterns. */ + __asm __volatile ("brk %0\n\t" ::"n"(0x0)); + __asm __volatile ("brk %0\n\t" ::"n"(0x900 + 0xf)); + __asm __volatile ("brk %0\n\t" ::"n"(0xf000)); + + return 0; +} diff --git a/gdb/testsuite/gdb.arch/aarch64-brk-patterns.exp b/gdb/testsuite/gdb.arch/aarch64-brk-patterns.exp new file mode 100644 index 0000000000..3532a0df95 --- /dev/null +++ b/gdb/testsuite/gdb.arch/aarch64-brk-patterns.exp @@ -0,0 +1,67 @@ +# Copyright 2020 Free Software Foundation, Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . +# +# This file is part of the gdb testsuite. + +# Test if GDB stops at various BRK instruction patterns inserted into +# the code. + +if {![is_aarch64_target]} { + verbose "Skipping ${gdb_test_file_name}." + return +} + +standard_testfile +if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } { + return -1 +} + +if ![runto_main] { + untested "could not run to main" + return -1 +} + +set keep_going 1 +set count 0 +set old_timeout $timeout +set timeout 10 + +while { $keep_going } { + + set test "brk instruction $count causes SIGTRAP" + + # Continue to next program breakpoint instruction. + gdb_test_multiple "continue" $test { + -re "Program received signal SIGTRAP, Trace/breakpoint trap.*$gdb_prompt $" { + pass $test + + # Insert a breakpoint at the program breakpoint instruction so GDB + # can step over it. + gdb_test "break" \ + "Breakpoint $decimal at $hex: file .*$srcfile, line $decimal.*" \ + "insert breakpoint at brk instruction $count" + } + -re "exited normally.*$gdb_prompt $" { + set keep_going 0 + } + timeout { + fail $test + set keep_going 0 + } + } + set count [expr "$count + 1"] +} + +set timeout $old_timeout