[PATCH v5 06/30] dyndbg: drop NUM_TYPE_ARRAY

ARRAY_SIZE works here, since array decl is complete. no functional change Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- include/linux/dynamic_debug.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h

[PATCH v5 13/30] dyndbg: DECLARE_DYNDBG_CLASSMAP needs stub defn

Add a DECLARE_DYNDBG_CLASSMAP stub/do-nothing defn, so that callers will compile properly with CONFIG_DYNAMIC_DEBUG=n Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 39 ++- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/include

[PATCH v5 05/30] dyndbg: make ddebug_class_param union members same size

they will both see the same value. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- include/linux/dynamic_debug.h | 2 +- lib/dynamic_debug.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h

[PATCH v5 02/30] docs/dyndbg: explain flags parse 1st

ead. cc: linux-...@vger.kernel.org Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- Documentation/admin-guide/dynamic-debug-howto.rst | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/Documentation/admin-guide/dynamic-debug-howto.rst b/Documentation/adm

[PATCH v5 23/30] dyndbg-test: change do_prints testpoint to accept a loopct

nts messages (once) to the-log. NB: the count is clamped to 1, chosen to be notice able, but not annoying, and not enough to accidentally flood the logs. Signed-off-by: Jim Cromie --- lib/test_dynamic_debug.c | 32 +--- 1 file changed, 21 insertions(+), 11 deleti

[PATCH v5 26/30] dyndbg: split multi-query strings with %

ng is actually lost here. selftests-dyndbg: test_percent_splitting This does basic testing of classmaps using '%' separated multi-queries. It modprobes test_dynamic_debug with several classes enabled, and counts to verify that the expected sites show the enablement in the control file. Signed-off-by: J

[PATCH v5 27/30] selftests-dyndbg: add test_mod_submod

chset, closer to the classmap patches its validating. With some tedium, the tests could be refactored to split out early tests which avoid multi-cmds, and test only the class-params. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- r3 - skip test if LACK_TMOD older drop -v used in test_

[PATCH v5 12/30] dyndbg: macrofy a 2-index for-loop pattern

forward. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- lib/dynamic_debug.c | 19 --- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 283926530a43..ddcaa87f3b02 100644 --- a/lib/dynamic_debug.c +++ b/lib

[PATCH v5 14/30] dyndbg, module: make proper substructs in _ddebug_info

ntains a _ddebug_info field and module/main.c sets it up, so that gets adjusted rather obviously. The __packed attribute on _ddebug_info and the 3 contained structs closes the holes otherwise created by the structification (which was my excuse for not doing it originally). Signed-off-by: Jim Cromie

Re: [PATCH v4 29/58] docs/dyndbg: add classmap info to howto

hi Louis, thx for the nudge. On Mon, Sep 8, 2025 at 8:38 AM Louis Chauvet wrote: > \ > > Le 03/08/2025 à 05:57, Jim Cromie a écrit : > > Describe the 3 API macros providing dynamic_debug's classmaps > > > > DYNDBG_CLASSMAP_DEFINE - create & export a classma

[PATCH v4 51/58] drm-dyndbg: add DRM_CLASSMAP_USE to udl driver

The udl driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/udl/udl_main.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/udl

[PATCH v4 54/58] drm-dyndbg: add DRM_CLASSMAP_USE to the qxl driver

The qxl driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/qxl/qxl_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/qxl/qxl_drv.c

[PATCH v4 53/58] drm-dyndbg: add DRM_CLASSMAP_USE to the gud driver

The gud driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/gud/gud_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/gud/gud_drv.c

[PATCH v4 49/58] drm-dyndbg: add DRM_CLASSMAP_USE to vmwgfx driver

The vmwgfx driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm

[PATCH v4 45/58] drm-dyndbg: add DRM_CLASSMAP_USE to bochs

tiny/bochs has 5 DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/tiny/bochs.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/tiny/boch

[PATCH v4 21/58] dyndbg: check DYNAMIC_DEBUG_CLASSMAP_DEFINE args at compile-time

