Revert "gallium: build ddebug, noop, rbug, trace as part of auxiliary"

This reverts commit 6f82b8d8d0. This broke scons build, and reportedly clover with autotools/meson too.
2018-02-07 23:47:39 +01:00 · 2018-02-07 23:47:39 +01:00 · 09f49b9e50
parent 6f82b8d8d0
commit 09f49b9e50
14 changed files with 101 additions and 57 deletions
--- 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 \
--- 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

--- 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"
-#include "trace/tr_public.h"
-#include "rbug/rbug_public.h"
-#include "noop/noop_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);
--- 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
--- 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
--- 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
--- 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
--- 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

--- 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)

--- 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) \
--- 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 \
--- 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 \
--- 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 = \
--- 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)