Module: Mesa Branch: master Commit: 09f49b9e50ee86e24c7e975f08535aa7237852a1 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=09f49b9e50ee86e24c7e975f08535aa7237852a1
Author: Roland Scheidegger <srol...@vmware.com> Date: Wed Feb 7 23:47:39 2018 +0100 Revert "gallium: build ddebug, noop, rbug, trace as part of auxiliary" This reverts commit 6f82b8d8d0a986aac28e7bec47fc313fb950475c. This broke scons build, and reportedly clover with autotools/meson too. --- src/gallium/Makefile.am | 12 +++++----- src/gallium/auxiliary/Makefile.am | 10 +-------- .../auxiliary/target-helpers/inline_debug_helper.h | 26 ++++++++++++++++++++++ src/gallium/drivers/ddebug/Makefile.sources | 14 ++++++------ src/gallium/drivers/noop/Makefile.sources | 8 +++---- src/gallium/drivers/rbug/Makefile.sources | 18 +++++++-------- src/gallium/drivers/trace/Makefile.sources | 26 +++++++++++----------- src/gallium/state_trackers/osmesa/Makefile.am | 3 ++- src/gallium/targets/d3dadapter9/Makefile.am | 8 ++++++- src/gallium/targets/dri/Makefile.am | 10 ++++++++- src/gallium/targets/libgl-xlib/Makefile.am | 6 ++++- src/gallium/targets/osmesa/Makefile.am | 4 +++- src/gallium/targets/pipe-loader/Makefile.am | 6 ++++- src/gallium/tests/unit/Makefile.am | 1 + 14 files changed, 98 insertions(+), 54 deletions(-) diff --git a/src/gallium/Makefile.am b/src/gallium/Makefile.am index 81eabef106..af010c89f8 100644 --- a/src/gallium/Makefile.am +++ b/src/gallium/Makefile.am @@ -11,6 +11,12 @@ SUBDIRS += auxiliary/pipe-loader ## Gallium pipe drivers and their respective winsys' ## +SUBDIRS += \ + drivers/ddebug \ + drivers/noop \ + drivers/trace \ + drivers/rbug + ## freedreno/msm/kgsl if HAVE_GALLIUM_FREEDRENO SUBDIRS += drivers/freedreno winsys/freedreno/drm @@ -182,12 +188,6 @@ endif EXTRA_DIST += \ include \ - drivers/noop/SConscript \ - drivers/rbug/README \ - drivers/rbug/SConscript \ - drivers/trace/trace.xsl \ - drivers/trace/README \ - drivers/trace/SConscript \ state_trackers/README \ state_trackers/wgl targets/libgl-gdi \ targets/graw-gdi targets/graw-null targets/graw-xlib \ diff --git a/src/gallium/auxiliary/Makefile.am b/src/gallium/auxiliary/Makefile.am index 7af3f3ce42..95a325f96b 100644 --- a/src/gallium/auxiliary/Makefile.am +++ b/src/gallium/auxiliary/Makefile.am @@ -1,8 +1,4 @@ include Makefile.sources -include $(top_srcdir)/src/gallium/drivers/ddebug/Makefile.sources -include $(top_srcdir)/src/gallium/drivers/noop/Makefile.sources -include $(top_srcdir)/src/gallium/drivers/rbug/Makefile.sources -include $(top_srcdir)/src/gallium/drivers/trace/Makefile.sources include $(top_srcdir)/src/gallium/Automake.inc noinst_LTLIBRARIES = libgallium.la @@ -23,11 +19,7 @@ AM_CXXFLAGS = \ libgallium_la_SOURCES = \ $(C_SOURCES) \ $(NIR_SOURCES) \ - $(GENERATED_SOURCES) \ - $(DDEBUG_SOURCES) \ - $(NOOP_SOURCES) \ - $(RBUG_SOURCES) \ - $(TRACE_SOURCES) + $(GENERATED_SOURCES) if HAVE_LIBDRM diff --git a/src/gallium/auxiliary/target-helpers/inline_debug_helper.h b/src/gallium/auxiliary/target-helpers/inline_debug_helper.h index 8556376940..2443bf2146 100644 --- a/src/gallium/auxiliary/target-helpers/inline_debug_helper.h +++ b/src/gallium/auxiliary/target-helpers/inline_debug_helper.h @@ -11,18 +11,44 @@ * one or more debug driver: rbug, trace. */ +#ifdef GALLIUM_DDEBUG #include "ddebug/dd_public.h" +#endif + +#ifdef GALLIUM_TRACE #include "trace/tr_public.h" +#endif + +#ifdef GALLIUM_RBUG #include "rbug/rbug_public.h" +#endif + +#ifdef GALLIUM_NOOP #include "noop/noop_public.h" +#endif +/* + * TODO: Audit the following *screen_create() - all of + * them should return the original screen on failuire. + */ static inline struct pipe_screen * debug_screen_wrap(struct pipe_screen *screen) { +#if defined(GALLIUM_DDEBUG) screen = ddebug_screen_create(screen); +#endif + +#if defined(GALLIUM_RBUG) screen = rbug_screen_create(screen); +#endif + +#if defined(GALLIUM_TRACE) screen = trace_screen_create(screen); +#endif + +#if defined(GALLIUM_NOOP) screen = noop_screen_create(screen); +#endif if (debug_get_bool_option("GALLIUM_TESTS", FALSE)) util_run_tests(screen); diff --git a/src/gallium/drivers/ddebug/Makefile.sources b/src/gallium/drivers/ddebug/Makefile.sources index d43a75ba40..1bd38274df 100644 --- a/src/gallium/drivers/ddebug/Makefile.sources +++ b/src/gallium/drivers/ddebug/Makefile.sources @@ -1,7 +1,7 @@ -DDEBUG_SOURCES := \ - $(top_srcdir)/src/gallium/drivers/ddebug/dd_context.c \ - $(top_srcdir)/src/gallium/drivers/ddebug/dd_draw.c \ - $(top_srcdir)/src/gallium/drivers/ddebug/dd_pipe.h \ - $(top_srcdir)/src/gallium/drivers/ddebug/dd_public.h \ - $(top_srcdir)/src/gallium/drivers/ddebug/dd_screen.c \ - $(top_srcdir)/src/gallium/drivers/ddebug/dd_util.h +C_SOURCES := \ + dd_context.c \ + dd_draw.c \ + dd_pipe.h \ + dd_public.h \ + dd_screen.c \ + dd_util.h diff --git a/src/gallium/drivers/noop/Makefile.sources b/src/gallium/drivers/noop/Makefile.sources index fea96e31e5..6d4228630b 100644 --- a/src/gallium/drivers/noop/Makefile.sources +++ b/src/gallium/drivers/noop/Makefile.sources @@ -1,4 +1,4 @@ -NOOP_SOURCES := \ - $(top_srcdir)/src/gallium/drivers/noop/noop_pipe.c \ - $(top_srcdir)/src/gallium/drivers/noop/noop_public.h \ - $(top_srcdir)/src/gallium/drivers/noop/noop_state.c +C_SOURCES := \ + noop_pipe.c \ + noop_public.h \ + noop_state.c diff --git a/src/gallium/drivers/rbug/Makefile.sources b/src/gallium/drivers/rbug/Makefile.sources index b21e51e309..f9f1d5c3a4 100644 --- a/src/gallium/drivers/rbug/Makefile.sources +++ b/src/gallium/drivers/rbug/Makefile.sources @@ -1,9 +1,9 @@ -RBUG_SOURCES := \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_context.c \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_context.h \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_core.c \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_objects.c \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_objects.h \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_public.h \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_screen.c \ - $(top_srcdir)/src/gallium/drivers/rbug/rbug_screen.h +C_SOURCES := \ + rbug_context.c \ + rbug_context.h \ + rbug_core.c \ + rbug_objects.c \ + rbug_objects.h \ + rbug_public.h \ + rbug_screen.c \ + rbug_screen.h diff --git a/src/gallium/drivers/trace/Makefile.sources b/src/gallium/drivers/trace/Makefile.sources index f3451d4c14..4c019a3bc6 100644 --- a/src/gallium/drivers/trace/Makefile.sources +++ b/src/gallium/drivers/trace/Makefile.sources @@ -1,13 +1,13 @@ -TRACE_SOURCES := \ - $(top_srcdir)/src/gallium/drivers/trace/tr_context.c \ - $(top_srcdir)/src/gallium/drivers/trace/tr_context.h \ - $(top_srcdir)/src/gallium/drivers/trace/tr_dump.c \ - $(top_srcdir)/src/gallium/drivers/trace/tr_dump_defines.h \ - $(top_srcdir)/src/gallium/drivers/trace/tr_dump.h \ - $(top_srcdir)/src/gallium/drivers/trace/tr_dump_state.c \ - $(top_srcdir)/src/gallium/drivers/trace/tr_dump_state.h \ - $(top_srcdir)/src/gallium/drivers/trace/tr_public.h \ - $(top_srcdir)/src/gallium/drivers/trace/tr_screen.c \ - $(top_srcdir)/src/gallium/drivers/trace/tr_screen.h \ - $(top_srcdir)/src/gallium/drivers/trace/tr_texture.c \ - $(top_srcdir)/src/gallium/drivers/trace/tr_texture.h +C_SOURCES := \ + tr_context.c \ + tr_context.h \ + tr_dump.c \ + tr_dump_defines.h \ + tr_dump.h \ + tr_dump_state.c \ + tr_dump_state.h \ + tr_public.h \ + tr_screen.c \ + tr_screen.h \ + tr_texture.c \ + tr_texture.h diff --git a/src/gallium/state_trackers/osmesa/Makefile.am b/src/gallium/state_trackers/osmesa/Makefile.am index 05e3ca0d73..e5f2a5e18a 100644 --- a/src/gallium/state_trackers/osmesa/Makefile.am +++ b/src/gallium/state_trackers/osmesa/Makefile.am @@ -33,7 +33,8 @@ AM_CPPFLAGS = \ -I$(top_srcdir)/src/gallium/winsys \ -I$(top_srcdir)/src/gallium/state_trackers/glx/xlib \ -I$(top_srcdir)/src/gallium/auxiliary \ - -DGALLIUM_SOFTPIPE + -DGALLIUM_SOFTPIPE \ + -DGALLIUM_TRACE noinst_LTLIBRARIES = libosmesa.la diff --git a/src/gallium/targets/d3dadapter9/Makefile.am b/src/gallium/targets/d3dadapter9/Makefile.am index 69cebbcd16..c2aae0c780 100644 --- a/src/gallium/targets/d3dadapter9/Makefile.am +++ b/src/gallium/targets/d3dadapter9/Makefile.am @@ -35,7 +35,10 @@ AM_CFLAGS = \ $(VISIBILITY_CFLAGS) AM_CPPFLAGS = \ - $(DEFINES) + $(DEFINES) \ + -DGALLIUM_DDEBUG \ + -DGALLIUM_RBUG \ + -DGALLIUM_TRACE ninedir = $(D3D_DRIVER_INSTALL_DIR) nine_LTLIBRARIES = d3dadapter9.la @@ -67,6 +70,9 @@ d3dadapter9_la_LIBADD = \ $(top_builddir)/src/gallium/auxiliary/libgallium.la \ $(top_builddir)/src/gallium/state_trackers/nine/libninetracker.la \ $(top_builddir)/src/util/libmesautil.la \ + $(top_builddir)/src/gallium/drivers/ddebug/libddebug.la \ + $(top_builddir)/src/gallium/drivers/rbug/librbug.la \ + $(top_builddir)/src/gallium/drivers/trace/libtrace.la \ $(EXPAT_LIBS) \ $(GALLIUM_COMMON_LIB_DEPS) diff --git a/src/gallium/targets/dri/Makefile.am b/src/gallium/targets/dri/Makefile.am index 9968828518..1d05d91a61 100644 --- a/src/gallium/targets/dri/Makefile.am +++ b/src/gallium/targets/dri/Makefile.am @@ -9,7 +9,11 @@ AM_CFLAGS = \ $(GALLIUM_TARGET_CFLAGS) AM_CPPFLAGS = \ - $(DEFINES) + $(DEFINES) \ + -DGALLIUM_DDEBUG \ + -DGALLIUM_NOOP \ + -DGALLIUM_RBUG \ + -DGALLIUM_TRACE dridir = $(DRI_DRIVER_INSTALL_DIR) dri_LTLIBRARIES = gallium_dri.la @@ -43,6 +47,10 @@ gallium_dri_la_LIBADD = \ $(top_builddir)/src/gallium/state_trackers/dri/libdri.la \ $(top_builddir)/src/gallium/auxiliary/libgalliumvl.la \ $(top_builddir)/src/gallium/auxiliary/libgallium.la \ + $(top_builddir)/src/gallium/drivers/ddebug/libddebug.la \ + $(top_builddir)/src/gallium/drivers/noop/libnoop.la \ + $(top_builddir)/src/gallium/drivers/rbug/librbug.la \ + $(top_builddir)/src/gallium/drivers/trace/libtrace.la \ $(top_builddir)/src/mapi/shared-glapi/libglapi.la \ $(SELINUX_LIBS) \ $(EXPAT_LIBS) \ diff --git a/src/gallium/targets/libgl-xlib/Makefile.am b/src/gallium/targets/libgl-xlib/Makefile.am index 56d548e7c1..1c622946a9 100644 --- a/src/gallium/targets/libgl-xlib/Makefile.am +++ b/src/gallium/targets/libgl-xlib/Makefile.am @@ -35,7 +35,9 @@ AM_CPPFLAGS = \ -I$(top_srcdir)/src/gallium/state_trackers/glx/xlib \ -I$(top_srcdir)/src/gallium/auxiliary \ -I$(top_srcdir)/src/gallium/winsys \ - -DGALLIUM_SOFTPIPE + -DGALLIUM_SOFTPIPE \ + -DGALLIUM_RBUG \ + -DGALLIUM_TRACE AM_CFLAGS = $(X11_INCLUDES) @@ -59,6 +61,8 @@ lib@GL_LIB@_la_LIBADD = \ $(top_builddir)/src/gallium/state_trackers/glx/xlib/libxlib.la \ $(top_builddir)/src/gallium/winsys/sw/xlib/libws_xlib.la \ $(top_builddir)/src/gallium/drivers/softpipe/libsoftpipe.la \ + $(top_builddir)/src/gallium/drivers/trace/libtrace.la \ + $(top_builddir)/src/gallium/drivers/rbug/librbug.la \ $(top_builddir)/src/mapi/glapi/libglapi.la \ $(top_builddir)/src/mesa/libmesagallium.la \ $(top_builddir)/src/gallium/auxiliary/libgallium.la \ diff --git a/src/gallium/targets/osmesa/Makefile.am b/src/gallium/targets/osmesa/Makefile.am index dc4d49e107..f8bee5ef67 100644 --- a/src/gallium/targets/osmesa/Makefile.am +++ b/src/gallium/targets/osmesa/Makefile.am @@ -33,7 +33,8 @@ AM_CPPFLAGS = \ -I$(top_srcdir)/src/gallium/drivers \ -I$(top_srcdir)/src/gallium/winsys \ -I$(top_srcdir)/src/gallium/auxiliary \ - -DGALLIUM_SOFTPIPE + -DGALLIUM_SOFTPIPE \ + -DGALLIUM_TRACE lib_LTLIBRARIES = lib@OSMESA_LIB@.la @@ -59,6 +60,7 @@ lib@OSMESA_LIB@_la_LIBADD = \ $(top_builddir)/src/mesa/libmesagallium.la \ $(top_builddir)/src/gallium/auxiliary/libgallium.la \ $(top_builddir)/src/gallium/winsys/sw/null/libws_null.la \ + $(top_builddir)/src/gallium/drivers/trace/libtrace.la \ $(top_builddir)/src/gallium/drivers/softpipe/libsoftpipe.la \ $(top_builddir)/src/gallium/state_trackers/osmesa/libosmesa.la \ $(top_builddir)/src/mapi/glapi/libglapi.la \ diff --git a/src/gallium/targets/pipe-loader/Makefile.am b/src/gallium/targets/pipe-loader/Makefile.am index 3c5010c669..4b84886f30 100644 --- a/src/gallium/targets/pipe-loader/Makefile.am +++ b/src/gallium/targets/pipe-loader/Makefile.am @@ -31,7 +31,9 @@ AM_CPPFLAGS = \ -I$(top_builddir)/src/util \ $(GALLIUM_PIPE_LOADER_DEFINES) \ $(LIBDRM_CFLAGS) \ - $(VISIBILITY_CFLAGS) + $(VISIBILITY_CFLAGS) \ + -DGALLIUM_RBUG \ + -DGALLIUM_TRACE pipedir = $(libdir)/gallium-pipe pipe_LTLIBRARIES = @@ -55,6 +57,8 @@ PIPE_LIBS += \ $(top_builddir)/src/gallium/auxiliary/libgallium.la \ $(top_builddir)/src/compiler/nir/libnir.la \ $(top_builddir)/src/util/libmesautil.la \ + $(top_builddir)/src/gallium/drivers/rbug/librbug.la \ + $(top_builddir)/src/gallium/drivers/trace/libtrace.la \ $(GALLIUM_COMMON_LIB_DEPS) AM_LDFLAGS = \ diff --git a/src/gallium/tests/unit/Makefile.am b/src/gallium/tests/unit/Makefile.am index 9f1d3b98cd..c9bede75be 100644 --- a/src/gallium/tests/unit/Makefile.am +++ b/src/gallium/tests/unit/Makefile.am @@ -14,6 +14,7 @@ AM_CPPFLAGS = \ LDADD = \ $(top_builddir)/src/gallium/auxiliary/libgallium.la \ $(top_builddir)/src/util/libmesautil.la \ + $(top_builddir)/src/gallium/drivers/trace/libtrace.la \ $(top_builddir)/src/gallium/winsys/sw/null/libws_null.la \ $(top_builddir)/src/gallium/drivers/softpipe/libsoftpipe.la \ $(GALLIUM_COMMON_LIB_DEPS) _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-commit