examples are added to test_dynamic_debug_submod.ko, and will fail compilation if -DDD_MACRO_ARGCHECK is added to cflags. This wouldn't be a useful CONFIG_ item, since it breaks the build. Signed-off-by: Jim Cromie --- v3- $macro_name =~ s/DYNDBG/DYNAMIC_DEBUG/ prev- - split sta

[PATCH v4 44/58] drm-dyndbg: add DRM_CLASSMAP_USE to simpledrm

tiny/simpledrm has 3 DRM_UT_DRIVER debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/sysfb/simpledrm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/

[PATCH v4 40/58] drm-dyndbg: DRM_CLASSMAP_USE in drm_dp_helper

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- drivers/gpu/drm/display/drm_dp_helper.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gp

[PATCH v4 46/58] drm-dyndbg: add DRM_CLASSMAP_USE to etnaviv

etnaviv has 5 DRM_UT_CORE debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs as well as plain-old pr_debug()s Signed-off-by: Jim Cromie --- drivers/gpu/drm/etnaviv/etnaviv_drv.c | 2 ++ 1 file changed, 2 insertions(+)

[PATCH v4 57/58] amdxdna: add -DDYNAMIC_DEBUG_MODULE to cflags - RFC

C: Maxime Ripard CC: Thomas Zimmermann CC: David Airlie CC: Simona Vetter Signed-off-by: Jim Cromie --- drivers/accel/amdxdna/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/accel/amdxdna/Makefile b/drivers/accel/amdxdna/Makefile index 0e9adf6890a01..800f81f5a7023 100644

[PATCH v4 43/58] drm-dyndbg: add DRM_CLASSMAP_USE to virtio_gpu

virtio_gpu has 10 DRM_UT_CORE debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/virtio/virtgpu_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/v

[PATCH v4 13/58] dyndbg, module: make proper substructs in _ddebug_info

contained structs closes the holes otherwise created by the structification (which was my excuse for not doing it originally). Signed-off-by: Jim Cromie --- -v2 rework towards front of series --- include/asm-generic/vmlinux.lds.h | 4 +- include/linux/dynamic_debug.h | 42 +++- ke

[PATCH v4 14/58] dyndbg: hoist classmap-filter-by-modname up to ddebug_add_module

Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 56 ++--- 1 file changed, 32 insertions(+), 24 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 53ce282554266..bbbdb8aba0716 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_de

[PATCH v4 30/58] dyndbg: reserve flag-bit _DPRINTK_FLAGS_PREFIX_CACHED

that gets called. The cache contents must avoid the Thread-id (its not callsite), and could avoid the line (to shrink the cache). But then line needs %d each time pr_debug is reached. ATM, this is just the flag-bit reservation. Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 1

[PATCH v4 02/58] docs/dyndbg: explain flags parse 1st

ead. cc: linux-...@vger.kernel.org Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- Documentation/admin-guide/dynamic-debug-howto.rst | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/Documentation/admin-guide/dynamic-debug-howto.rst b/Documentation/adm

[PATCH v4 19/58] dyndbg-API: replace DECLARE_DYNDBG_CLASSMAP

tristate, allowing 3 super/sub combos: Y/Y, Y/M, M/M (not N/Y, since this is disallowed by dependence). Y/Y testing exposed a missing __align(8) in the _METADATA macro, which M/M didn't see because the module-loader memory placement constrains it instead. Fixes: aad0214f3026 ("dyndbg: add

[PATCH v4 42/58] drm-dyndbg: add DRM_CLASSMAP_USE to Xe driver

Invoke DRM_CLASSMAP_USE from xe_drm_client.c. When built with CONFIG_DRM_USE_DYNAMIC_DEBUG=y, this tells dydnbg that Xe has drm.debug callsites. Signed-off-by: Jim Cromie --- drivers/gpu/drm/xe/xe_drm_client.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/xe

[PATCH v4 56/58] drm: restore CONFIG_DRM_USE_DYNAMIC_DEBUG un-BROKEN

