Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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>

  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