Mesa (lp-binning): llvmpipe: use LP_DBG() macro everywhere

Brian Paul brianp at kemper.freedesktop.org
Fri Dec 4 13:11:45 PST 2009


Module: Mesa
Branch: lp-binning
Commit: a08d6302168341001003da32d42cfcff2311fa04
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a08d6302168341001003da32d42cfcff2311fa04

Author: Brian Paul <brianp at vmware.com>
Date:   Fri Dec  4 14:11:25 2009 -0700

llvmpipe: use LP_DBG() macro everywhere

---

 src/gallium/drivers/llvmpipe/lp_rast.c     |   24 ++++++++--------
 src/gallium/drivers/llvmpipe/lp_rast_tri.c |    3 +-
 src/gallium/drivers/llvmpipe/lp_setup.c    |   40 ++++++++++++++--------------
 3 files changed, 34 insertions(+), 33 deletions(-)

diff --git a/src/gallium/drivers/llvmpipe/lp_rast.c b/src/gallium/drivers/llvmpipe/lp_rast.c
index 8f37a28..5891a2a 100644
--- a/src/gallium/drivers/llvmpipe/lp_rast.c
+++ b/src/gallium/drivers/llvmpipe/lp_rast.c
@@ -27,13 +27,13 @@
 
 #include "util/u_memory.h"
 
+#include "lp_debug.h"
 #include "lp_state.h"
 #include "lp_rast.h"
 #include "lp_rast_priv.h"
 #include "lp_tile_soa.h"
 #include "lp_bld_debug.h"
 
