Commit ccc2b496 authored by Jim Cromie's avatar Jim Cromie Committed by Greg Kroah-Hartman
Browse files

drm_print: prefer bare printk KERN_DEBUG on generic fn 



drm_print.c calls pr_debug() just once, from __drm_printfn_debug(),
which is a generic/service fn.  The callsite is compile-time enabled
by DEBUG in both DYNAMIC_DEBUG=y/n builds.

For dyndbg builds, reverting this callsite back to bare printk is
correcting a few anti-features:

1- callsite is generic, serves multiple drm users.
   it is soft-wired on currently by #define DEBUG
   could accidentally: #> echo -p > /proc/dynamic_debug/control

2- optional "decorations" by dyndbg are unhelpful/misleading here,
   they describe only the generic site, not end users

IOW, 1,2 are unhelpful at best, and possibly confusing.

reverting yields a nominal data and text shrink:

   text    data     bss     dec     hex filename
 462583   36604   54592 553779   87333 /kernel/drivers/gpu/drm/drm.ko
 462515   36532   54592 553639   872a7 -dirty/kernel/drivers/gpu/drm/drm.ko

Signed-off-by: default avatarJim Cromie <jim.cromie@gmail.com>
Link: https://lore.kernel.org/r/20220912052852.1123868-9-jim.cromie@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 6ce6fae8

drivers/gpu/drm/drm_print.c

+2 −3
Original line number Diff line number Diff line
@@ -23,8 +23,6 @@ 
 * Rob Clark <robdclark@gmail.com>

 */



#define DEBUG /* for pr_debug() */



#include <linux/stdarg.h>



#include <linux/io.h>
@@ -185,7 +183,8 @@ EXPORT_SYMBOL(__drm_printfn_info); 


void __drm_printfn_debug(struct drm_printer *p, struct va_format *vaf)

{

	pr_debug("%s %pV", p->prefix, vaf);

	/* pr_debug callsite decorations are unhelpful here */

	printk(KERN_DEBUG "%s %pV", p->prefix, vaf);

}

EXPORT_SYMBOL(__drm_printfn_debug);