From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 117885 invoked by alias); 6 Feb 2016 20:54:38 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 117856 invoked by uid 89); 6 Feb 2016 20:54:37 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.5 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=tfind, tfile, sk:foll-ex, sk:follex X-HELO: xyzzy.0x04.net Received: from xyzzy.0x04.net (HELO xyzzy.0x04.net) (109.74.193.254) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 06 Feb 2016 20:54:35 +0000 Received: from hogfather.0x04.net (89-65-66-135.dynamic.chello.pl [89.65.66.135]) by xyzzy.0x04.net (Postfix) with ESMTPS id 6FEAC3FE5C for ; Sat, 6 Feb 2016 21:55:21 +0100 (CET) Received: by hogfather.0x04.net (Postfix, from userid 1000) id BC18258008C; Sat, 6 Feb 2016 21:54:32 +0100 (CET) From: =?UTF-8?q?Marcin=20Ko=C5=9Bcielnicki?= To: gdb-patches@sourceware.org Cc: =?UTF-8?q?Marcin=20Ko=C5=9Bcielnicki?= Subject: [PATCH 6/6] gdb.trace: Add a testcase for tdesc in tfile. Date: Sat, 06 Feb 2016 20:54:00 -0000 Message-Id: <1454792070-25410-2-git-send-email-koriakin@0x04.net> In-Reply-To: <1454792070-25410-1-git-send-email-koriakin@0x04.net> References: <1454773157-31569-1-git-send-email-koriakin@0x04.net> <1454792070-25410-1-git-send-email-koriakin@0x04.net> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2016-02/txt/msg00167.txt.bz2 This tests whether $ymm15 can be correctly collected and printed from tfile. It covers: - storing tdesc in tfile (without that, $ymm15 doesn't exist) - ax_pseudo_register_collect for x86 (without that, $ymm15 cannot be collected) - register order in tfile_fetch_registers (without that, $ymm15h is fetched from wrong position) - off-by-one in tfile_fetch_registers (without that, $ymm15h is incorrectly considered to be out of bounds) - using proper tdesc in encoding tracepoint actions (without that, internal error happens due to $ymm15h being considered unavailable) gdb/testsuite/ChangeLog: * gdb.trace/tfile-avx.c: New test. * gdb.trace/tfile-avx.exp: New test. --- This test requires https://sourceware.org/ml/gdb-patches/2016-01/msg00115.html and all the previous 5 patches in this series. gdb/testsuite/ChangeLog | 5 +++ gdb/testsuite/gdb.trace/tfile-avx.c | 48 ++++++++++++++++++++ gdb/testsuite/gdb.trace/tfile-avx.exp | 82 +++++++++++++++++++++++++++++++++++ 3 files changed, 135 insertions(+) create mode 100644 gdb/testsuite/gdb.trace/tfile-avx.c create mode 100644 gdb/testsuite/gdb.trace/tfile-avx.exp diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 046f112..c9a596c 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2016-02-06 Marcin Kościelnicki + + * gdb.trace/tfile-avx.c: New test. + * gdb.trace/tfile-avx.exp: New test. + 2016-02-04 Yao Qi * gdb.base/foll-exec-mode.c: Include limits.h. diff --git a/gdb/testsuite/gdb.trace/tfile-avx.c b/gdb/testsuite/gdb.trace/tfile-avx.c new file mode 100644 index 0000000..71107b8 --- /dev/null +++ b/gdb/testsuite/gdb.trace/tfile-avx.c @@ -0,0 +1,48 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2016 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 . */ + +/* + * Test program for reading target description from tfile: collects AVX + * registers on x86_64. + */ + +#include + +void +dummy (void) +{} + +static void +end (void) +{} + +int main (void) +{ + register __v8si a asm("ymm15") = { + 0x12340001, + 0x12340002, + 0x12340003, + 0x12340004, + 0x12340005, + 0x12340006, + 0x12340007, + 0x12340008, + }; + asm volatile ("traceme: call dummy" : : "x" (a)); + end (); + return 0; +} diff --git a/gdb/testsuite/gdb.trace/tfile-avx.exp b/gdb/testsuite/gdb.trace/tfile-avx.exp new file mode 100644 index 0000000..1acf4cb --- /dev/null +++ b/gdb/testsuite/gdb.trace/tfile-avx.exp @@ -0,0 +1,82 @@ +# Copyright 2016 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 . + +if { ! [istarget "x86_64*-*-*linux*"] } { + verbose "Skipping tfile AVX test (target is not x86_64 linux)." + return +} + +load_lib "trace-support.exp" + +standard_testfile + +if {[prepare_for_testing $testfile.exp $testfile $srcfile \ + [list debug nowarnings additional_flags=-mavx]]} { + return -1 +} + +if ![runto_main] { + fail "Can't run to main to check for trace support" + return -1 +} + +if ![gdb_target_supports_trace] { + unsupported "target does not support trace" + return -1 +} + +send_gdb "print \$ymm15\n" +gdb_expect { + -re "\\$\[0-9\]+ = void.*$gdb_prompt \$" { + verbose "Skipping tfile AVX test (target doesn't support AVX)." + return + } + -re "\\$\[0-9\]+ = \\{.*}.*$gdb_prompt \$" { + # All is well. + } +} + +gdb_test "trace traceme" ".*" + +gdb_trace_setactions "set actions for tracepoint" "" \ + "collect \$ymm15" "^$" + +gdb_breakpoint "end" + +gdb_test_no_output "tstart" + +gdb_test "continue" ".*Breakpoint \[0-9\]+, end .*" + +set tracefile [standard_output_file ${testfile}] + +# Save trace frames to tfile. +gdb_test "tsave ${tracefile}.tf" \ + "Trace data saved to file '${tracefile}.tf'.*" \ + "save tfile trace file" + +# Change target to tfile. +set test "change to tfile target" +gdb_test_multiple "target tfile ${tracefile}.tf" "$test" { + -re "A program is being debugged already. Kill it. .y or n. " { + send_gdb "y\n" + exp_continue + } + -re "$gdb_prompt $" { + pass "$test" + } +} + +gdb_test "tfind 0" "Found trace frame 0, tracepoint .*" + +gdb_test "print/x \$ymm15.v8_int32" "\\$\[0-9\]+ = \\{0x12340001, .*, 0x12340008}.*" -- 2.7.0