Time for some thorough CI. Also, the previous 18 patches could perhaps be replaced by a single invocation of DYNDBG_CLASSMAP_USE, from a C-file linked into all drm drivers & helpers. I didn't find such a file, nor a drm-client linkage item in the Makefile. Signed-off-by: J

[PATCH v4 34/58] drm-dyndbg: adapt drm core to use dyndbg classmaps-v2

r now, so they still redundantly re-declare the classmap, but we can convert the drivers later to DYNDBG_CLASSMAP_USE Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- this ignores a checkpatch WARNING: Argument 'name' is not used in function-like macro #70: FILE: include/

[PATCH v4 32/58] dyndbg: refactor *dynamic_emit_prefix to split lookups

orage which is not fast enough for every-time pr_debugs, but would work to fill the cache the 1st time its enabled and invoked. Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 40 +--- 1 file changed, 25 insertions(+), 15 deletions(-) diff --git

[PATCH v4 37/58] drm-dyndbg: DRM_CLASSMAP_USE in amdgpu driver

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gp

[PATCH v4 52/58] drm-dyndbg: add DRM_CLASSMAP_USE to mgag200 driver

The mgag200 driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/mgag200/mgag200_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm

[PATCH v4 03/58] test-dyndbg: fixup CLASSMAP usage error

t value, in this case it was _base + 1. This leaves HI class un-selectable. NB: the macro could better validate its arguments. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet Tested-by: Louis Chauvet --- lib/test_dynamic_debug.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH v4 39/58] drm-dyndbg: DRM_CLASSMAP_USE in drm_crtc_helper

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- drivers/gpu/drm/drm_crtc_helper.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gpu/drm

[PATCH v4 48/58] drm-dyndbg: add DRM_CLASSMAP_USE to radeon

radeon has some DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg about its use of the class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/radeon/radeon_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/r

[PATCH v4 29/58] docs/dyndbg: add classmap info to howto

TBD: some of this might be over-specification, or just over-talked. NB: The _DEFINE & _USE model makes the user dependent on the definer, just like EXPORT_SYMBOL(__drm_debug) already does. cc: linux-...@vger.kernel.org Signed-off-by: Jim Cromie --- v3- rework protection around PARAM v0

[PATCH v4 31/58] dyndbg: add _DPRINTK_FLAGS_INCL_LOOKUP for +mfsl flags

dule:function:sourcfile:line: " So this patch extracts _DPRINTK_FLAGS_INCL_LOOKUP flags-combo out of _DPRINTK_FLAGS_INCL_ANY, then redefs latter. Next re-refactor dynamic_emit_prefix inner/outer fns accordingly. Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 9 + 1 file changed,

[PATCH v4 06/58] dyndbg: drop NUM_TYPE_ARRAY

ARRAY_SIZE works here, since array decl is complete. no functional change Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- include/linux/dynamic_debug.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h

[PATCH v4 04/58] dyndbg: reword "class unknown, " to "class:_UNKNOWN_"

NKNOWN_" to loudly announce the erroneous situation, and to make it uniquely greppable. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- lib/dynamic_debug.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 5a007952f7f2

[PATCH v4 58/58] accel: add -DDYNAMIC_DEBUG_MODULE to subdir-cflags - RFC

ssmap and controlling kparam. Signed-off-by: Jim Cromie --- drivers/accel/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/accel/Makefile b/drivers/accel/Makefile index a301fb6089d4c..e437549cba8ac 100644 --- a/drivers/accel/Makefile +++ b/drivers/accel/Makefile @@ -1,5 +1,7 @

[PATCH v4 47/58] drm-dyndbg: add DRM_CLASSMAP_USE to gma500 driver

The gma500 has 126 DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/gma500/psb_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/g

[PATCH v4 01/58] docs/dyndbg: update examples \012 to \n

commit 47ea6f99d06e ("dyndbg: use ESCAPE_SPACE for cat control") changed the control-file to display format strings with "\n" rather than "\012". Update the docs to match the new reality. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet Tested-by: Louis Cha

[PATCH v4 11/58] dyndbg: replace classmap list with a vector

