pauth: Create new feature string for pauth to prevent crashing older gdb's
authorLuis Machado <luis.machado@arm.com>
Mon, 3 Apr 2023 09:43:34 +0000 (10:43 +0100)
committerLuis Machado <luis.machado@arm.com>
Fri, 14 Apr 2023 12:44:11 +0000 (13:44 +0100)
commitacdf60711d44d20608873bec0376688c9a80e281
treef04f326c4e234c02c15d14d9cf489b00acb9a6ec
parente10d82fc3ed03ee03f7c831a5b1e73c6c5ed3722
pauth: Create new feature string for pauth to prevent crashing older gdb's

Older gdb's (9, 10, 11 and 12) have a bug that causes them to crash whenever
a target reports the pauth feature string in the target description and also
provide additional register outside of gdb's known and expected feature
strings.

This was fixed in gdb 13 onwards, but that means we're stuck with gdb's out
there that will crash on connection to the above targets.

QEMU has postponed inclusion of the pauth feature string in version 8, and
instead we agreed to use a new feature name to prevent crashing those older
gdb's.

Initially there was a plan to backport a trivial fix all the way to gdb 9, but
given QEMU's choice, this is no longer needed.

This new feature string is org.gnu.gdb.aarch64.pauth_v2, and should be used
by all targets going forward, except native linux gdb and gdbserver, for
backwards compatibility with older gdb's/gdbserver's.

gdb/gdbserver will still emit the old feature string for Linux since it doesn't
report additional system registers and thus doesn't cause a crash of older
gdb's. We can revisit this in the future once the problematic gdb's are likely
no longer in use.

I've added some documentation to explain the situation.
gdb/NEWS
gdb/aarch64-tdep.c
gdb/doc/gdb.texinfo