[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests
From: |
Peter Maydell |
Subject: |
Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests |
Date: |
Fri, 10 Mar 2023 18:07:36 +0000 |
On Fri, 10 Mar 2023 at 18:00, Fabiano Rosas <farosas@suse.de> wrote:
>
> Peter Maydell <peter.maydell@linaro.org> writes:
>
> > On Fri, 10 Mar 2023 at 10:31, Alex Bennée <alex.bennee@linaro.org> 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 now until we
> >> can properly probe support in gdb.
> >>
> >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> >
> > If it makes gdb fall over, then shouldn't we be disabling
> > the pauth gdbstub stuff entirely ? Otherwise even if our
> > tests are fine our users will not be...
> >
>
> Have you seem my message on IRC about changing the feature name in the
> XML? I think the issue is that we're putting the .xml in a "namespace"
> where GDB expects to only find stuff which it has code to
> support. Changing from "org.gnu.gdb.aarch64.pauth" to
> "org.qemu.aarch64.pauth" made it stop crashing and I can read the
> registers just fine.
But then presumably a pauth-aware GDB won't actually know
the values it needs to be able to convert between with-PAC
and without-PAC addresses for backtracing?
Luis, how is this intended to work? Is there some way the
stub can check with the gdb that's connected whether the
gdb is able to cope with the pauth XML, so it can avoid
sending it to a gdb that is going to crash if it sees it ?
thanks
-- PMM
- [PATCH 00/11] tweaks and fixes for 8.0-rc1 (tests, plugins, docs), Alex Bennée, 2023/03/10
- [PATCH 02/11] tests/docker: all add DOCKER_BUILDKIT to RUNC environment, Alex Bennée, 2023/03/10
- [PATCH 01/11] tests/avocado: update AArch64 tests to Alpine 3.17.2, Alex Bennée, 2023/03/10
- [PATCH 03/11] scripts/ci: add libslirp-devel to build-environment, Alex Bennée, 2023/03/10
- [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Alex Bennée, 2023/03/10
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Richard Henderson, 2023/03/10
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Peter Maydell, 2023/03/10
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Alex Bennée, 2023/03/10
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Peter Maydell, 2023/03/13
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Luis Machado, 2023/03/13
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Richard Henderson, 2023/03/13
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Peter Maydell, 2023/03/13
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Luis Machado, 2023/03/14
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Luis Machado, 2023/03/15
- Re: [PATCH 09/11] tests/tcg: disable pauth for aarch64 gdb tests, Peter Maydell, 2023/03/17