-#define RAST_DEBUG debug_printf
 
 struct lp_rasterizer *lp_rast_create( struct pipe_screen *screen )
 {
@@ -65,7 +65,7 @@ boolean lp_rast_begin( struct lp_rasterizer *rast,
 {
    struct pipe_screen *screen = rast->screen;
 
-   RAST_DEBUG("%s %dx%d\n", __FUNCTION__, width, height);
+   LP_DBG(DEBUG_RAST, "%s %dx%d\n", __FUNCTION__, width, height);
 
    pipe_surface_reference(&rast->state.cbuf, cbuf);
    pipe_surface_reference(&rast->state.zsbuf, zsbuf);
@@ -152,7 +152,7 @@ void lp_rast_start_tile( struct lp_rasterizer *rast,
 			 unsigned x,
 			 unsigned y )
 {
-   RAST_DEBUG("%s %d,%d\n", __FUNCTION__, x, y);
+   LP_DBG(DEBUG_RAST, "%s %d,%d\n", __FUNCTION__, x, y);
 
    rast->x = x;
    rast->y = y;
@@ -168,7 +168,7 @@ void lp_rast_clear_color( struct lp_rasterizer *rast,
 {
    const uint8_t *clear_color = arg.clear_color;
    
-   RAST_DEBUG("%s 0x%x,0x%x,0x%x,0x%x\n", __FUNCTION__, 
+   LP_DBG(DEBUG_RAST, "%s 0x%x,0x%x,0x%x,0x%x\n", __FUNCTION__, 
               clear_color[0],
               clear_color[1],
               clear_color[2],
@@ -198,7 +198,7 @@ void lp_rast_clear_zstencil( struct lp_rasterizer *rast,
 {
    unsigned i, j;
    
-   RAST_DEBUG("%s 0x%x\n", __FUNCTION__, arg.clear_zstencil);
+   LP_DBG(DEBUG_RAST, "%s 0x%x\n", __FUNCTION__, arg.clear_zstencil);
 
    for (i = 0; i < TILE_SIZE; i++)
       for (j = 0; j < TILE_SIZE; j++)
@@ -213,7 +213,7 @@ void lp_rast_clear_zstencil( struct lp_rasterizer *rast,
 void lp_rast_load_color( struct lp_rasterizer *rast,
                          const union lp_rast_cmd_arg arg)
 {
-   RAST_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_RAST, "%s\n", __FUNCTION__);
 
    /* call u_tile func to load colors from surface */
 }
@@ -226,7 +226,7 @@ void lp_rast_load_color( struct lp_rasterizer *rast,
 void lp_rast_load_zstencil( struct lp_rasterizer *rast,
                             const union lp_rast_cmd_arg arg )
 {
-   RAST_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_RAST, "%s\n", __FUNCTION__);
 
    /* call u_tile func to load depth (and stencil?) from surface */
 }
@@ -237,7 +237,7 @@ void lp_rast_set_state( struct lp_rasterizer *rast,
 {
    const struct lp_rast_state *state = arg.set_state;
 
-   RAST_DEBUG("%s %p\n", __FUNCTION__, (void *) state);
+   LP_DBG(DEBUG_RAST, "%s %p\n", __FUNCTION__, (void *) state);
 
    /* just set the current state pointer for this rasterizer */
    rast->current_state = state;
@@ -260,7 +260,7 @@ void lp_rast_shade_tile( struct lp_rasterizer *rast,
    const unsigned mask = ~0;
    unsigned x, y;
 
-   RAST_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_RAST, "%s\n", __FUNCTION__);
 
    /* Use the existing preference for 4x4 (four quads) shading:
     */
@@ -398,7 +398,7 @@ static void lp_rast_store_color( struct lp_rasterizer *rast )
    if (y + h > rast->height)
       h -= y + h - rast->height;
 
-   RAST_DEBUG("%s %d,%d %dx%d\n", __FUNCTION__, x, y, w, h);
+   LP_DBG(DEBUG_RAST, "%s %d,%d %dx%d\n", __FUNCTION__, x, y, w, h);
 
    lp_tile_write_4ub(rast->cbuf_transfer->format,
                      rast->tile.color,
@@ -440,7 +440,7 @@ static void lp_rast_store_zstencil( struct lp_rasterizer *rast )
    if (y + h > rast->height)
       h -= y + h - rast->height;
 
-   RAST_DEBUG("%s %d,%d %dx%d\n", __FUNCTION__, x, y, w, h);
+   LP_DBG(DEBUG_RAST, "%s %d,%d %dx%d\n", __FUNCTION__, x, y, w, h);
 
    assert(rast->zsbuf_transfer->format == PIPE_FORMAT_Z32_UNORM);
    lp_tile_write_z32(rast->tile.depth,
@@ -455,7 +455,7 @@ static void lp_rast_store_zstencil( struct lp_rasterizer *rast )
  */
 void lp_rast_end_tile( struct lp_rasterizer *rast )
 {
-   RAST_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_RAST, "%s\n", __FUNCTION__);
 
    if (rast->state.write_color)
       lp_rast_store_color(rast);
diff --git a/src/gallium/drivers/llvmpipe/lp_rast_tri.c b/src/gallium/drivers/llvmpipe/lp_rast_tri.c
index e772a01..81a9c1c 100644
--- a/src/gallium/drivers/llvmpipe/lp_rast_tri.c
+++ b/src/gallium/drivers/llvmpipe/lp_rast_tri.c
@@ -30,6 +30,7 @@
  */
 
 #include "util/u_math.h"
+#include "lp_debug.h"
 #include "lp_rast_priv.h"
 #include "lp_tile_soa.h"
 
@@ -185,7 +186,7 @@ lp_rast_triangle( struct lp_rasterizer *rast,
 
    assert(Elements(rast->blocks) == (TILE_SIZE * TILE_SIZE) / (4*4));
 
-   debug_printf("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_RAST, "lp_rast_triangle\n");
 
    rast->nr_blocks = 0;
 
diff --git a/src/gallium/drivers/llvmpipe/lp_setup.c b/src/gallium/drivers/llvmpipe/lp_setup.c
index 6136d1b..7f31df6 100644
--- a/src/gallium/drivers/llvmpipe/lp_setup.c
+++ b/src/gallium/drivers/llvmpipe/lp_setup.c
@@ -37,12 +37,12 @@
 #include "util/u_math.h"
 #include "util/u_memory.h"
 #include "util/u_pack_color.h"
+#include "lp_debug.h"
 #include "lp_state.h"
 #include "lp_buffer.h"
 #include "lp_texture.h"
 #include "lp_setup_context.h"
 
-#define SETUP_DEBUG debug_printf
 
 static void set_state( struct setup_context *, unsigned );
 
@@ -82,7 +82,7 @@ static void reset_context( struct setup_context *setup )
 {
    unsigned i, j;
 
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    /* Reset derived state */
    setup->constants.stored_size = 0;
@@ -238,7 +238,7 @@ rasterize_bins( struct setup_context *setup,
    struct lp_rasterizer *rast = setup->rast;
    unsigned i, j;
 
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    lp_rast_begin( rast,
                   setup->fb.cbuf, 
@@ -261,7 +261,7 @@ rasterize_bins( struct setup_context *setup,
 
    reset_context( setup );
 
-   SETUP_DEBUG("%s done \n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s done \n", __FUNCTION__);
 }
 
 
@@ -269,7 +269,7 @@ rasterize_bins( struct setup_context *setup,
 static void
 begin_binning( struct setup_context *setup )
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    if (setup->fb.cbuf) {
       if (setup->clear.flags & PIPE_CLEAR_COLOR)
@@ -289,7 +289,7 @@ begin_binning( struct setup_context *setup )
          bin_everywhere( setup, lp_rast_load_zstencil, lp_rast_arg_null() );
    }
 
-   SETUP_DEBUG("%s done\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s done\n", __FUNCTION__);
 }
 
 
@@ -301,7 +301,7 @@ begin_binning( struct setup_context *setup )
 static void
 execute_clears( struct setup_context *setup )
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    begin_binning( setup );
    rasterize_bins( setup, TRUE );
@@ -317,7 +317,7 @@ set_state( struct setup_context *setup,
    if (old_state == new_state)
       return;
        
-   SETUP_DEBUG("%s old %d new %d\n", __FUNCTION__, old_state, new_state);
+   LP_DBG(DEBUG_SETUP, "%s old %d new %d\n", __FUNCTION__, old_state, new_state);
 
    switch (new_state) {
    case SETUP_ACTIVE:
@@ -347,7 +347,7 @@ void
 lp_setup_flush( struct setup_context *setup,
                 unsigned flags )
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    set_state( setup, SETUP_FLUSHED );
 }
@@ -358,7 +358,7 @@ lp_setup_bind_framebuffer( struct setup_context *setup,
                            struct pipe_surface *color,
                            struct pipe_surface *zstencil )
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    set_state( setup, SETUP_FLUSHED );
 
@@ -401,7 +401,7 @@ lp_setup_clear( struct setup_context *setup,
 {
    unsigned i;
 
-   SETUP_DEBUG("%s state %d\n", __FUNCTION__, setup->state);
+   LP_DBG(DEBUG_SETUP, "%s state %d\n", __FUNCTION__, setup->state);
 
 
    if (flags & PIPE_CLEAR_COLOR) {
@@ -451,7 +451,7 @@ lp_setup_set_triangle_state( struct setup_context *setup,
                              unsigned cull_mode,
                              boolean ccw_is_frontface)
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    setup->ccw_is_frontface = ccw_is_frontface;
    setup->cullmode = cull_mode;
@@ -465,7 +465,7 @@ lp_setup_set_fs_inputs( struct setup_context *setup,
                         const struct lp_shader_input *input,
                         unsigned nr )
 {
-   SETUP_DEBUG("%s %p %u\n", __FUNCTION__, (void *) input, nr);
+   LP_DBG(DEBUG_SETUP, "%s %p %u\n", __FUNCTION__, (void *) input, nr);
 
    memcpy( setup->fs.input, input, nr * sizeof input[0] );
    setup->fs.nr_inputs = nr;
@@ -475,7 +475,7 @@ void
 lp_setup_set_fs( struct setup_context *setup,
                  struct lp_fragment_shader *fs )
 {
-   SETUP_DEBUG("%s %p\n", __FUNCTION__, (void *) fs);
+   LP_DBG(DEBUG_SETUP, "%s %p\n", __FUNCTION__, (void *) fs);
    /* FIXME: reference count */
 
    setup->fs.current.jit_function = fs ? fs->current->jit_function : NULL;
@@ -486,7 +486,7 @@ void
 lp_setup_set_fs_constants(struct setup_context *setup,
                           struct pipe_buffer *buffer)
 {
-   SETUP_DEBUG("%s %p\n", __FUNCTION__, (void *) buffer);
+   LP_DBG(DEBUG_SETUP, "%s %p\n", __FUNCTION__, (void *) buffer);
 
    pipe_buffer_reference(&setup->constants.current, buffer);
 
@@ -498,7 +498,7 @@ void
 lp_setup_set_alpha_ref_value( struct setup_context *setup,
                               float alpha_ref_value )
 {
-   SETUP_DEBUG("%s %f\n", __FUNCTION__, alpha_ref_value);
+   LP_DBG(DEBUG_SETUP, "%s %f\n", __FUNCTION__, alpha_ref_value);
 
    if(setup->fs.current.jit_context.alpha_ref_value != alpha_ref_value) {
       setup->fs.current.jit_context.alpha_ref_value = alpha_ref_value;
@@ -510,7 +510,7 @@ void
 lp_setup_set_blend_color( struct setup_context *setup,
                           const struct pipe_blend_color *blend_color )
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    assert(blend_color);
 
@@ -527,7 +527,7 @@ lp_setup_set_sampler_textures( struct setup_context *setup,
    struct pipe_texture *dummy;
    unsigned i;
 
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
 
    assert(num <= PIPE_MAX_SAMPLERS);
@@ -569,7 +569,7 @@ lp_setup_is_texture_referenced( struct setup_context *setup,
 static INLINE void
 lp_setup_update_shader_state( struct setup_context *setup )
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    assert(setup->fs.current.jit_function);
 
@@ -685,7 +685,7 @@ lp_setup_tri(struct setup_context *setup,
              const float (*v1)[4],
              const float (*v2)[4])
 {
-   SETUP_DEBUG("%s\n", __FUNCTION__);
+   LP_DBG(DEBUG_SETUP, "%s\n", __FUNCTION__);
 
    lp_setup_update_shader_state(setup);
    setup->triangle( setup, v0, v1, v2 );



More information about the mesa-commit mailing list