ing of classmaps currently, and doesn't need to. For now, just add comment. Signed-off-by: Jim Cromie --- v3- drop __outvar as promising magic it doesnt have --- include/linux/dynamic_debug.h | 1 - lib/dynamic_debug.c | 62 ++- 2 files changed, 32

[PATCH v4 23/58] dyndbg-API: promote DYNAMIC_DEBUG_CLASSMAP_PARAM to API

nside the fn, to give access to all kp members. Also clean up and improve comments in test-code, and add MODULE_DESCRIPTIONs. cc: linux-...@vger.kernel.org Signed-off-by: Jim Cromie --- -v9 - fixup drm-print.h add PARAM_REF forwarding macros with DYNAMIC_DEBUG_CLASSMAP_PARAM_REF in the API

[PATCH v4 24/58] dyndbg: treat comma as a token separator

onfigurations. Signed-off-by: Jim Cromie Co-developed-by: Łukasz Bartosik Signed-off-by: Łukasz Bartosik --- - skip comma tests if no builtins -v3 squash in tests and doc --- .../admin-guide/dynamic-debug-howto.rst | 9 +--- lib/dynamic_debug.c

[PATCH v4 55/58] drm-dyndbg: add DRM_CLASSMAP_USE to the drm_gem_shmem_helper driver

The drm_gem_shmem_helper driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/drm_gem_shmem_helper.c | 1 + 1 file changed, 1 insertion(+) diff --git a

[PATCH v4 25/58] dyndbg: split multi-query strings with %

ng is actually lost here. selftests-dyndbg: test_percent_splitting This does basic testing of classmaps using '%' separated multi-queries. It modprobes test_dynamic_debug with several classes enabled, and counts to verify that the expected sites show the enablement in the control file. Signed-off-by: J

[PATCH v4 35/58] drm-dyndbg: adapt DRM to invoke DYNAMIC_DEBUG_CLASSMAP_PARAM

Invoke DYNAMIC_DEBUG_CLASSMAP_PARAM to hook drm.debug (__drm_debug) to the DRM_UT_* classmap, replacing the ad-hoc wiring previously doing it. Add DRM_CLASSMAP_* adapter macros to selectively use DYNAMIC_DEBUG_CLASSMAP_* when DRM_USE_DYNAMIC_DEBUG=y is configured. Signed-off-by: Jim Cromie

[PATCH v4 16/58] dyndbg-API: remove DD_CLASS_TYPE_(DISJOINT|LEVEL)_NAMES and code

d for by anyone else, so reduce LOC & test-surface; simplify things. Also rename enum class_map_type to enum ddebug_class_map_type. Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 23 ++-- lib/dynamic_debug.c | 102 +++--- lib/test_dyn

[PATCH v4 05/58] dyndbg: make ddebug_class_param union members same size

they will both see the same value. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- include/linux/dynamic_debug.h | 2 +- lib/dynamic_debug.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h

[PATCH v4 38/58] drm-dyndbg: DRM_CLASSMAP_USE in i915 driver

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- drivers/gpu/drm/i915/i915_params.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gpu/drm/

[PATCH v4 09/58] dyndbg: refactor param_set_dyndbg_classes and below

ddebug_apply_class_bitmap(), also check for actual changes to the bits before announcing them, to declutter logs. No functional change. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- v3 s/modnm/mod_name/g --- lib/dynamic_debug.c | 65 - 1 file

[PATCH v4 10/58] dyndbg: tighten fn-sig of ddebug_apply_class_bitmap

old_bits arg is currently a pointer to the input bits, but this could allow inadvertent changes to the input by the fn. Disallow this. And constify new_bits while here. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- lib/dynamic_debug.c | 21 +++-- 1 file changed, 11

[PATCH v4 07/58] dyndbg: tweak pr_fmt to avoid expansion conflicts

Disambiguate pr_fmt(fmt) arg, by changing it to _FMT_, to avoid naming confusion with many later macros also using that argname. no functional change Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/dynamic_debug.c b/lib

