From: Eli Zaretskii <eliz@gnu.org>
To: Hannes Domani <ssbssa@yahoo.de>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH v2 11/11] Add aarch64-windows support
Date: Sat, 17 Jan 2026 18:04:33 +0200 [thread overview]
Message-ID: <86fr84ql7y.fsf@gnu.org> (raw)
In-Reply-To: <20260117135448.2666604-11-ssbssa@yahoo.de> (message from Hannes Domani on Sat, 17 Jan 2026 14:54:34 +0100)
> From: Hannes Domani <ssbssa@yahoo.de>
> Date: Sat, 17 Jan 2026 14:54:34 +0100
>
> This makes most debugging work, except unwinding doesn't always work
> from inside dlls where no debug info is available, because SEH unwinding
> is not implemented.
>
> The number of available hardware breakpoints is taken from
> ID_AA64DFR0_EL1 (registry key "CP 4028").
> As for hardware watchpoints, even though ARM64_MAX_WATCHPOINTS is 2,
> testing showed that only 1 ever works, so it's fixed to that value.
> ---
> Changes in v2:
> - added new source files to ALLDEPFILES
> - created NEWS entry
> - removed set_gdbarch_auto_wide_charset and set_gdbarch_long_bit calls
> since these are now done in windows-tdep.c
> - added comments to aarch64_windows_breakpoint and
> set_gdbarch_decr_pc_after_break
> - removed unused 32bit code in
> aarch64_windows_nat_target::fetch_one_register()
> - changed stopped_data_address() to stopped_data_addresses() in
> aarch64_windows_nat_target, and updated stopped_by_watchpoint()
> - initialize windows_process inside INIT_GDB_FILE(aarch64_windows_nat)
> ---
> gdb/Makefile.in | 3 +
> gdb/NEWS | 2 +
> gdb/aarch64-windows-nat.c | 346 +++++++++++++++++++++++++++++++++++++
> gdb/aarch64-windows-tdep.c | 70 ++++++++
> gdb/coff-pe-read.c | 6 +-
> gdb/configure.host | 3 +
> gdb/configure.nat | 4 +
> gdb/configure.tgt | 5 +
> gdb/nat/windows-nat.c | 5 +
> gdb/nat/windows-nat.h | 2 +
> 10 files changed, 444 insertions(+), 2 deletions(-)
> create mode 100644 gdb/aarch64-windows-nat.c
> create mode 100644 gdb/aarch64-windows-tdep.c
Thanks, the NEWS part is okay.
Reviewed-By: Eli Zaretskii <eliz@gnu.org>
next prev parent reply other threads:[~2026-01-17 16:05 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260117134052.2660009-1-ssbssa.ref@yahoo.de>
2026-01-17 13:36 ` [PATCH v2 01/11] Remove duplicate code from windows_nat_target::resume Hannes Domani
2026-01-17 13:36 ` [PATCH v2 02/11] Simplify windows_nat_target::resume Hannes Domani
2026-01-19 13:55 ` Schimpe, Christina
2026-01-21 15:42 ` Tom Tromey
2026-01-23 19:11 ` Hannes Domani
2026-01-17 13:36 ` [PATCH v2 03/11] Move struct declarations into windows-nat.h Hannes Domani
2026-01-17 13:36 ` [PATCH v2 04/11] Create x86-windows-nat.c Hannes Domani
2026-01-21 15:38 ` Tom Tromey
2026-01-17 13:36 ` [PATCH v2 05/11] Move x86 debug registers and related code into x86-windows-nat.c Hannes Domani
2026-01-19 12:49 ` Schimpe, Christina
2026-01-20 15:49 ` Hannes Domani
2026-01-21 8:06 ` Schimpe, Christina
2026-01-23 13:17 ` [PATCH v3] " Hannes Domani
2026-01-23 18:50 ` Tom Tromey
2026-01-23 19:20 ` Hannes Domani
2026-01-23 19:56 ` Tom Tromey
2026-01-17 13:36 ` [PATCH v2 06/11] Move x86 register " Hannes Domani
2026-01-19 12:50 ` Schimpe, Christina
2026-01-17 13:36 ` [PATCH v2 07/11] Move x86 selector " Hannes Domani
2026-01-19 13:56 ` Schimpe, Christina
2026-01-17 13:36 ` [PATCH v2 08/11] Move software breakpoint recognition " Hannes Domani
2026-01-19 12:51 ` Schimpe, Christina
2026-01-21 15:43 ` Tom Tromey
2026-01-23 19:12 ` Hannes Domani
2026-01-17 13:36 ` [PATCH v2 09/11] Move auto_wide_charset gdbarch method to windows-tdep Hannes Domani
2026-01-17 13:36 ` [PATCH v2 10/11] Move setting size of long " Hannes Domani
2026-01-21 15:47 ` Tom Tromey
2026-01-17 13:54 ` [PATCH v2 04/11] Create x86-windows-nat.c Hannes Domani
2026-01-17 14:01 ` Hannes Domani
2026-01-17 14:14 ` Hannes Domani
2026-01-17 15:04 ` Simon Marchi
2026-01-17 15:15 ` Hannes Domani
2026-01-17 13:54 ` [PATCH v2 05/11] Move x86 debug registers and related code into x86-windows-nat.c Hannes Domani
2026-01-17 13:54 ` [PATCH v2 06/11] Move x86 register " Hannes Domani
2026-01-21 16:08 ` Tom Tromey
2026-01-17 13:54 ` [PATCH v2 09/11] Move auto_wide_charset gdbarch method to windows-tdep Hannes Domani
2026-01-21 15:48 ` Tom Tromey
2026-01-23 19:13 ` Hannes Domani
2026-01-17 13:54 ` [PATCH v2 10/11] Move setting size of long " Hannes Domani
2026-01-21 15:59 ` Tom Tromey
2026-01-17 13:54 ` [PATCH v2 11/11] Add aarch64-windows support Hannes Domani
2026-01-17 16:04 ` Eli Zaretskii [this message]
2026-01-21 16:18 ` Tom Tromey
2026-01-23 19:14 ` Hannes Domani
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=86fr84ql7y.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=ssbssa@yahoo.de \
/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