[LTP] [PATCH 1/1] README: Document gdb debugging

[LTP] [PATCH 1/1] README: Document gdb debugging

[Petr Vorel]

From: Petr Vorel <petr.vorel@gmail.com>

Suggested-by: Cyril Hrubis <chrubis@suse.cz>
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
 README.md | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/README.md b/README.md
index fa9559398..adeac5fdf 100644
--- a/README.md
+++ b/README.md
@@ -215,6 +215,12 @@ SUSE also publishes a
 [smaller LTP container](https://registry.opensuse.org/cgi-bin/cooverview?srch_term=project%3D%5Ebenchmark+container%3D.*)
 that is not based on the Containerfile.
 
+Debugging with gdb
+==================
+
+To get stack trace of a crashing test in gdb it's useful to
+[`set follow-fork-mode child`](https://ftp.gnu.org/old-gnu/Manuals/gdb/html_node/gdb_25.html).
+
 Developers corner
 =================
 
-- 
2.42.0


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] README: Document gdb debugging

[Cyril Hrubis]

On Thu, Oct 12, 2023 at 11:42:54AM +0200, Petr Vorel wrote:
> From: Petr Vorel <petr.vorel@gmail.com>
> 
> Suggested-by: Cyril Hrubis <chrubis@suse.cz>
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
>  README.md | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/README.md b/README.md
> index fa9559398..adeac5fdf 100644
> --- a/README.md
> +++ b/README.md
> @@ -215,6 +215,12 @@ SUSE also publishes a
>  [smaller LTP container](https://registry.opensuse.org/cgi-bin/cooverview?srch_term=project%3D%5Ebenchmark+container%3D.*)
>  that is not based on the Containerfile.
>  
> +Debugging with gdb
> +==================
> +
> +To get stack trace of a crashing test in gdb it's useful to
> +[`set follow-fork-mode child`](https://ftp.gnu.org/old-gnu/Manuals/gdb/html_node/gdb_25.html).

Maybe we should start with a sentence that actually describes how things
are organized so that it's clear why this is needed. Maybe just:

The new test library runs the actual test, i.e. the `test()` function in
a forked process. To get the stack trace of a crashing test it's needed
to [`set ....


>  Developers corner
>  =================
>  
> -- 
> 2.42.0
> 

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] README: Document gdb debugging

[Petr Vorel]

Hi Cyril,

...
> > +Debugging with gdb
> > +==================
> > +
> > +To get stack trace of a crashing test in gdb it's useful to
> > +[`set follow-fork-mode child`](https://ftp.gnu.org/old-gnu/Manuals/gdb/html_node/gdb_25.html).

> Maybe we should start with a sentence that actually describes how things
> are organized so that it's clear why this is needed. Maybe just:

> The new test library runs the actual test, i.e. the `test()` function in
> a forked process. To get the stack trace of a crashing test it's needed
> to [`set ....

+1, thank you.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] README: Document gdb debugging

[Cyril Hrubis]

Hi!
Ping?

Feel free to push with my Reviewed-by: with the changes I've proposed.

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] README: Document gdb debugging

[Petr Vorel]

> Hi!
> Ping?

> Feel free to push with my Reviewed-by: with the changes I've proposed.

Thanks for pinging me, merged with your changes.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp