Re: [PATCH] Fix AIX SIGSEGV for multi thread debugging.

[Aditya Kamath <Aditya.Kamath1@ibm.com>]

[-- Attachment #1: Type: text/plain, Size: 1401 bytes --]

Thank you.

I have pushed the same.

https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=5c5998bb64bf567ec9194208522a5fde7e6e01cb

From: Ulrich Weigand <Ulrich.Weigand@de.ibm.com>
Date: Tuesday, 23 June 2026 at 5:27 PM
To: akamath996@gmail.com <akamath996@gmail.com>; tom@tromey.com <tom@tromey.com>; simon.marchi@polymtl.ca <simon.marchi@polymtl.ca>
Cc: gdb-patches@sourceware.org <gdb-patches@sourceware.org>; SANGAMESH MALLAYYA <sangamesh.swamy@in.ibm.com>; Aditya Kamath <Aditya.Kamath1@ibm.com>
Subject: Re: [PATCH] Fix AIX SIGSEGV for multi thread debugging.

Aditya Vidyadhar Kamath <akamath996@gmail.com> wrote:

>diff --git a/gdb/aix-thread.c b/gdb/aix-thread.c
>index 675d8d5070c..5ac71d22237 100644
>--- a/gdb/aix-thread.c
>+++ b/gdb/aix-thread.c
>@@ -974,7 +974,12 @@ pd_enable (inferior *inf)
>     = lookup_minimal_symbol (current_program_space, stub_name);
>   if (ms.minsym == NULL)
>     return;
>-  data->pd_brk_addr = ms.value_address ();
>+
>+  /* On AIX, symbols can be function descriptors, so we need to
>resolve
>+     them to get the actual code address.  */
>+  if (!msymbol_is_function (ms.objfile, ms.minsym, &data-
>>pd_brk_addr))
>+    return;
>+
>   if (!create_thread_event_breakpoint (current_inferior ()->arch (),
>                                      data->pd_brk_addr))
>     return;

This is OK.

Thanks,
Ulrich

[-- Attachment #2: Type: text/html, Size: 3634 bytes --]