[PATCH v4 08/58] dyndbg: reduce verbose/debug clutter

ed 1 queries, with 0 matches, 0 errs Also reduce verbose=3 messages in ddebug_add_module When modprobing a module, dyndbg currently logs/says "add-module", and then "skipping" if the module has no prdbgs. Instead just check 1st and return quietly. no functional change Signe

[PATCH v4 33/58] drm: use correct ccflags-y spelling

; modules built with CONFIG_DYNAMIC_DEBUG=y (!1) don't need the flag. Fixes: 84ec67288c10 ("drm_print: wrap drm_*_dbg in dyndbg descriptor factory macro") Signed-off-by: Jim Cromie --- drivers/gpu/drm/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/

[PATCH v4 15/58] dyndbg: ddebug_table.mod_name down to _ddebug_info

f up. ddebug_table contains a _ddebug_info member, so its users keep access to mod_name, just now with "->info." added in. In static ddebug_add_module(&di), reinforce the cursor-model by dropping the modname arg, and setting di->mod_name at each caller. Signed-off-by: Jim Cromie

[PATCH v4 36/58] drm-print: fix config-dependent unused variable

tic-key patches in the proper state. The compiler evidently sees this lack of reference and complains. So this drops the local var and refs the field directly in the macro-call, which avoids the warning/error. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- drivers/gpu/drm/drm_pri

[PATCH v4 27/58] dyndbg: drop "protection" of class'd pr_debugs from legacy queries

ow the module defining the classmap to protect its classes from unintended alterations by legacy/class-less queries. Next: Author choice: use of DYNAMIC_DEBUG_CLASSMAP_PARAM() means they want the drm.debug style control point. We should presume they want it to reflect whats set underneath, with o

[PATCH v4 28/58] dyndbg: restore classmap protection when theres a controlling_param

tection macro'd to false -v2.with-this-patch - sysfs knob decides -v2.speculative - module decides wrt classmap protection seems unneeded now, TBD NOTE: protection is only against class-less queries, explicit "class FOO" adjustments are allowed (that is the mechanism

[PATCH v4 41/58] drm-dyndbg: DRM_CLASSMAP_USE in nouveau

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie --- drivers/gpu/drm/nouveau/nouveau_drm.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_drm

[PATCH v4 50/58] drm-dyndbg: add DRM_CLASSMAP_USE to vkms driver

The vkms driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/vkms/vkms_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/vkms

[PATCH v4 22/58] dyndbg-test: change do_prints testpoint to accept a loopct

nts messages (once) to the-log. NB: the count is clamped to 1, chosen to be notice able, but not annoying, and not enough to accidentally flood the logs. Signed-off-by: Jim Cromie --- lib/test_dynamic_debug.c | 37 ++--- 1 file changed, 26 insertions(+),

[PATCH v4 17/58] selftests-dyndbg: add a dynamic_debug run_tests target

t a common or desirable build/config. Signed-off-by: Jim Cromie Co-developed-by: Łukasz Bartosik Signed-off-by: Łukasz Bartosik --- -r3 turn off green at end drop config dep on TEST_DYNAMIC_DEBUG, since basic-test uses builtin params --- MAINTAINERS

[PATCH v4 26/58] selftests-dyndbg: add test_mod_submod

chset, closer to the classmap patches its validating. With some tedium, the tests could be refactored to split out early tests which avoid multi-cmds, and test only the class-params. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- r3 - skip test if LACK_TMOD older drop -v used in test_

[PATCH v4 18/58] dyndbg: change __dynamic_func_call_cls* macros into expressions

ore stuff } Fix for this usage by changing __dynamic_func_call_cls{,_no_desc} macros into expressions, by replacing the do-while-0s with a ({ }) wrapper. In the common usage, the trailing ';' converts the expression into a statement. drm_dbg("statement form"); Signed

[PATCH v4 12/58] dyndbg: macrofy a 2-index for-loop pattern

forward. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- lib/dynamic_debug.c | 20 +--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 283926530a43b..7c5a1ae2c0a65 100644 --- a/lib/dynamic_debug.c +++ b/lib

