mbox series

[v3,0/2] Documentation: Debugging guide

Message ID 20241028-media_docs_improve_v3-v3-0-edf5c5b3746f@collabora.com (mailing list archive)
Headers show
Series Documentation: Debugging guide | expand

Message

Sebastian Fricke Nov. 18, 2024, 7:03 a.m. UTC
The series contains:
- a general debugging guide split into debugging for driver developers and
debugging from userspace
- a new summary page for all media related documentation. This is inspired by
other subsystems, which first of all allows a user to find the subsystem
under the subsystems page and secondly eases general navigation through the
documentation that is sprinkled onto multiple places.
- a guide on how to debug code in the media subsystem, which points to the
parts of the general documentation and adds own routines.

WHY do we need this?
--------------------

For anyone without years of experience in the Linux kernel, knowing which tool
to use or even which tools are available is not as straightforward as some
senior developers might perceive.
We realized that there is a general need for a kind of "start page", that
allows especially beginners to get up-to-speed with the codebase and the
documentation. The documentation in particular is currently quite hard to navigate
as you mostly have to know what you are searching for to find it.

WHAT do we cover?
-----------------

The document is structured into two sections:

1. A problem-focused approach: This means, a developer facing an issue matching
one of the given examples, will find suggestions for how to approach that
problem (e.g. which tool to use) in this section
2. A tool-focused approach: This sections highlights the available tools, with
comparisions between the tools if sensible. The goal of this work is
**duplicate as little as possible** from the existing documentation and
instead provide a rough overview that provides:
- A link to the actual documentation
- A minimal example for how it can be used (from a media perspective,
  if the usage isn't absolutely trivial like printk)
- A rational for why it should be used

To: Jonathan Corbet <corbet@lwn.net>
Cc: rdunlap@infradead.org
Cc: linux-doc@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-media@vger.kernel.org
Cc: laurent.pinchart@ideasonboard.com
Cc: hverkuil-cisco@xs4all.nl
Cc: mchehab@kernel.org
Cc: kernel@collabora.com
Cc: bob.beckett@collabora.com
Cc: nicolas.dufresne@collabora.com
Signed-off-by: Sebastian Fricke <sebastian.fricke@collabora.com>

---
Changes in v3:
- Fix reported grammatical errors and typos
- Shrink further lines to 80 characters
- Add PSI section
- Rebase to latest docs-next
- Link to v2: https://lore.kernel.org/r/20241028-media_docs_improve_v3-v2-0-f1960ae22c5d@collabora.com

Changes in v2:
- Rebase to docs-next
- Shrink remaining lines to 80 wherever I missed it on the previous version
- Replace italic markup with litteral markup
- Add more detailed description where suggested
- Remove unnecassary labels and jumps
- Remove the separate general advice file again and add its content to the
  index file
- Add links to the process document and the media admin guide
- Remove links from the top page
- Split toc listing in the index file into general and specific
- Improve commit titles
- Link to v1: https://lore.kernel.org/r/20241028-media_docs_improve_v3-v1-0-2b1b486c223e@collabora.com

Changes in v1 (first non-RFC):
- Move the guides from Documentation/debugging to Documentation/process/debugging
- Remove the new Documentation/media folder and place the media-specific guide
  instead into Documentation/process/debugging
- Reduce the line length to 80 character wherever possible
- Exchange |code| with © and remove the include
- Remove :code: specifiers
- Exchange html links with :doc: and :ref: links, to allow sphinx to convert them correctly
- Use markdown links only when necessary 
- Various style fixes
- Improve the description for how to read a crash dump
- Split the general advice into a separate file
- Remove unnecessary labels
- Replace duplicated ftrace example with links to the documentation
- Add 2 additional debugging sections to the media debugging guide
- Replace the lkml link with the matching lore link
- Extend the error checkers section with further details
- Add intro sentences on the media debugging guide to the various sections
- Remove ftrace examples and point to the documentation instead
- Change the section depth to allow cross references via the autosectionlabels
- Add Elixir links whenever I point to a specific file

Changes in v2 (RFC):
- Split the media debugging guide into a general and a media specific guide,
which contains mostly references to the general guide and a few media
specific aspects.
- Fill out TBD sections
- Add device coredump section

---
Sebastian Fricke (2):
      docs: Add debugging section to process
      docs: Add debugging guide for the media subsystem

 Documentation/admin-guide/media/index.rst          |   5 +
 .../driver_development_debugging_guide.rst         | 223 ++++++++++++++++
 Documentation/process/debugging/index.rst          |  78 ++++++
 .../debugging/media_specific_debugging_guide.rst   | 180 +++++++++++++
 .../debugging/userspace_debugging_guide.rst        | 280 +++++++++++++++++++++
 Documentation/process/index.rst                    |   8 +-
 6 files changed, 771 insertions(+), 3 deletions(-)
---
base-commit: c818d5c64c9a8cc14853d716bf7ce6674a6126d0
change-id: 20241028-media_docs_improve_v3-3d7c16017713

Best regards,