Re: [PATCH 00/17] dynamic-debug cleanups refactors maintenance

[Andrew Morton]

On Mon, 04 May 2026 14:45:06 -0600 Jim Cromie <jim.cromie@xxxxxxxxx> wrote:

> This series is nearly all maintenance: it refactors/splits functions,
> tightens an internal callchain, drops linked-listing where an
> array,length is already available, reduces verbose=3 logging for
> usability, and reorganizes several structs for better organization.
> 
> Getting these into linux-next for integration testing would clear the
> deck for fixing Dynamic-debug classmaps, which is needed to restore
> the un-BROKEN status of DRM_USE_DYNAMIC_DEBUG.
> 
> Theyre on master at v7.1-rc2
> 
> The "modules" intersection is on the re-composing of struct
> ddebug_info, which is a member in struct module, and maps a module's
> __dyndbg* elf sections.
> 
> The user visible change to /proc/dynamic_debug/control is s/class
> unknown/class:_UNKNOWN_/, which is a more visible/greppable indication
> of incomplete class definitions.

Wait.  We can't make userspace-visible changes?

> Coder visible change is to drop the enum ddebug_class_map_type's
> unused vals - namely: DD_CLASS_TYPE_DISJOINT_NAMES
> & DD_CLASS_TYPE_LEVEL_NAMES
> 
> These allowed more symbolic named inputs:
>   echo +DRM_UT_CORE > /sys/module/drm/parameters/debug
> 
> But theyre unused 3 years later, and probably not worth keeping.
> With a removal commit in the log, its easy enough to restore them later.
> 
> ...
> 
>  MAINTAINERS                                        |   1 +
>  include/linux/dynamic_debug.h                      | 106 ++---
>  kernel/module/main.c                               |  12 +-
>  lib/dynamic_debug.c                                | 504 ++++++++++-----------
>  lib/test_dynamic_debug.c                           |  28 +-
>  tools/testing/selftests/Makefile                   |   1 +
>  tools/testing/selftests/dynamic_debug/Makefile     |   9 +
>  tools/testing/selftests/dynamic_debug/config       |   7 +
>  .../selftests/dynamic_debug/dyndbg_selftest.sh     | 257 +++++++++++
>  9 files changed, 582 insertions(+), 343 deletions(-)

No Documentation/ updates?