Re: [PULL 06/23] tests/tcg: add an explicit gdbstub register tester

2023-11-16 Thread Luis Machado
On 11/15/23 20:56, Alex Bennée via Gdb wrote: > "Nicholas Piggin" writes: > >> On Wed Nov 8, 2023 at 12:23 AM AEST, Alex Bennée wrote: >>> We already do a couple of "info registers" for specific tests but this >>> is a more comprehensive multiarch test. It also has some output >>> helpful for debu

Re: [PATCH 3/4] target/arm: Support reading ZA[] from gdbstub

2023-06-27 Thread Luis Machado
On 6/27/23 14:07, Peter Maydell wrote: > On Thu, 22 Jun 2023 at 16:12, Richard Henderson > wrote: >> >> Mirror the existing support for SVE. >> >> Signed-off-by: Richard Henderson > > >> @@ -247,6 +247,61 @@ int aarch64_gdb_set_pauth_reg(CPUARMState *env, uint8_t >> *buf, int reg) >> return

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-17 Thread Luis Machado
On 3/17/23 17:12, Luis Machado wrote: On 3/17/23 17:07, Peter Maydell wrote: On Fri, 17 Mar 2023 at 16:55, Luis Machado wrote: On 3/17/23 16:37, Peter Maydell wrote: Having run into this problem in another couple of situations, one of which involved gdb 10, I think I'm increasingly favo

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-17 Thread Luis Machado
On 3/17/23 17:07, Peter Maydell wrote: On Fri, 17 Mar 2023 at 16:55, Luis Machado wrote: On 3/17/23 16:37, Peter Maydell wrote: Having run into this problem in another couple of situations, one of which involved gdb 10, I think I'm increasingly favouring option 2 here. The affected gdbs

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-17 Thread Luis Machado
On 3/17/23 16:37, Peter Maydell wrote: On Wed, 15 Mar 2023 at 09:51, Luis Machado wrote: On 3/13/23 11:44, Luis Machado wrote: On 3/13/23 11:22, Peter Maydell via Gdb wrote: Luis and I came up with two options: (1) leave QEMU outputting the pauth xml as-is, and tell people whose gdb 12

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-15 Thread Luis Machado
Hi, On 3/13/23 11:44, Luis Machado wrote: On 3/13/23 11:22, Peter Maydell via Gdb wrote: On Fri, 10 Mar 2023 at 18:20, Alex Bennée wrote: (adding some more gdb types to CC) Fabiano Rosas writes: Peter Maydell writes: On Fri, 10 Mar 2023 at 10:31, Alex Bennée wrote: You need a

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-14 Thread Luis Machado
On 3/13/23 19:21, Richard Henderson wrote: On 3/13/23 04:44, Luis Machado wrote: Luis: I think that rather than doing (2) with a QEMU namespace, we should define a gdb namespace for this. That makes it clear that this is still a gdb-upstream-sanctioned way of exposing the pauth registers

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-13 Thread Luis Machado
On 3/13/23 11:22, Peter Maydell via Gdb wrote: On Fri, 10 Mar 2023 at 18:20, Alex Bennée wrote: (adding some more gdb types to CC) Fabiano Rosas writes: Peter Maydell writes: On Fri, 10 Mar 2023 at 10:31, Alex Bennée wrote: You need a very new gdb to be able to run with pauth suppor

Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests

2023-03-13 Thread Luis Machado
On 3/10/23 18:07, Peter Maydell wrote: On Fri, 10 Mar 2023 at 18:00, Fabiano Rosas wrote: Peter Maydell writes: On Fri, 10 Mar 2023 at 10:31, Alex Bennée wrote: You need a very new gdb to be able to run with pauth support otherwise your likely to hit asserts and aborts. Disable pauth for

Re: [PATCH v2 11/14] target/arm: Implement gdbstub pauth extension

2023-02-21 Thread Luis Machado
with Luis Machado, add two more masks in order to support pointers within the high half of the address space (i.e. TTBR1 vs TTBR0). Cc: Luis Machado Cc: Thiago Jung Bauermann Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1105 Signed-off-by: Richard Henderson --- configs/targets

Re: [PATCH 14/14] target/arm: Support reading m-profile system registers from gdb

2023-02-20 Thread Luis Machado
Hi, On 2/20/23 17:37, Peter Maydell wrote: On Mon, 20 Feb 2023 at 17:00, Richard Henderson wrote: On 2/20/23 06:02, Peter Maydell wrote: +g_string_printf(s, ""); +g_string_append_printf(s, ""); +g_string_append_printf(s, "\n"); Half of these need to be in org.gnu.gdb.arm.secext

