mbox series

[v3,0/3] Fix attaching to process when it has zombie threads

Message ID 20240424231551.1577518-1-thiago.bauermann@linaro.org
Headers
Series Fix attaching to process when it has zombie threads |

Message

Thiago Jung Bauermann April 24, 2024, 11:15 p.m. UTC 
  Hello,

This version just adjusts the comment in patch 1 as suggested by Pedro.
This reflects in patch 2, which has a slightly abbreviated version of the
comment.

v2 of this series is here:

https://inbox.sourceware.org/gdb-patches/20240420055652.819024-1-thiago.bauermann@linaro.org/

v1 of this series is here:

https://inbox.sourceware.org/gdb-patches/20240321231149.519549-1-thiago.bauermann@linaro.org/


Thiago Jung Bauermann (3):
  gdb/nat: Use procfs(5) indexes in linux_common_core_of_thread
  gdb/nat: Factor linux_proc_get_stat_field out of
    linux_common_core_of_thread
  gdb/nat/linux: Fix attaching to process when it has zombie threads

 gdb/nat/linux-osdata.c | 38 +++--------------
 gdb/nat/linux-osdata.h |  3 ++
 gdb/nat/linux-procfs.c | 97 ++++++++++++++++++++++++++++++++++++++++++
 gdb/nat/linux-procfs.h | 11 +++++
 4 files changed, 116 insertions(+), 33 deletions(-)


base-commit: f5ef12c3f1af890dcdc6dc54f5ea58a05c6cf65a

Comments

Pedro Alves April 26, 2024, 4:23 p.m. UTC | #1 
On 2024-04-25 00:15, Thiago Jung Bauermann wrote:
> Hello,
> 
> This version just adjusts the comment in patch 1 as suggested by Pedro.
> This reflects in patch 2, which has a slightly abbreviated version of the
> comment.
> 
> v2 of this series is here:
> 
> https://inbox.sourceware.org/gdb-patches/20240420055652.819024-1-thiago.bauermann@linaro.org/
> 
> v1 of this series is here:
> 
> https://inbox.sourceware.org/gdb-patches/20240321231149.519549-1-thiago.bauermann@linaro.org/
> 
> 
> Thiago Jung Bauermann (3):
>   gdb/nat: Use procfs(5) indexes in linux_common_core_of_thread
>   gdb/nat: Factor linux_proc_get_stat_field out of
>     linux_common_core_of_thread
>   gdb/nat/linux: Fix attaching to process when it has zombie threads
> 
>  gdb/nat/linux-osdata.c | 38 +++--------------
>  gdb/nat/linux-osdata.h |  3 ++
>  gdb/nat/linux-procfs.c | 97 ++++++++++++++++++++++++++++++++++++++++++
>  gdb/nat/linux-procfs.h | 11 +++++
>  4 files changed, 116 insertions(+), 33 deletions(-)
> 

For series:

  Approved-By: Pedro Alves <pedro@palves.net>
Thiago Jung Bauermann April 30, 2024, 2:39 a.m. UTC | #2 
Pedro Alves <pedro@palves.net> writes:

> On 2024-04-25 00:15, Thiago Jung Bauermann wrote:
>> Hello,
>> 
>> This version just adjusts the comment in patch 1 as suggested by Pedro.
>> This reflects in patch 2, which has a slightly abbreviated version of the
>> comment.
>> 
>> v2 of this series is here:
>> 
>> https://inbox.sourceware.org/gdb-patches/20240420055652.819024-1-thiago.bauermann@linaro.org/
>> 
>> v1 of this series is here:
>> 
>> https://inbox.sourceware.org/gdb-patches/20240321231149.519549-1-thiago.bauermann@linaro.org/
>> 
>> 
>> Thiago Jung Bauermann (3):
>>   gdb/nat: Use procfs(5) indexes in linux_common_core_of_thread
>>   gdb/nat: Factor linux_proc_get_stat_field out of
>>     linux_common_core_of_thread
>>   gdb/nat/linux: Fix attaching to process when it has zombie threads
>> 
>>  gdb/nat/linux-osdata.c | 38 +++--------------
>>  gdb/nat/linux-osdata.h |  3 ++
>>  gdb/nat/linux-procfs.c | 97 ++++++++++++++++++++++++++++++++++++++++++
>>  gdb/nat/linux-procfs.h | 11 +++++
>>  4 files changed, 116 insertions(+), 33 deletions(-)
>> 
>
> For series:
>
>   Approved-By: Pedro Alves <pedro@palves.net>

Thank you! Pushed as commits 3de4256ca3e8, 16a447bec542 and
c930a077225e.