From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id ylQ2MGFT6mgpiykAWB0awg (envelope-from ) for ; Sat, 11 Oct 2025 08:53:53 -0400 Authentication-Results: simark.ca; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20230601 header.b=eIhX32Dr; dkim-atps=neutral Received: by simark.ca (Postfix, from userid 112) id B2B881E0B6; Sat, 11 Oct 2025 08:53:53 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-25) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-2.4 required=5.0 tests=ARC_SIGNED,ARC_VALID,BAYES_00, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,RCVD_IN_VALIDITY_CERTIFIED_BLOCKED, RCVD_IN_VALIDITY_RPBL_BLOCKED,RCVD_IN_VALIDITY_SAFE_BLOCKED autolearn=ham autolearn_force=no version=4.0.1 Received: from server2.sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 065171E047 for ; Sat, 11 Oct 2025 08:53:52 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 7C7653857810 for ; Sat, 11 Oct 2025 12:53:51 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 7C7653857810 Authentication-Results: sourceware.org; dkim=pass (2048-bit key, unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20230601 header.b=eIhX32Dr Received: from mail-wm1-x335.google.com (mail-wm1-x335.google.com [IPv6:2a00:1450:4864:20::335]) by sourceware.org (Postfix) with ESMTPS id C3CED385780D for ; Sat, 11 Oct 2025 12:53:08 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org C3CED385780D Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gmail.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org C3CED385780D Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2a00:1450:4864:20::335 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1760187189; cv=none; b=hTtMlC1udq4Ent85t1sn9DgZCifcSrhp3FaSB5m0RsXVz1S3a81kL4lusJQ3+4X0xRlADbOdgi2LGBzKnIFeiwcwkqjuHd58Kl5U27dAHzUCRd47x5QjWLV/MB8fu9sELQNAaFrcbQKib2lJ5kRG6hYuJOtb67icacvGZhXicXY= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1760187189; c=relaxed/simple; bh=mgEmnO+1IGCIv+9fcVM2G38bjtj6xWQmaapbGgXkbEc=; h=DKIM-Signature:Message-ID:Date:MIME-Version:Subject:To:From; b=TsUVcpPE9iLLg+HX9TBvADU3G0Hb8StAlM+PbA5+0yovNwAHwxOMr5eGTcAC0prrZUzYZv/CXdi+pYOHud0W8M1wlMN4goamU4gvCM3iRjAld0BT6xMPyHiEs+LWhbWkTMU5LYTAP8Y47JD3Yl7k6hYcLzGoXzUsZL3nFBM3Dds= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C3CED385780D Received: by mail-wm1-x335.google.com with SMTP id 5b1f17b1804b1-46e6a689bd0so20345505e9.1 for ; Sat, 11 Oct 2025 05:53:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1760187187; x=1760791987; darn=sourceware.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=DKutOhEtzlDvUvtlLYjbTtAg92WUgM/Ggdd/BYVUsnA=; b=eIhX32DrY2G/eLNn1SpWkAOlKjYdTnprSKB5EvlSjFhq29DvC4zV3beUAp9eccQm5P up8bOCZcHK30UIxB4t3Xh0kYG2YBDzUdppc4kQe+fPDjjEST8o4bGRG2IehUZ/Ry3/I1 xN2g7ovtMBugHE6mdKZiFTkuSCFswjFHeAcgc85CFN2f2FdSMkA0MQyB/A0YBMuBGcsM Ah6hJsQ6diqt0FfpMYm1binkRHDcoOp25meBFYJDKuyHICPE1IS2YzlgtV7Ac2/pr9vH mOt3oH31bqVrnG5zLvaIWsWAyiaIqTv6jQ9+0ngoGGcQugwUURmifFf/nGuki1W+NBgG VTZg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1760187187; x=1760791987; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=DKutOhEtzlDvUvtlLYjbTtAg92WUgM/Ggdd/BYVUsnA=; b=nPkc9+pjNJPmNsJUT62ceG59gdkq3mjmDmPnKkix2ASjLdqP9uPZv5q4QhWFeNeBSv lOo07RsRF56vwNFAJKoGQQLg5KeroNl2JyhdEVeg6UI4UIy8EhIHZovpyX7mv8iyR/Fs RgUzsCRHh8blt1Y50yAla5xE8aWpe3rPeXnUkE2lFlsf1SKD5pzpyR+Dq01UOSmP9+Bz VXlaDgoRzER757szPdPmapwJRfPwq97Ri3KV0eydWUAJ+wTvU7HAN3LRd6yrxJ4IUGCR acVVfZczntT1dEml2ZGtjn0eoVrzXHIFsgYdLnCtwjATYIPZJQxPZbppKxRdT4nqaZsc vfCA== X-Forwarded-Encrypted: i=1; AJvYcCWyP+t5dcscDaRZbOu4ec/9tlEdPaUBgY+e6OdDJiuH9q77aPfBIKR5IM36jMc72pET+5Oi/VH7Ywysaw==@sourceware.org X-Gm-Message-State: AOJu0YzYcmnd7T8glv387omUw83wBI+xqwNDBfVIfh73xihoLxbUw7p9 APLwTLWjOpp7T2jooQ6x2f+Qp6zcDbr2uIk8MzmhS0rpDqYECQHy9UFPekU9aA== X-Gm-Gg: ASbGncuiP1fb3WmWjeXCIJzBBhqyYmvufs7eEn4VLihMl8fSR7Sdz0VR65bJ/8pVKXi DnnHe22c1VYlRErUN8EOV0Vbw3ZE7F7bfcyCZ8gBJkum4gkml338baqw7Nu8R+FMH0/7YyCObKp q/FdBoovoQqGVeql4rKaIchzoQWnw6SzS2248gN4DizOFP8QxrXjOJonvCsNZhl/OXwx//4vNWM LQPNxSDmv1ef09Gt3sqeMLwh4nFNSc4gd4MnKRbs57skabjBqcyZceUHoKdsVbKEyouBfBmthdy i6HKkHmfY9uOwRmRJqMmFtUYuGQVWeTjqk0C8+yurPUc0ZDDgm8GqiKGg4trzYj9fEDnQ1YawMD 3NxvaRmdWM/IBXlI1ZRFRJNT7kAVfj/d2rDHJSaUyunkDYcmzYUjF7Q== X-Google-Smtp-Source: AGHT+IErYGwaK3RN4jvd/uW2bFVF6UkWptVE5Y8Zw5T/wOWLJ7zM/nxwDEN5vUMyhgYp/0ZuLjkMVg== X-Received: by 2002:a05:600c:4745:b0:46d:45e:3514 with SMTP id 5b1f17b1804b1-46fa9af3045mr106879065e9.17.1760187187159; Sat, 11 Oct 2025 05:53:07 -0700 (PDT) Received: from [192.168.0.38] ([86.12.216.189]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-46fab3d92c0sm81653165e9.3.2025.10.11.05.53.05 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 11 Oct 2025 05:53:06 -0700 (PDT) Message-ID: <5289b4c6-90be-4314-b3d0-b21a3f219843@gmail.com> Date: Sat, 11 Oct 2025 13:53:05 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 5/5] gdb/aarch64: Tests for fpmr Content-Language: en-US To: Ezra.Sitorus@arm.com, gdb-patches@sourceware.org Cc: thiago.bauermann@linaro.org References: <20251007123132.26769-1-Ezra.Sitorus@arm.com> <20251007123132.26769-6-Ezra.Sitorus@arm.com> From: Luis In-Reply-To: <20251007123132.26769-6-Ezra.Sitorus@arm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces~public-inbox=simark.ca@sourceware.org There seems to be a few whitespace issues with this patch. Please check them with "git diff --check" or "git show --check" and it should show you. Also, running tclint on your exp files seems to point out these: gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp:22:5: expected braced word or word without substitutions in argument interpreted as expr [command-args] gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp:48:1: expected braced word or word without substitutions in argument interpreted as expr [command-args] gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.exp:28:1: expected braced word or word without substitutions in argument interpreted as expr [command-args] gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.exp:51:21: expression with substitutions should be enclosed by braces [unbraced-expr] gdb/testsuite/gdb.arch/aarch64-fpmr.exp:26:1: expected braced word or word without substitutions in argument interpreted as expr [command-args] On 07/10/2025 13:31, Ezra.Sitorus@arm.com wrote: > From: Ezra Sitorus > > Add tests for FPMR support in gdb/gdbserver. These tests check > availability of FPMR, reading/writing to FPMR, core file generation and > preservation under sighandler frame unwinding. > > A run of the full gdb testsuite has been done on aarch64-none-linux-gnu > without FPMR support. I have run the gdb.arch tests on Shrinkwrap with > FPMR support. > --- > Changes from v1->v2: > * Removed fpmr modification functions to make the tests easier to > follow. > * Test writing to fpmr in aarch64-fpmr.exp from outside the inferior > program. > * Rewrite allow_aarch64_fpmr_tests to follow allow_aarch64_sve_tests. > > Ezra > > gdb/testsuite/gdb.arch/aarch64-fpmr-core.c | 40 ++++++ > gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp | 97 +++++++++++++++ > .../gdb.arch/aarch64-fpmr-sighandler.c | 55 ++++++++ > .../gdb.arch/aarch64-fpmr-sighandler.exp | 77 ++++++++++++ > gdb/testsuite/gdb.arch/aarch64-fpmr.c | 117 ++++++++++++++++++ > gdb/testsuite/gdb.arch/aarch64-fpmr.exp | 72 +++++++++++ > gdb/testsuite/lib/gdb.exp | 66 ++++++++++ > 7 files changed, 524 insertions(+) > create mode 100644 gdb/testsuite/gdb.arch/aarch64-fpmr-core.c > create mode 100644 gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp > create mode 100644 gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.c > create mode 100644 gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.exp > create mode 100644 gdb/testsuite/gdb.arch/aarch64-fpmr.c > create mode 100644 gdb/testsuite/gdb.arch/aarch64-fpmr.exp > > diff --git a/gdb/testsuite/gdb.arch/aarch64-fpmr-core.c b/gdb/testsuite/gdb.arch/aarch64-fpmr-core.c > new file mode 100644 > index 00000000000..785fbd42d5c > --- /dev/null > +++ b/gdb/testsuite/gdb.arch/aarch64-fpmr-core.c > @@ -0,0 +1,40 @@ > +/* This file is part of GDB, the GNU debugger. > + > + Copyright 2008-2025 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 . */ > + > +#include > + > +uint64_t crash_address = 0; > + > +void > +set_fpmr (uint64_t value) > +{ > + register uint64_t x0_val asm ("x0") = value; > + /* msr fpmr, x0 */ > + __asm__ volatile (".inst 0xd51b4440" : : ); > +} > + > +int > +main (void) > +{ > + set_fpmr (0x3fff7fc049); > + > + /* Check FPMR */ > + > + *((uint64_t *) crash_address) = 0xDEAD; /* crash point */ > + > + return 1; > +} > diff --git a/gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp b/gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp > new file mode 100644 > index 00000000000..6f8ee996d27 > --- /dev/null > +++ b/gdb/testsuite/gdb.arch/aarch64-fpmr-core.exp > @@ -0,0 +1,97 @@ > +# Copyright (C) 2018-2025 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 generating and reading a core file with FPMR. > + > +proc check_fpmr_core_file { core_filename } { > + # Load the core file. > + if [gdb_test "core $core_filename" \ > + [multi_line \ > + "Core was generated by .*" \ > + "Program terminated with signal SIGSEGV, Segmentation fault\\." \ > + "#0 ${::hex} in main \\(.*\\) at .*" \ > + ".* \\*\\(\\(uint64_t \\*\\) crash_address\\) = 0xDEAD.*"] \ > + "load core file"] { > + untested "failed to generate core file" > + return -1 > + } > + > + # Check the value of FPMR in the core file. > + gdb_test "print/x \$fpmr" " = 0x3fff7fc049" \ > + "fpmr contents from core file" > +} > + > +require is_aarch64_target > +require allow_aarch64_fpmr_tests > + > +standard_testfile > +if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } { > + return -1 > +} > + > +set binfile [standard_output_file ${testfile}] > + > +if ![runto_main] { > + untested "could not run to main" I don´t quite recall if we decided to not issue these untested/fail messages here. Looking at the overall testsuite, looks like we mostly just return -1. > + return -1 > +} > + > +set crash_breakpoint "crash point" > +gdb_breakpoint [gdb_get_line_number $crash_breakpoint] > +gdb_continue_to_breakpoint $crash_breakpoint > + > +gdb_test "print/x \$fpmr" " = 0x3fff7fc049" \ > + "fpmr contents from core file" > + > +gdb_test "continue" \ > +[multi_line \ > + "Program received signal SIGSEGV, Segmentation fault\\." \ > + "${::hex} in main \\(\\).* at .*" \ > + ".* \\*\\(\\(uint64_t \\*\\) crash_address\\) = 0xDEAD.*"] \ > + "run to crash" > + > +# Generate the gcore core file. > +set gcore_filename [standard_output_file "${testfile}.gcore"] > +set gcore_generated [gdb_gcore_cmd "$gcore_filename" "generate gcore file"] > + > +# Generate a native core file. > +set core_filename [core_find ${binfile}] > +set core_generated [expr {$core_filename != ""}] > + > +# At this point we have a couple core files, the gcore one generated by GDB > +# and the native one generated by the Linux Kernel. Make sure GDB can read > +# both correctly. > + > +if {$gcore_generated} { > + clean_restart > + gdb_load ${binfile} > + with_test_prefix "gcore corefile" { > + check_fpmr_core_file $gcore_filename > + } > +} else { > + fail "gcore corefile not generated" > +} > + > +if {$core_generated} { > + clean_restart > + gdb_load ${binfile} > + with_test_prefix "native corefile" { > + check_fpmr_core_file $core_filename > + } > +} else { > + untested "native corefile not generated" > +} > diff --git a/gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.c b/gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.c > new file mode 100644 > index 00000000000..bbdfbb0597c > --- /dev/null > +++ b/gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.c > @@ -0,0 +1,55 @@ > +/* This file is part of GDB, the GNU debugger. > + > + Copyright 2008-2025 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 . */ > + > +#include > +#include > +#include > +#include > + > +void > +set_fpmr (uint64_t value) > +{ > + register uint64_t x0_val asm ("x0") = value; > + /* msr fpmr, x0 */ > + __asm__ volatile (".inst 0xd51b4440" : : ); > +} > + > +void > +handler (int sig) > +{ > + set_fpmr (0xff008041); > + exit(0); > +} > + > +int > +main () > +{ > + /* Ensure all the signals aren't blocked. */ Nit. Make it "Ensure no signals are blocked.". Makes it clearer. > + sigset_t newset; > + sigemptyset (&newset); > + sigprocmask (SIG_SETMASK, &newset, NULL); > + > + signal (SIGILL, handler); > + > + set_fpmr (0x3fff7fc049); > + > + /* 0x06000000 : Cause an illegal instruction. Value undefined as per ARM Formatting. Two spaces after period. And maybe s/Cause/Causes or s/Cause/Execute, depending on what you meant. > + Architecture Reference Manual ARMv8, Section C4.1. */ > + __asm __volatile (".inst 0x06000000"); > + > + return 0; > +} > diff --git a/gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.exp b/gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.exp > new file mode 100644 > index 00000000000..a776976eb7d > --- /dev/null > +++ b/gdb/testsuite/gdb.arch/aarch64-fpmr-sighandler.exp > @@ -0,0 +1,77 @@ > +# Copyright 2018-2025 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 FPMR register set is properly preserved when unwiding sighandler frames. > + > +require is_aarch64_target > +require allow_aarch64_fpmr_tests > + > +standard_testfile > +if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } { > + return -1 > +} > + > +if ![runto_main] { > + return -1 > +} > + > +set reg_main_value "0x3fff7fc049" > +set reg_handler_value "0xff008041" > + > +proc check_fpmr {value} { > + gdb_test "print /x \$fpmr" \ > + ".* = {?$value}?" \ > + "check register \$fpmr has value $value" > +} > + > +# Run until end of signal handler > + > +gdb_test "continue" \ > + "Continuing.*Program received signal SIGILL.*" \ > + "continue until signal" > + > +gdb_breakpoint [gdb_get_line_number "exit(0)"] > +gdb_continue_to_breakpoint "exit" ".*exit.*" > + > +set handlerframe [get_current_frame_number] > +set mainframe [expr $handlerframe + 2] > + > +# Check register values > + > +with_test_prefix "handler frame 1st" { > + check_fpmr $reg_handler_value > +} > + > +# Switch to the frame for main(), and check register values > + > +gdb_test "frame $mainframe" \ > + "#$mainframe.*main ().*" \ > + "set to main frame" > + > +with_test_prefix "main frame" { > + check_fpmr $reg_main_value > +} > + > +# Switch back to the signal handler frame, and check register values > + > +gdb_test "frame $handlerframe" \ > + "#$handlerframe.*handler \\\(sig=4\\\).*" \ > + "set to signal handler frame" > + > +with_test_prefix "handler frame 2nd" { > + check_fpmr $reg_handler_value > +} > diff --git a/gdb/testsuite/gdb.arch/aarch64-fpmr.c b/gdb/testsuite/gdb.arch/aarch64-fpmr.c > new file mode 100644 > index 00000000000..f4bf889c3a8 > --- /dev/null > +++ b/gdb/testsuite/gdb.arch/aarch64-fpmr.c > @@ -0,0 +1,117 @@ > +/* This file is part of GDB, the GNU debugger. > + > + Copyright 2008-2025 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 . */ > + > +#include > + > +enum FPM_FORMAT > +{ > + E5M2, > + E4M3, > +}; > + > +enum FPM_OVERFLOW > +{ > + INFNAN, > + SATURATE, > +}; > + > +void > +set_fpmr (uint64_t value) > +{ > + register uint64_t x0_val asm ("x0") = value; > + __asm__ volatile (".inst 0xd51b4440" : : ); Add the following here as well? /* msr fpmr, x0 */ > +} > + > +uint64_t > +modify_src1_fmt (uint64_t fpmr, uint64_t fmt) > +{ > + return (fpmr & ~(0x7)) | (fmt & 0x7); > +} > + > +uint64_t > +modify_src2_fmt (uint64_t fpmr, uint64_t fmt) > +{ > + return (fpmr & ~((0x7) << 3)) | ((fmt & 0x7) << 3); > +} > + > +uint64_t > +modify_dst_fmt (uint64_t fpmr, uint64_t fmt) > +{ > + return (fpmr & ~((0x7) << 6)) | ((fmt & 0x7) << 6); > +} > + > +uint64_t > +modify_osm (uint64_t fpmr, uint64_t overflow) > +{ > + return (fpmr & ~((0x1) << 14)) | ((overflow & 0x1) << 14); > +} > + > +uint64_t > +modify_osc (uint64_t fpmr, uint64_t overflow) > +{ > + return (fpmr & ~((0x1) << 15)) | ((overflow & 0x1) << 15); > +} > + > +uint64_t > +modify_lscale (uint64_t fpmr, uint64_t scale) > +{ > + return (fpmr & ~((0x7f) << 16)) | ((scale & 0x7f) << 16); > +} > + > +uint64_t > +modify_nscale (uint64_t fpmr, uint64_t scale) > +{ > + return (fpmr & ~((0xff) << 24)) | ((scale & 0xff) << 24); > +} > + > +uint64_t > +modify_lscale2 (uint64_t fpmr, uint64_t scale) > +{ > + return (fpmr & ~((uint64_t)(0x3f) << 32)) | ((uint64_t)(scale & 0x3f) << 32); > +} > + > +int > +main (void) > +{ > + uint64_t fpmr = 0; > + > + fpmr = modify_src1_fmt (fpmr, E4M3); > + set_fpmr (fpmr); /* MODIFY SRC1 */ > + > + fpmr = modify_src2_fmt (fpmr, E4M3); > + set_fpmr (fpmr); /* MODIFY SRC2 */ > + > + fpmr = modify_dst_fmt (fpmr, E4M3); > + set_fpmr (fpmr); /* MODIFY DST */ > + > + fpmr = modify_osm (fpmr, SATURATE); > + set_fpmr (fpmr); /* MODIFY OSM */ > + > + fpmr = modify_osc (fpmr, SATURATE); > + set_fpmr (fpmr); /* MODIFY OSC */ > + > + fpmr = modify_lscale (fpmr, -1); > + set_fpmr (fpmr); /* MODIFY LSCALE */ > + > + fpmr = modify_nscale (fpmr, -1); > + set_fpmr (fpmr); /* MODIFY NSCALE */ > + > + fpmr = modify_lscale2 (fpmr, -1); > + set_fpmr (fpmr); /* MODIFY LSCALE2 */ > + > + return 1; > +} > diff --git a/gdb/testsuite/gdb.arch/aarch64-fpmr.exp b/gdb/testsuite/gdb.arch/aarch64-fpmr.exp > new file mode 100644 > index 00000000000..947d47e981d > --- /dev/null > +++ b/gdb/testsuite/gdb.arch/aarch64-fpmr.exp > @@ -0,0 +1,72 @@ > +# Copyright 2023-2025 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 . */ > + > +# Exercise reading/writing FPMR when it is present. > + > +require is_aarch64_target > +require allow_aarch64_fpmr_tests > + > +standard_testfile > +if { [prepare_for_testing "failed to prepare" ${testfile} ${srcfile}] } { > + return -1 > +} > + > +if ![runto_main] { > + untested "could not run to main" As mentioned above, we should just drop this and return -1 here. > + return -1 > +} > + > +gdb_test_multiple "info register \$fpmr" "Test FPMR SRC1 E5M2" { > + -re ".*\r\n.*\[ F8S1=E5M2 F8S2=E5M2 F8D=E5M2 OSM=Inf OSC=Inf/NaN LSCALE=0 NSCALE=0 LSCALE2=0 \]" { > + pass "FPMR SRC1 matches E5M2" > + } > +} > + > +set breakpoints [list "MODIFY SRC1" "MODIFY SRC2" "MODIFY DST" "MODIFY OSM" "MODIFY OSC" "MODIFY LSCALE" "MODIFY NSCALE" "MODIFY LSCALE2"] Nit. For simplicity, you might want to drop the "MODIFY" from all of these and just use the field name, as all of them modify something. And then update the source file accordingly. > +set reg_values [list \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E5M2 F8D=E5M2 OSM=Inf OSC=Inf/NaN LSCALE=0 NSCALE=0 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E5M2 OSM=Inf OSC=Inf/NaN LSCALE=0 NSCALE=0 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E4M3 OSM=Inf OSC=Inf/NaN LSCALE=0 NSCALE=0 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E4M3 OSM=MaxNormal OSC=Inf/NaN LSCALE=0 NSCALE=0 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E4M3 OSM=MaxNormal OSC=MaxNormal LSCALE=0 NSCALE=0 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E4M3 OSM=MaxNormal OSC=MaxNormal LSCALE=127 NSCALE=0 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E4M3 OSM=MaxNormal OSC=MaxNormal LSCALE=127 NSCALE=255 LSCALE2=0 \]" \ > + ".*\r\n.*\[ F8S1=E4M3 F8S2=E4M3 F8D=E4M3 OSM=MaxNormal OSC=MaxNormal LSCALE=127 NSCALE=255 LSCALE2=63 \]" \ > +] > +set pass_messages [list "FPMR SRC1 matches E4M3" "FPMR SRC2 matches E4M3" "FPMR DST matches E4M3" "FPMR OSM matches MaxNormal" "FPMR OSC matches MaxNormal" "FPMR LSCALE matches" "FPMR NSCALE matches" "FPMR LSCALE2 matches"] > + > +for {set i 0} {$i < 8} {incr i} { > + set bp [lindex $breakpoints $i] > + gdb_breakpoint [gdb_get_line_number $bp] > + gdb_continue_to_breakpoint $bp > + > + gdb_test_multiple "info register \$fpmr" "" { > + -re [lindex $reg_values $i] { > + pass [lindex $pass_messages $i] > + } > + } > +} > + > +gdb_test_multiple "set \$fpmr=0x0" "" { > + -re ".*\r\n.*\[ F8S1=E5M2 F8S2=E5M2 F8D=E5M2 OSM=Inf OSC=Inf/NaN LSCALE=0 NSCALE=0 LSCALE2=0 \]" { > + pass "Reset FPMR to 0 from GDB" > + } > +} > + > +gdb_test_multiple "set \$fpmr=0x3f007f4008" "" { > + -re ".*\r\n.*\[ F8S1=E5M2 F8S2=E4M3 F8D=E5M2 OSM=MaxNormal OSC=Inf/NaN LSCALE=127 NSCALE=0 LSCALE2=63 \]" { > + pass "Write to FPMR from GDB" > + } > +} > diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp > index 930462f63fa..0d0369511bb 100644 > --- a/gdb/testsuite/lib/gdb.exp > +++ b/gdb/testsuite/lib/gdb.exp > @@ -5201,6 +5201,72 @@ proc aarch64_supports_sme_svl { length } { > return 1 > } > > +# Run a test on the target to see if it supports AArch64 FPMR hardware. Nit. s/AArch64 FPMR hardware/the AArch64 FPMR feature. > +# Return 1 if so, 0 if it does not. Note this causes a restart of GDB. > + > +gdb_caching_proc allow_aarch64_fpmr_tests {} { > + global srcdir subdir gdb_prompt inferior_exited_re > + > + set me "allow_aarch64_fpmr_tests" > + > + if { ![is_aarch64_target]} { > + return 0 > + } > + > + set compile_flags "{additional_flags=-march=armv8-a}" > + > + # Compile a test program that writes to FPMR. Here it says "writes"... > + set src { > + int main() { > + asm volatile ("msr x0, fpmr"); > + return 0; > + } > + } > + if {![gdb_simple_compile $me $src executable $compile_flags]} { > + # Try again, but with a raw hex instruction so we don't rely on > + # assembler support for FPMR. > + > + set compile_flags "{additional_flags=-march=armv8-a}" > + > + # Compile a test program reading FPMR. ... but here it says reading. Which one is it? > + set src { > + int main() { > + asm volatile (".word 0xD51B4440"); > + return 0; > + } > + } > + > + if {![gdb_simple_compile $me $src executable $compile_flags]} { > + return 0 > + } > + } > + > + # Compilation succeeded so now run it via gdb. > + clean_restart > + gdb_load $obj > + gdb_run_cmd > + > + gdb_expect { > + -re ".*Illegal instruction.*${gdb_prompt} $" { > + verbose -log "\n$me fpmr support not detected" > + set allow_fpmr_tests 0 > + } > + -re ".*$inferior_exited_re normally.*${gdb_prompt} $" { > + verbose -log "\n$me: fpmr support detected" > + set allow_fpmr_tests 1 > + } > + default { > + warning "\n$me: default case taken" > + set allow_fpmr_tests 0 > + } > + } > + gdb_exit > + remote_file build delete $obj > + > + verbose "$me: returning $allow_fpmr_tests" 2 > + return $allow_fpmr_tests > +} > + > # Run a test on the target to see if it supports AArch64 MOPS (Memory > # Operations) extensions. Return 1 if so, 0 if it does not. Note this > # causes a restart of GDB.