[PATCH v4 20/58] dyndbg: detect class_id reservation conflicts

igned-off-by: Jim Cromie --- --- lib/dynamic_debug.c | 29 + lib/test_dynamic_debug.c | 8 2 files changed, 37 insertions(+) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index fa8d1ee214066..d6687623c5a96 100644 --- a/lib/dynamic_debug.c +++

[PATCH v4 00/58] fix CONFIG_DRM_USE_DYNAMIC_DEBUG=y

minor: .rework several cleanup patches forward .code/doc/commit-msg tweaks for Louis Chauvet feedback. .refactor dyndbg dynamic-prefix to split TID from cacheable prefix .cflags for drivers/accel borrowing DRM.debug Jim Cromie (58): docs/dyndbg: update examples \012 to \n docs/dyndbg:

Re: [PATCH v3 28/54] dyndbg: restore classmap protection when theres a controlling_param

On Fri, Jun 20, 2025 at 8:47 AM Sean Paul wrote: > On Thu, Apr 3, 2025 at 9:48 AM Jim Cromie wrote: > > > > \snip > > > > > -static void ddebug_match_apply_kparam(const struct kernel_param *kp, > > - c

Re: [PATCH v3 21/54] dyndbg-test: change do_prints testpoint to accept a loopct

On Tue, Apr 15, 2025 at 4:04 AM Louis Chauvet wrote: > > > > Le 02/04/2025 à 19:41, Jim Cromie a écrit : > > echo 1000 > /sys/module/test_dynamic_debug/parameters/do_prints > > > > This allows its use as a scriptable load generator, to generate > > dynami

Re: [PATCH v3 23/54] dyndbg: treat comma as a token separator

On Tue, Apr 15, 2025 at 4:05 AM Louis Chauvet wrote: > > > > Le 02/04/2025 à 19:41, Jim Cromie a écrit : > > Treat comma as a token terminator, just like a space. This allows a > > user to avoid quoting hassles when spaces are otherwise needed: > > > > :#>

Re: [PATCH v3 17/54] dyndbg-API: replace DECLARE_DYNDBG_CLASSMAP

On Tue, Apr 15, 2025 at 4:01 AM Louis Chauvet wrote: > > > > Le 02/04/2025 à 19:41, Jim Cromie a écrit : > > DECLARE_DYNDBG_CLASSMAP() has a design error; its usage fails a basic > > K&R rule: "define once, refer many times". > > > > When D

Re: [PATCH v3 20/54] dyndbg: check DYNAMIC_DEBUG_CLASSMAP_DEFINE args at compile-time

On Tue, Apr 15, 2025 at 4:04 AM Louis Chauvet wrote: > > > > Le 02/04/2025 à 19:41, Jim Cromie a écrit : > > Add __DYNAMIC_DEBUG_CLASSMAP_CHECK to implement the following > > arg-checks at compile-time: > > > > 0 <= _base < 63 > > class_

Re: [PATCH v3 26/54] dyndbg: change __dynamic_func_call_cls* macros into expressions

On Tue, Apr 15, 2025 at 4:06 AM Louis Chauvet wrote: > > > > Le 02/04/2025 à 19:41, Jim Cromie a écrit : > > The Xe driver's XE_IOCTL_DBG macro calls drm_dbg() from inside an if > > (expression). This breaks when CONFIG_DRM_USE_DYNAMIC_DEBUG=y because > >

Re: [PATCH v3 18/54] selftests-dyndbg: add tools/testing/selftests/dynamic_debug/*

On Tue, Apr 15, 2025 at 4:02 AM Louis Chauvet wrote: > > > > Le 02/04/2025 à 19:41, Jim Cromie a écrit : > > Add a selftest script for dynamic-debug. The config requires > > CONFIG_TEST_DYNAMIC_DEBUG=m and CONFIG_TEST_DYNAMIC_DEBUG_SUBMOD=m, > > which tacitly requir

[PATCH v2 43/59] drm-dyndbg: DRM_CLASSMAP_USE in nouveau

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie --- drivers/gpu/drm/nouveau/nouveau_drm.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_drm

[PATCH v2 15/59] dyndbg: hoist classmap-filter-by-modname up to ddebug_add_module

Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 57 - 1 file changed, 35 insertions(+), 22 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index f7ec2365ab40..192783ff7b98 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_debu

[PATCH v3 43/54] drm-dyndbg: add DRM_CLASSMAP_USE to etnaviv

etnaviv has 5 DRM_UT_CORE debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs as well as plain-old pr_debug()s Signed-off-by: Jim Cromie --- drivers/gpu/drm/etnaviv/etnaviv_drv.c | 2 ++ 1 file changed, 2 insertions(+)

[PATCH v3 40/54] drm-dyndbg: add DRM_CLASSMAP_USE to virtio_gpu

virtio_gpu has 10 DRM_UT_CORE debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/virtio/virtgpu_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/v

[PATCH v2 08/59] dyndbg: refactor param_set_dyndbg_classes and below

ddebug_apply_class_bitmap(), also check for actual changes to the bits before announcing them, to declutter logs. No functional change. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- --- lib/dynamic_debug.c | 65 - 1 file changed, 40 insertions

[PATCH v3 10/54] dyndbg: tighten fn-sig of ddebug_apply_class_bitmap

old_bits arg is currently a pointer to the input bits, but this could allow inadvertent changes to the input by the fn. Disallow this. And constify new_bits while here. Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- lib/dynamic_debug.c | 21 +++-- 1 file changed, 11

[PATCH v3 22/54] dyndbg-API: promote DYNAMIC_DEBUG_CLASSMAP_PARAM to API

nside the fn, to give access to all kp members. Also clean up and improve comments in test-code, and add MODULE_DESCRIPTIONs. cc: linux-...@vger.kernel.org Signed-off-by: Jim Cromie --- -v9 - fixup drm-print.h add PARAM_REF forwarding macros with DYNAMIC_DEBUG_CLASSMAP_PARAM_REF in the API

[PATCH v2 12/59] dyndbg, module: make proper substructs in _ddebug_info

contained structs closes the holes otherwise created by the structification (which was my excuse for not doing it originally). Signed-off-by: Jim Cromie --- -v2 rework towards front of series --- include/asm-generic/vmlinux.lds.h | 4 +- include/linux/dynamic_debug.h | 28 +++--- ke

[PATCH v2 51/59] drm-dyndbg: add DRM_CLASSMAP_USE to vmwgfx driver

The vmwgfx driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm

[PATCH v2 29/59] dyndbg: change __dynamic_func_call_cls* macros into expressions

this usage by changing __dynamic_func_call_cls{,_no_desc} macros into expressions, by replacing the do-while-0s with a ({ }) wrapper. In the common usage, the trailing ';' converts the expression into a statement. drm_dbg("statement form"); Signed-off-by: Jim Cromie

[PATCH v2 39/59] drm-dyndbg: DRM_CLASSMAP_USE in amdgpu driver

drm_dbg()s. Fixes: f158936b60a7 ("drm: POC drm on dyndbg - use in core, 2 helpers, 3 drivers.") Signed-off-by: Jim Cromie --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu

[PATCH v3 32/54] drm-dyndbg: adapt DRM to invoke DYNAMIC_DEBUG_CLASSMAP_PARAM

Invoke DYNAMIC_DEBUG_CLASSMAP_PARAM to hook drm.debug (__drm_debug) to the DRM_UT_* classmap, replacing the ad-hoc wiring previously doing it. Add DRM_CLASSMAP_* adapter macros to selectively use DYNAMIC_DEBUG_CLASSMAP_* when DRM_USE_DYNAMIC_DEBUG=y is configured. Signed-off-by: Jim Cromie

[PATCH v2 23/59] dyndbg: move .mod_name from/to structs ddebug_table/_ddebug_info

a different offset. Signed-off-by: Jim Cromie --- include/linux/dynamic_debug.h | 1 + lib/dynamic_debug.c | 41 ++- 2 files changed, 22 insertions(+), 20 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h index

[PATCH v3 42/54] drm-dyndbg: add DRM_CLASSMAP_USE to bochs

tiny/bochs has 5 DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/tiny/bochs.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/tiny/boch

[PATCH v3 19/54] dyndbg: detect class_id reservation conflicts

igned-off-by: Jim Cromie --- --- lib/dynamic_debug.c | 29 + lib/test_dynamic_debug.c | 8 2 files changed, 37 insertions(+) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 53e261dbf81e..56b503af0b31 100644 --- a/lib/dynamic_debug.c +++

[PATCH v3 47/54] drm-dyndbg: add DRM_CLASSMAP_USE to vkms driver

The vkms driver has a number of DRM_UT_* debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module uses them. Signed-off-by: Jim Cromie --- drivers/gpu/drm/vkms/vkms_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/vkms

[PATCH v3 23/54] dyndbg: treat comma as a token separator

onfigurations. Signed-off-by: Jim Cromie Co-developed-by: Łukasz Bartosik Signed-off-by: Łukasz Bartosik --- - skip comma tests if no builtins -v3 squash in tests and doc --- .../admin-guide/dynamic-debug-howto.rst | 9 +--- lib/dynamic_debug.c

[PATCH v3 14/54] dyndbg: hoist classmap-filter-by-modname up to ddebug_add_module

Signed-off-by: Jim Cromie --- lib/dynamic_debug.c | 56 ++--- 1 file changed, 32 insertions(+), 24 deletions(-) diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c index 104cf8abdf33..046c4ffb38f8 100644 --- a/lib/dynamic_debug.c +++ b/lib/dynamic_de

[PATCH v2 38/59] drm-print: fix config-dependent unused variable

tic-key patches in the proper state. The compiler evidently sees this lack of reference and complains. So this drops the local var and refs the field directly in the macro-call, which avoids the warning/error. Signed-off-by: Jim Cromie --- drivers/gpu/drm/drm_print.c | 3 +-- 1 file changed, 1

[PATCH v2 06/59] dyndbg: drop NUM_TYPE_ARRAY

ARRAY_SIZE works here, since array decl is complete. no functional change Signed-off-by: Jim Cromie Reviewed-by: Louis Chauvet --- include/linux/dynamic_debug.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/linux/dynamic_debug.h b/include/linux/dynamic_debug.h

[PATCH v2 46/59] drm-dyndbg: add DRM_CLASSMAP_USE to simpledrm

tiny/simpledrm has 3 DRM_UT_DRIVER debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs. Signed-off-by: Jim Cromie --- drivers/gpu/drm/tiny/simpledrm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm

[PATCH v2 20/59] dyndbg: check DYNDBG_CLASSMAP_DEFINE args at compile-time

uld soften ERR to WARN and qualify advice wrt file-vs-fn scope, & new-scope-declaratives exception (forex: _METADATA_) The known exceptions by name/pattern works well (_METADATA_ is covered by "struct"), this patch just wants static_assert added. On my list, with above. Signed-off

[PATCH v2 28/59] selftests-dyndbg: add test_mod_submod

chset, closer to the classmap patches its validating. With some tedium, the tests could be refactored to split out early tests which avoid multi-cmds, and test only the class-params. Signed-off-by: Jim Cromie --- - drop -v used in test_mod_submod(). V=1 does it for whole test

[PATCH v2 48/59] drm-dyndbg: add DRM_CLASSMAP_USE to etnaviv

etnaviv has 5 DRM_UT_CORE debugs, make them controllable when CONFIG_DRM_USE_DYNAMIC_DEBUG=y by telling dyndbg that the module has class'd debugs as well as plain-old pr_debug()s Signed-off-by: Jim Cromie --- drivers/gpu/drm/etnaviv/etnaviv_drv.c | 2 ++ 1 file changed, 2 insertions(+)

  1   2   3   4   5   6   7   8   9   10   >