[Mesa-dev] [PATCH] Rename the DEBUG macro to MESA_DEBUG

Rob Clark robdclark at gmail.com
Fri Jul 22 01:35:44 UTC 2016


On Thu, Jul 21, 2016 at 1:48 PM, Vedran Miletić <vedran at miletic.net> wrote:
> LLVM and Mesa both define the DEBUG macro in incompatible ways. As a
> general practice, we should avoid using such generic names when it is
> possible to do so.
>
> This patch renames all occurrences of the DEBUG macro to MESA_DEBUG,
> and removes workarounds previously used to enable building Mesa with
> LLVM (pop_macro() and push_macro() function calls).
>
> Please let me know if I missed any.

I guess at least some in-flight patches (at least my
pipe_mutex_assert_locked() patch, but I guess DEBUG is common enough
that it might effect others).. not sure if there is a better way to
deal with that without things falling through the cracks..  maybe
introduce MESA_DEBUG which is the same as DEBUG first, and then a
later patch to remove DEBUG.  Or at least including sed/etc rule to
re-do the mass-change on a later baseline in the commit msg?

I don't mind rebasing my patch, just more worried about things falling
through the cracks with other in-progress stuff, since it seems like
the end result would be a silent fail to enable intended debug code..

BR,
-R

> Signed-off-by: Vedran Miletić <vedran at miletic.net>
> ---
>  configure.ac                                       |  2 +-
>  src/compiler/glsl/ir_validate.cpp                  |  4 +-
>  src/compiler/nir/nir.h                             |  6 +-
>  src/compiler/nir/nir_metadata.c                    |  4 +-
>  src/compiler/nir/nir_validate.c                    |  5 +-
>  src/egl/drivers/haiku/egl_haiku.cpp                |  6 +-
>  src/egl/main/eglconfig.c                           |  6 +-
>  src/gallium/auxiliary/draw/draw_cliptest_tmp.h     |  4 +-
>  src/gallium/auxiliary/gallivm/lp_bld_debug.h       | 12 ++--
>  src/gallium/auxiliary/gallivm/lp_bld_init.c        | 16 +++---
>  src/gallium/auxiliary/gallivm/lp_bld_misc.cpp      | 23 ++------
>  src/gallium/auxiliary/gallivm/lp_bld_struct.c      | 16 +++---
>  src/gallium/auxiliary/os/os_memory.h               |  6 +-
>  src/gallium/auxiliary/os/os_misc.c                 |  4 +-
>  src/gallium/auxiliary/os/os_misc.h                 |  6 +-
>  .../auxiliary/pipebuffer/pb_buffer_fenced.c        | 10 ++--
>  src/gallium/auxiliary/pipebuffer/pb_bufmgr_debug.c |  6 +-
>  src/gallium/auxiliary/tgsi/tgsi_exec.c             | 16 +++---
>  src/gallium/auxiliary/tgsi/tgsi_ureg.c             |  8 +--
>  src/gallium/auxiliary/util/u_cache.c               | 16 +++---
>  src/gallium/auxiliary/util/u_cpu_detect.c          |  8 +--
>  src/gallium/auxiliary/util/u_debug.c               | 18 +++---
>  src/gallium/auxiliary/util/u_debug.h               | 66 +++++++++++-----------
>  src/gallium/auxiliary/util/u_debug_flush.c         |  4 +-
>  src/gallium/auxiliary/util/u_debug_flush.h         |  6 +-
>  src/gallium/auxiliary/util/u_debug_image.c         |  4 +-
>  src/gallium/auxiliary/util/u_debug_image.h         |  8 +--
>  src/gallium/drivers/freedreno/ir3/ir3.c            | 16 +++---
>  src/gallium/drivers/freedreno/ir3/ir3.h            | 18 +++---
>  src/gallium/drivers/freedreno/ir3/ir3_print.c      |  4 +-
>  src/gallium/drivers/freedreno/ir3/ir3_ra.c         |  4 +-
>  src/gallium/drivers/i915/i915_debug.c              |  6 +-
>  src/gallium/drivers/i915/i915_debug.h              |  6 +-
>  src/gallium/drivers/ilo/core/ilo_debug.h           | 17 +++---
>  src/gallium/drivers/llvmpipe/lp_debug.h            |  6 +-
>  src/gallium/drivers/llvmpipe/lp_perf.h             |  6 +-
>  src/gallium/drivers/llvmpipe/lp_rast.c             |  4 +-
>  src/gallium/drivers/llvmpipe/lp_rast.h             |  4 +-
>  src/gallium/drivers/llvmpipe/lp_rast_priv.h        |  6 +-
>  src/gallium/drivers/llvmpipe/lp_scene.c            |  4 +-
>  src/gallium/drivers/llvmpipe/lp_screen.c           |  8 +--
>  src/gallium/drivers/llvmpipe/lp_setup_line.c       |  4 +-
>  src/gallium/drivers/llvmpipe/lp_setup_point.c      |  4 +-
>  src/gallium/drivers/llvmpipe/lp_state_sampler.c    |  4 +-
>  src/gallium/drivers/llvmpipe/lp_test_main.c        |  4 +-
>  src/gallium/drivers/llvmpipe/lp_texture.c          | 24 ++++----
>  src/gallium/drivers/llvmpipe/lp_texture.h          |  4 +-
>  .../drivers/nouveau/codegen/nv50_ir_driver.h       |  6 +-
>  .../drivers/nouveau/codegen/nv50_ir_inlines.h      |  4 +-
>  src/gallium/drivers/nouveau/nouveau_screen.h       |  4 +-
>  src/gallium/drivers/nouveau/nouveau_statebuf.h     | 10 ++--
>  src/gallium/drivers/nouveau/nv50/nv50_program.c    |  6 +-
>  src/gallium/drivers/nouveau/nvc0/nvc0_program.c    | 14 ++---
>  src/gallium/drivers/nouveau/nvc0/nve4_compute.c    | 12 ++--
>  src/gallium/drivers/r300/r300_cb.h                 |  6 +-
>  src/gallium/drivers/r300/r300_context.c            |  6 +-
>  src/gallium/drivers/r300/r300_cs.h                 |  6 +-
>  src/gallium/drivers/softpipe/sp_tex_sample.c       |  4 +-
>  src/gallium/drivers/svga/svga_debug.h              |  8 +--
>  src/gallium/drivers/svga/svga_draw.c               |  6 +-
>  src/gallium/drivers/svga/svga_format.c             |  6 +-
>  src/gallium/drivers/svga/svga_pipe_draw.c          |  4 +-
>  .../drivers/svga/svga_resource_buffer_upload.c     |  4 +-
>  src/gallium/drivers/svga/svga_screen.c             | 18 +++---
>  src/gallium/drivers/svga/svga_screen.h             |  6 +-
>  src/gallium/drivers/svga/svga_state.c              |  6 +-
>  src/gallium/drivers/svga/svga_state_constants.c    |  4 +-
>  src/gallium/drivers/svga/svga_state_fs.c           | 10 ++--
>  .../drivers/swr/rasterizer/jitter/JitManager.cpp   |  5 --
>  .../drivers/swr/rasterizer/jitter/JitManager.h     |  6 --
>  src/gallium/drivers/swr/swr_shader.cpp             |  4 --
>  src/gallium/drivers/swr/swr_state.cpp              |  4 --
>  src/gallium/drivers/trace/tr_context.h             |  4 +-
>  src/gallium/drivers/vc4/vc4_cl.h                   | 20 +++----
>  src/gallium/drivers/vc4/vc4_qir_validate.c         |  4 +-
>  src/gallium/drivers/vc4/vc4_qpu_validate.c         |  4 +-
>  src/gallium/state_trackers/glx/xlib/glx_usefont.c  | 16 +++---
>  src/gallium/state_trackers/hgl/hgl.c               |  6 +-
>  src/gallium/state_trackers/nine/basetexture9.c     |  8 +--
>  src/gallium/state_trackers/nine/basetexture9.h     |  6 +-
>  src/gallium/state_trackers/nine/nine_debug.h       | 30 +++++-----
>  src/gallium/state_trackers/nine/nine_dump.c        |  4 +-
>  src/gallium/state_trackers/nine/nine_dump.h        |  4 +-
>  src/gallium/state_trackers/nine/nine_ff.c          |  4 +-
>  src/gallium/state_trackers/nine/surface9.c         |  4 +-
>  src/gallium/state_trackers/nine/surface9.h         |  4 +-
>  src/gallium/state_trackers/nine/volume9.c          |  4 +-
>  src/gallium/state_trackers/wgl/stw_device.c        |  8 +--
>  src/gallium/state_trackers/wgl/stw_device.h        |  4 +-
>  src/gallium/state_trackers/wgl/stw_ext_pbuffer.c   |  4 +-
>  .../targets/haiku-softpipe/GalliumContext.cpp      |  6 +-
>  .../targets/haiku-softpipe/SoftwareRenderer.cpp    |  6 +-
>  src/gallium/winsys/i915/drm/i915_drm_buffer.c      |  4 +-
>  .../winsys/svga/drm/pb_buffer_simple_fenced.c      | 10 ++--
>  src/gallium/winsys/svga/drm/vmw_buffer.c           | 16 +++---
>  src/gallium/winsys/svga/drm/vmw_buffer.h           |  6 +-
>  src/gallium/winsys/svga/drm/vmw_context.c          | 32 +++++------
>  src/gallium/winsys/svga/drm/vmw_screen_ioctl.c     |  4 +-
>  src/gallium/winsys/svga/drm/vmw_screen_pools.c     |  4 +-
>  src/gallium/winsys/svga/drm/vmw_shader.c           |  4 +-
>  src/gallium/winsys/svga/drm/vmw_surface.c          |  4 +-
>  src/gallium/winsys/sw/hgl/hgl_sw_winsys.c          |  6 +-
>  src/gallium/winsys/sw/kms-dri/kms_dri_sw_winsys.c  |  6 +-
>  src/glx/glxext.c                                   |  8 +--
>  src/glx/xfont.c                                    | 18 +++---
>  src/intel/vulkan/anv_entrypoints_gen.py            |  6 +-
>  src/intel/vulkan/anv_private.h                     | 18 +++---
>  src/mapi/mapi_abi.py                               |  6 +-
>  src/mesa/drivers/dri/i915/intel_batchbuffer.h      | 18 +++---
>  src/mesa/drivers/dri/i965/brw_context.h            |  4 +-
>  src/mesa/drivers/dri/i965/brw_disasm.c             |  4 +-
>  .../drivers/dri/i965/brw_performance_monitor.c     |  8 +--
>  src/mesa/drivers/dri/i965/intel_batchbuffer.c      |  4 +-
>  src/mesa/drivers/dri/i965/intel_batchbuffer.h      | 14 ++---
>  src/mesa/drivers/dri/radeon/radeon_debug.h         |  6 +-
>  src/mesa/drivers/x11/xfonts.c                      | 16 +++---
>  src/mesa/drivers/x11/xm_tri.c                      | 10 ++--
>  src/mesa/main/context.c                            | 12 ++--
>  src/mesa/main/debug.c                              |  8 +--
>  src/mesa/main/errors.c                             | 22 ++++----
>  src/mesa/main/feedback.c                           |  4 +-
>  src/mesa/main/formats.c                            |  4 +-
>  src/mesa/main/imports.c                            |  8 +--
>  src/mesa/main/macros.h                             |  6 +-
>  src/mesa/main/mtypes.h                             |  6 +-
>  src/mesa/main/shaderapi.c                          |  8 +--
>  src/mesa/main/teximage.c                           |  4 +-
>  src/mesa/program/prog_optimize.c                   |  8 +--
>  src/mesa/sparc/sparc.c                             |  4 +-
>  src/mesa/state_tracker/st_atom.c                   | 12 +---
>  src/mesa/state_tracker/st_atom_framebuffer.c       |  4 +-
>  src/mesa/state_tracker/st_debug.c                  |  8 +--
>  src/mesa/state_tracker/st_debug.h                  |  6 +-
>  src/mesa/state_tracker/st_format.c                 |  6 +-
>  src/mesa/swrast/s_lines.c                          |  6 +-
>  src/mesa/swrast/s_span.c                           |  4 +-
>  src/mesa/swrast/s_texfetch.c                       |  4 +-
>  src/mesa/swrast/s_triangle.c                       |  6 +-
>  src/mesa/vbo/vbo_exec.h                            |  4 +-
>  src/mesa/vbo/vbo_exec_api.c                        | 12 ++--
>  src/mesa/x86-64/x86-64.c                           |  4 +-
>  src/util/ralloc.c                                  | 12 ++--
>  142 files changed, 569 insertions(+), 611 deletions(-)


More information about the mesa-dev mailing list