2023-01-12 16:26:58

by Jakob Koschel

[permalink] [raw]
Subject: [PATCH] docs/scripts/gdb: add necessary make scripts_gdb step

In order to debug the kernel successfully with gdb you need to run
'make scripts_gdb' nowadays.

This was changed with the following commit:

Commit 67274c083438340ad16c ("scripts/gdb: delay generation of gdb
constants.py")

In order to have a complete guide for beginners this remark
should be added to the offial documentation.

---


Signed-off-by: Jakob Koschel <[email protected]>
---
Documentation/dev-tools/gdb-kernel-debugging.rst | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/Documentation/dev-tools/gdb-kernel-debugging.rst b/Documentation/dev-tools/gdb-kernel-debugging.rst
index 8e0f1fe8d17a..895285c037c7 100644
--- a/Documentation/dev-tools/gdb-kernel-debugging.rst
+++ b/Documentation/dev-tools/gdb-kernel-debugging.rst
@@ -39,6 +39,10 @@ Setup
this mode. In this case, you should build the kernel with
CONFIG_RANDOMIZE_BASE disabled if the architecture supports KASLR.

+- Build the gdb scripts (required on kernels v5.1 and above)::
+
+ make scripts_gdb
+
- Enable the gdb stub of QEMU/KVM, either

- at VM startup time by appending "-s" to the QEMU command line

---
base-commit: c757fc92a3f73734872c7793b97f06434773d65d
change-id: 20230112-documentation-gdb-af00d23b2b60

Best regards,
--
Jakob Koschel <[email protected]>


2023-01-13 08:53:16

by Bagas Sanjaya

[permalink] [raw]
Subject: Re: [PATCH] docs/scripts/gdb: add necessary make scripts_gdb step

On Thu, Jan 12, 2023 at 05:09:25PM +0100, Jakob Koschel wrote:
> In order to debug the kernel successfully with gdb you need to run
> 'make scripts_gdb' nowadays.
>
> This was changed with the following commit:
>
> Commit 67274c083438340ad16c ("scripts/gdb: delay generation of gdb
> constants.py")
>
> In order to have a complete guide for beginners this remark
> should be added to the offial documentation.
>

What about below?

```
Since commit <commit>, debugging kernel requires gdb scripts to be built
with "make scripts_gdb" first. Reflect this requirement in the
"Debugging kernel and modules via gdb" documentation.
```

Thanks.

--
An old man doll... just what I always wanted! - Clara


Attachments:
(No filename) (727.00 B)
signature.asc (235.00 B)
Download all attachments

2023-01-13 10:14:53

by Jakob Koschel

[permalink] [raw]
Subject: Re: [PATCH] docs/scripts/gdb: add necessary make scripts_gdb step



> On 13. Jan 2023, at 09:21, Bagas Sanjaya <[email protected]> wrote:
>
> On Thu, Jan 12, 2023 at 05:09:25PM +0100, Jakob Koschel wrote:
>> In order to debug the kernel successfully with gdb you need to run
>> 'make scripts_gdb' nowadays.
>>
>> This was changed with the following commit:
>>
>> Commit 67274c083438340ad16c ("scripts/gdb: delay generation of gdb
>> constants.py")
>>
>> In order to have a complete guide for beginners this remark
>> should be added to the offial documentation.
>>
>
> What about below?
>
> ```
> Since commit <commit>, debugging kernel requires gdb scripts to be built
> with "make scripts_gdb" first. Reflect this requirement in the
> "Debugging kernel and modules via gdb" documentation.
> ```

I guess you are suggesting this as an alternative commit message?

I'm happy to change this in v2.

>
> Thanks.
>
> --
> An old man doll... just what I always wanted! - Clara

- jakob

2023-01-19 21:32:21

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH] docs/scripts/gdb: add necessary make scripts_gdb step

Jakob Koschel <[email protected]> writes:

> In order to debug the kernel successfully with gdb you need to run
> 'make scripts_gdb' nowadays.
>
> This was changed with the following commit:
>
> Commit 67274c083438340ad16c ("scripts/gdb: delay generation of gdb
> constants.py")
>
> In order to have a complete guide for beginners this remark
> should be added to the offial documentation.
>
> ---
>
>
> Signed-off-by: Jakob Koschel <[email protected]>

Applied, thanks.

jon