[Intel-gfx] [PATCH] drm/print: Add drm_dbg_ratelimited

Andi Shyti andi.shyti at linux.intel.com
Mon Oct 9 19:43:55 UTC 2023


Hi John,

> > > From: Nirmoy Das <nirmoy.das at intel.com>
> > > 
> > > Add a function for ratelimitted debug print.
> > > 
> > > Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> > > Cc: Maxime Ripard <mripard at kernel.org>
> > > Cc: Thomas Zimmermann <tzimmermann at suse.de>
> > > Cc: David Airlie <airlied at gmail.com>
> > > Cc: Daniel Vetter <daniel at ffwll.ch>
> > > Reviewed-by: Matthew Auld <matthew.auld at intel.com>
> > > Reviewed-by: Andi Shyti <andi.shyti at linux.intel.com>
> > > Signed-off-by: Nirmoy Das <nirmoy.das at intel.com>
> > > Signed-off-by: Andi Shyti <andi.shyti at linux.intel.com>
> > Just a kind reminder!
> > 
> > This is the second time this patch has been sent and we have seen
> > some potential use of the drm_dbg_ratelimited().
> But this patch does not actually add a user. So it is dead code at this
> point, which is not allowed.
> 
> If you have code that wants to use such a helper then the helper should be
> part of the patch that adds that code.

this is the kind of "Argh! If I had it!" patch. At the 3/4th
"Argh! If I had it!" then you decide to send it but don't
remember for which case you needed it.

I'm sure that once this goes in won't be long until poeple will
start using it. In any case, if it doesn't go in I will keep it
ready until the case comes.

Andi


More information about the dri-devel mailing list