Re: [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-11-05 Thread Luis Machado
On 11/5/21 1:15 PM, Alex Bennée wrote: Luis Machado writes: 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 wrote: Claudio Fontana writes: On 1/19/21 3:50 PM

Re: [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-11-05 Thread Luis Machado
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 wrote: Claudio Fontana writes: On 1/19/21 3:50 PM, Alex Bennée wrote: Claudio Fontana writes: qemu-system-aarch64

Re: [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-11-04 Thread Luis Machado
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 wrote: Claudio Fontana writes: On 1/19/21 3:50 PM, Alex Bennée wrote: Claudio Fontana writes: qemu-system-aarch64: -gdb unix:path=/tmp/tmp9ru5tgk8qemu

Re: Deprecate the ppc405 boards in QEMU? (was: [PATCH v3 4/7] MAINTAINERS: Orphan obscure ppc platforms)

2021-10-28 Thread Luis Machado
On 10/20/21 10:16 AM, LEROY Christophe wrote: Le 20/10/2021 à 14:43, Cédric Le Goater a écrit : On 10/20/21 13:42, BALATON Zoltan wrote: On Wed, 20 Oct 2021, Philippe Mathieu-Daudé wrote: On 10/5/21 14:29, Thomas Huth wrote: On 05/10/2021 14.20, BALATON Zoltan wrote: On Tue, 5 Oct 2021,

Re: [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-10-04 Thread Luis Machado
Hi, On 9/21/21 10:55 AM, Peter Maydell wrote: On Tue, 19 Jan 2021 at 15:57, Alex Bennée wrote: Claudio Fontana writes: On 1/19/21 3:50 PM, Alex Bennée wrote: Claudio Fontana writes: qemu-system-aarch64: -gdb unix:path=/tmp/tmp9ru5tgk8qemu-gdbstub/gdbstub.socket,server: info: QEMU wa

Re: Best approach for supporting snapshots for QEMU's gdbstub?

2021-05-17 Thread Luis Machado
On 5/17/21 2:27 PM, Alex Bennée wrote: Luis Machado writes: Hi, On 5/14/21 1:06 PM, Alex Bennée wrote: Hi, I've been playing around with QEMU's reverse debugging support which I have working with Pavel's latest patches for supporting virtio with record/replay. Once yo

Re: Best approach for supporting snapshots for QEMU's gdbstub?

2021-05-17 Thread Luis Machado
Hi, On 5/14/21 1:06 PM, Alex Bennée wrote: Hi, I've been playing around with QEMU's reverse debugging support which I have working with Pavel's latest patches for supporting virtio with record/replay. Once you get the right command line it works well enough although currently each step backward

Re: [PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-01-19 Thread Luis Machado
> > > >> > >>> > >>> Thanks, > >>> > >>> Claudio > >>> > >>> On 1/15/21 2:08 PM, Alex Bennée wrote: > >>>> While GDB can work with any XML description given to it there is > >>>> special

Re: [PATCH v1 10/20] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-01-11 Thread Luis Machado
Hi, On 1/11/21 11:36 AM, Alex Bennée wrote: Luis Machado writes: For the record, the layout looks OK to me. So a Reviewed-by? Yes. Just a reminder that GDB will soon support bfloat16 types. A patch may be pushed this month. Will we be able to probe for the support - or will an

Re: [PATCH v1 10/20] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers

2021-01-11 Thread Luis Machado
nable ieee_half (as we know gdb supports it now) - $vg is now a 64 bit int - check $vN and $zN aliasing in test Signed-off-by: Alex Bennée Cc: Luis Machado Message-Id: <20201218112707.28348-10-alex.ben...@linaro.org> Signed-off-by: Alex Bennée --- target/ar

Re: [PATCH v3 12/20] target/arm: generate xml description of our SVE registers

2019-12-20 Thread Luis Machado
On 12/20/19 10:14 AM, Alex Bennée wrote: Luis Machado writes: On 12/19/19 4:15 PM, Alex Bennée wrote: Richard Henderson writes: On 12/11/19 9:05 AM, Alex Bennée wrote: +static struct TypeSize vec_lanes[] = { const. +case 51: +return gdb_get_reg64(buf, (cpu

Re: [PATCH v3 12/20] target/arm: generate xml description of our SVE registers

2019-12-20 Thread Luis Machado
On 12/19/19 4:15 PM, Alex Bennée wrote: Richard Henderson writes: On 12/11/19 9:05 AM, Alex Bennée wrote: +static struct TypeSize vec_lanes[] = { const. +case 51: +return gdb_get_reg64(buf, (cpu->env.vfp.zcr_el[1] & 0xf) + 1); You need to use sve_zcr_len_for_el to get th