[RFC PATCH 0/4] use DYNAMIC_DEBUG in drm/print

Jim Cromie jim.cromie at gmail.com
Sun May 30 19:21:42 UTC 2021


hi everyone,

this patchset reworks the drm.debug controlled debug categories,
and remaps them to use dynamic_debug_exec_queries().

To do this "smoothly", DRM_UT_* is converted from an enum to a
class-prefix string, which is prepended## to the real format string.
This lets us use:

 $> echo module drm\* format ^drm:kms: +p > /proc/dynamic_debug/control

3rd patch fixes a conflict while rebasing to rc3. I kept the fixup
separate for more visibility.

commit c5261e93758a6b36f4292403027af383ec9da129
Author: Lyude Paul <lyude at redhat.com>
Date:   Fri Mar 26 16:37:54 2021 -0400

    drm/print: Fixup DRM_DEBUG_KMS_RATELIMITED()
    
    Since we're about to move drm_dp_helper.c over to drm_dbg_*(), we'll want
    to make sure that we can also add ratelimited versions of these macros in
    order to retain some of the previous debugging output behavior we had.

I can see I havent been paying attention.
Are plans laid out where I can catch up quickly ?

Separately, but related, I have an RFC patchset on LKML to reduce
DYNAMIC_DEBUG's memory overhead.  Patch 34 hints that rate-limiting
can be done dynamically, on demand, for any prdbg known to
dynamic_debug.

https://lore.kernel.org/lkml/20210529200029.205306-1-jim.cromie@gmail.com/T/#ma5052696fe9171649b93b3e1f482de90314a9b61

Jim Cromie (4):
  drm: fixup comment spelling
  drm: RFC add choice to use dynamic debug in drm-debug
  drm: fixup DRM_DEBUG_KMS_RATELIMITED merge punt
  i915: map gvt pr_debug categories to bits in parameters/debug_gvt

 drivers/gpu/drm/Kconfig            |  13 ++++
 drivers/gpu/drm/drm_print.c        |  75 ++++++++++++++++++-
 drivers/gpu/drm/i915/gvt/Makefile  |   4 +
 drivers/gpu/drm/i915/i915_params.c |  76 +++++++++++++++++++
 include/drm/drm_print.h            | 116 ++++++++++++++++++++---------
 5 files changed, 243 insertions(+), 41 deletions(-)

-- 
2.31.1



More information about the dri-devel mailing list