From: Luis Machado <luis.machado@linaro.org>
To: "Peter Maydell" <peter.maydell@linaro.org>,
"Alex Bennée" <alex.bennee@linaro.org>
Cc: "open list:ARM TCG CPUs" <qemu-arm@nongnu.org>,
Claudio Fontana <cfontana@suse.de>,
QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers
Date: Fri, 5 Nov 2021 10:35:27 -0300 [thread overview]
Message-ID: <1b2ee804-6360-c77f-45f2-8fe39e39f9d9@linaro.org> (raw)
In-Reply-To: <e7c11bc8-e7ba-8192-e4ba-887c38f27b15@linaro.org>
On 11/4/21 6:03 PM, Luis Machado wrote:
> On 10/4/21 3:44 PM, Luis Machado wrote:
>> Hi,
>>
>> On 9/21/21 10:55 AM, Peter Maydell wrote:
>>> On Tue, 19 Jan 2021 at 15:57, Alex Bennée <alex.bennee@linaro.org>
>>> wrote:
>>>>
>>>>
>>>> Claudio Fontana <cfontana@suse.de> writes:
>>>>
>>>>> On 1/19/21 3:50 PM, Alex Bennée wrote:
>>>>>>
>>>>>> Claudio Fontana <cfontana@suse.de> writes:
>>>>>>> qemu-system-aarch64: -gdb
>>>>>>> unix:path=/tmp/tmp9ru5tgk8qemu-gdbstub/gdbstub.socket,server:
>>>>>>> info: QEMU waiting for connection on:
>>>>>>> disconnected:unix:/tmp/tmp9ru5tgk8qemu-gdbstub/gdbstub.socket,server
>>>>>>> warning: while parsing target description (at line 47): Vector
>>>>>>> "svevhf" references undefined type "ieee_half"
>>>>>>> warning: Could not load XML target description; ignoring
>>>>>>> qemu-system-aarch64: QEMU: Terminated via GDBstub
>>>>>>>
>>>>>>> Seems to indicate it is "ieee_half" -related?
>>>
>>>> So it looks like TDESC_TYPE_IEEE_HALF was only implemented in GDB 9.1
>>>> and there is no probing possible during the gdbstub connection. I guess
>>>> I can either go back to stubbing it out (which would break gdb's SVE
>>>> understanding) or up our minimum GDB version check for running tests.
>>>> That would mean less people test GDB (or at least until the distros
>>>> catch up) but considering it was zero people not too long ago maybe
>>>> that's acceptable?
>>>
>>> I just ran into this trying to connect qemu-aarch64 to the
>>> Ubuntu gdb-multiarch. I don't care about SVE at all in this
>>> case, but the 'max' CPU includes SVE by default, so we report
>>> it to gdb even if the guest program being run doesn't use SVE at all.
>>> This effectively means that usecases that used to work no longer do :-(
>>>
>>> Luis: do we really have to report to gdb all the possible
>>> data types that might be in SVE vector registers? Won't
>>> gdb autogenerate pseudoregisters the way it does with
>>> Neon d0..d31 ?
>>>
>>> thanks
>>> -- PMM
>>>
>>
>> I'll check what can be done here.
>
> Apologies. I got sidetracked with a few other things. I'm getting back
> to this one.
>
> I'm guessing the less painful solution would be for QEMU to report a
> more compact XML description for SVE, supported by the oldest GDB we
> would like to validate things on, and leave it to GDB (newer releases)
> to add whatever pseudo-registers it deems appropriate.
Is it only the ieee_half type that is causing issues here? Or are there
other types?
next prev parent reply other threads:[~2021-11-05 13:44 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-15 13:07 [PULL 00/30] testing, gdbstub and semihosting Alex Bennée
2021-01-15 13:07 ` [PULL 01/30] tests/docker: Remove Debian 9 remnant lines Alex Bennée
2021-01-15 13:08 ` [PULL 02/30] Makefile: add GNU global tags support Alex Bennée
2021-01-15 13:08 ` [PULL 03/30] shippable.yml: Remove jobs duplicated on Gitlab-CI Alex Bennée
2021-01-15 13:08 ` [PULL 04/30] Add newline when generating Dockerfile Alex Bennée
2021-01-15 13:08 ` [PULL 05/30] Makefile: wrap ctags in quiet-command calls Alex Bennée
2021-01-18 18:36 ` Philippe Mathieu-Daudé
2021-01-19 10:00 ` Alex Bennée
2021-01-19 14:24 ` Philippe Mathieu-Daudé
2021-01-19 14:27 ` Daniel P. Berrangé
2021-01-19 14:42 ` Philippe Mathieu-Daudé
2021-01-15 13:08 ` [PULL 06/30] Makefile: wrap etags " Alex Bennée
2021-01-15 13:08 ` [PULL 07/30] Makefile: wrap cscope " Alex Bennée
2021-01-15 13:08 ` [PULL 08/30] docker: expand debian-amd64 image to include tag tools Alex Bennée
2021-01-15 13:08 ` [PULL 09/30] gitlab: move docs and tools build across from Travis Alex Bennée
2021-01-15 13:08 ` [PULL 10/30] Fix build with new yank feature by adding stubs Alex Bennée
2021-01-15 13:08 ` [PULL 11/30] gitlab: migrate the minimal tools and unit tests from Travis Alex Bennée
2021-01-15 13:08 ` [PULL 12/30] scripts/checkpatch.pl: fix git-show invocation to include diffstat Alex Bennée
2021-01-15 13:08 ` [PULL 13/30] test/guest-debug: echo QEMU command as well Alex Bennée
2021-01-15 13:08 ` [PULL 14/30] configure: gate our use of GDB to 8.3.1 or above Alex Bennée
2021-01-15 13:08 ` [PULL 15/30] Revert "tests/tcg/multiarch/Makefile.target: Disable run-gdbstub-sha1 test" Alex Bennée
2021-01-15 13:08 ` [PULL 16/30] gdbstub: implement a softmmu based test Alex Bennée
2021-01-15 13:08 ` [PULL 17/30] gdbstub: add support to Xfer:auxv:read: packet Alex Bennée
2021-01-15 13:08 ` [PULL 18/30] gdbstub: drop CPUEnv from gdb_exit() Alex Bennée
2021-01-15 13:08 ` [PULL 19/30] gdbstub: drop gdbserver_cleanup in favour of gdb_exit Alex Bennée
2021-01-15 13:08 ` [PULL 20/30] gdbstub: ensure we clean-up when terminated Alex Bennée
2021-01-15 13:08 ` [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers Alex Bennée
2021-01-19 13:38 ` Claudio Fontana
2021-01-19 13:49 ` Claudio Fontana
2021-01-19 14:50 ` Alex Bennée
2021-01-19 15:11 ` Claudio Fontana
2021-01-19 15:54 ` Alex Bennée
2021-01-19 16:19 ` Luis Machado
2021-09-21 13:55 ` Peter Maydell
2021-10-04 18:44 ` Luis Machado
2021-11-04 21:03 ` Luis Machado
2021-11-05 13:35 ` Luis Machado [this message]
2021-11-05 16:15 ` Alex Bennée
2021-11-05 16:29 ` Luis Machado
2021-01-15 13:08 ` [PULL 22/30] semihosting: Move ARM semihosting code to shared directories Alex Bennée
2021-01-15 13:08 ` [PULL 23/30] semihosting: Change common-semi API to be architecture-independent Alex Bennée
2021-01-15 13:08 ` [PULL 24/30] semihosting: Change internal common-semi interfaces to use CPUState * Alex Bennée
2021-02-17 15:02 ` Peter Maydell
2021-01-15 13:08 ` [PULL 25/30] semihosting: Support SYS_HEAPINFO when env->boot_info is not set Alex Bennée
2021-01-15 13:08 ` [PULL 26/30] riscv: Add semihosting support Alex Bennée
2021-01-15 13:08 ` [PULL 27/30] riscv: Add semihosting support for user mode Alex Bennée
2021-01-15 13:08 ` [PULL 28/30] semihosting: Implement SYS_ELAPSED and SYS_TICKFREQ Alex Bennée
2021-01-15 13:08 ` [PULL 29/30] semihosting: Implement SYS_TMPNAM Alex Bennée
2021-01-15 13:08 ` [PULL 30/30] semihosting: Implement SYS_ISERROR Alex Bennée
2021-01-15 15:31 ` [PULL 00/30] testing, gdbstub and semihosting Peter Maydell
2021-01-18 12:18 ` Alex Bennée
2021-01-18 13:33 ` Philippe Mathieu-Daudé
2021-01-18 15:38 ` Alex Bennée
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1b2ee804-6360-c77f-45f2-8fe39e39f9d9@linaro.org \
--to=luis.machado@linaro.org \
--cc=alex.bennee@linaro.org \
--cc=cfontana@suse.de \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).