drm: Pull the test on drm_debug in the logging macros
authorLespiau, Damien <damien.lespiau@intel.com>
Mon, 24 Mar 2014 15:53:15 +0000 (15:53 +0000)
committerDave Airlie <airlied@redhat.com>
Fri, 28 Mar 2014 02:57:36 +0000 (12:57 +1000)
commita73d4e91fbb5ed6821ec5b906028e0e94868ef79
tree23c53972b71fc5dae0ed7e28a45b23ff3d928c19
parent8fa6a9e7e23ff163235d86b4d6f30692092dd1b5
drm: Pull the test on drm_debug in the logging macros

In the logging code, we are currently checking is we need to output in
drm_ut_debug_printk(). This is too late. The problem is that when we write
something like:

    DRM_DEBUG_DRIVER("ELD on [CONNECTOR:%d:%s], [ENCODER:%d:%s]\n",
                     connector->base.id,
                     drm_get_connector_name(connector),
                     connector->encoder->base.id,
                     drm_get_encoder_name(connector->encoder));

We start by evaluating the arguments (so call drm_get_connector_name() and
drm_get_connector_name()) before ending up in drm_ut_debug_printk() which will
then does nothing.

This means we execute a lot of instructions (drm_get_connector_name(), in turn,
calls snprintf() for example) to happily discard them in the normal case,
drm.debug=0.

So, let's put the test on drm_debug earlier, in the macros themselves.
Sprinkle an unlikely() as well for good measure.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_stub.c
include/drm/drmP.h