6f1db5f37e
We already enable -DDEBUG unconditionally (meaning regardless of this is a debug build or a performance build) for zdb and ztest as they are mostly used for development and debugging. This patch enables -DDEBUG for libzpool extending the debugging checks for zdb, ztest, and a couple of other test utilities. In addition to passing -DDEBUG we also enable -DZFS_DEBUG so all assertion checks work s expected. We do so not only in libzpool but in every utility that links to it, even if the utility doesn't directly use any functionality wrapped in ZFS_DEBUG macro definitions. The reason is that these utilities may still include headers that contain structs that have more fields when ZFS_DEBUG is defined. This can be a problem as enabling that flag for libzpool but not for zdb can lead into random problems (e.g. segmentation faults) as zdb may be have an incorrect view of a struct passed to it by libzpool. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Reviewed-by: Matthew Ahrens <mahrens@delphix.com> Signed-off-by: Serapheim Dimitropoulos <serapheim@delphix.com> Closes #10549
24 lines
595 B
Makefile
24 lines
595 B
Makefile
include $(top_srcdir)/config/Rules.am
|
|
|
|
# Get rid of compiler warning for unchecked truncating snprintfs on gcc 7.1.1
|
|
AM_CFLAGS += $(NO_FORMAT_TRUNCATION)
|
|
|
|
# Includes kernel code, generate warnings for large stack frames
|
|
AM_CFLAGS += $(FRAME_LARGER_THAN)
|
|
|
|
# Unconditionally enable ASSERTs
|
|
AM_CPPFLAGS += -DDEBUG -UNDEBUG -DZFS_DEBUG
|
|
|
|
sbin_PROGRAMS = ztest
|
|
|
|
ztest_SOURCES = \
|
|
ztest.c
|
|
|
|
ztest_LDADD = \
|
|
$(abs_top_builddir)/lib/libzpool/libzpool.la \
|
|
$(abs_top_builddir)/lib/libzfs_core/libzfs_core.la \
|
|
$(abs_top_builddir)/lib/libnvpair/libnvpair.la
|
|
|
|
ztest_LDADD += -lm
|
|
ztest_LDFLAGS = -pthread
|