Re: [PATCH v2 1/2] docs: Add debugging section to process

[Randy Dunlap <rdunlap@infradead.org>]

On 11/14/24 10:05 PM, Sebastian Fricke wrote:
> Hey Randy,
> 
> Thank you for the review, I'll apply these changes, I just found one
> advice below not terribly helpful maybe you can clarify ...
> 
> On 14.11.2024 21:06, Randy Dunlap wrote:
>>
>>
>> On 11/13/24 3:17 AM, Sebastian Fricke wrote:
>>> This idea was formed after noticing that new developers experience
>>> certain difficulty to navigate within the multitude of different
>>> debugging options in the Kernel and while there often is good
>>> documentation for the tools, the developer has to know first that they
>>> exist and where to find them.
>>> Add a general debugging section to the Kernel documentation, as an
>>> easily locatable entry point to other documentation and as a general
>>> guideline for the topic.
>>>
>>> Signed-off-by: Sebastian Fricke <sebastian.fricke@collabora.com>
>>> ---
>>>  .../driver_development_debugging_guide.rst         | 214 ++++++++++++++++
>>>  Documentation/process/debugging/index.rst          |  65 +++++
>>>  .../debugging/userspace_debugging_guide.rst        | 278 +++++++++++++++++++++
>>>  Documentation/process/index.rst                    |   8 +-
>>>  4 files changed, 562 insertions(+), 3 deletions(-)
>>>
>>
>>
>>> diff --git a/Documentation/process/debugging/userspace_debugging_guide.rst b/Documentation/process/debugging/userspace_debugging_guide.rst
>>> new file mode 100644
>>> index 000000000000..a7c94407bcae
>>> --- /dev/null
>>> +++ b/Documentation/process/debugging/userspace_debugging_guide.rst
>>> @@ -0,0 +1,278 @@
>>> +.. SPDX-License-Identifier: GPL-2.0
>>> +
>>> +==========================
>>> +Userspace debugging advice
>>> +==========================
>>> +
>>> +A brief overview of common tools to debug the Linux Kernel from userspace.
>>
>> Make that a sentence?
> 
> Can you clarify this?
> 
> This could either mean:
> - What you even bother to make a sentence out of that?
> - Please make a proper sentence out of this because it is hard to
>   understand
> - Please go into more detail because this is too brief
> 
> Or maybe something completely different :)

Sorry for my brevity.
I just meant "do you want to make that line a sentence, because it is not a sentence
in its current form." But I'm not going to lose any sleep over it.

Thanks.

-- 
~Randy