From 5c67820265c18e82c2d79c77ab09fc46904b58b8 Mon Sep 17 00:00:00 2001 From: Rob Norris Date: Tue, 10 Sep 2024 07:13:27 +1000 Subject: [PATCH] libzstd: also build with LIBZPOOL_CPPFLAGS libzstd now also allocates its own abd_t, and so has the same issue as zstream did, so this applies the same workaround: compile it with ZFS_DEBUG. See 92fca1c2d. This looks weird, because libzstd doesn't appear to look related to the ZFS kernel, but there is already a cross-dependency there: zstd needs zfs_lz4_compress, and zfs needs zfs_zstd_compress (and others), so the two can never really be separated without more work. Another job for another time. Sponsored-by: Klara, Inc. Sponsored-by: Wasabi Technology, Inc. Reviewed by: Brian Behlendorf Reviewed-by: Mark Maybee Signed-off-by: Rob Norris Closes #16489 --- lib/libzstd/Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/libzstd/Makefile.am b/lib/libzstd/Makefile.am index 49bfb328a6..8561751379 100644 --- a/lib/libzstd/Makefile.am +++ b/lib/libzstd/Makefile.am @@ -1,4 +1,6 @@ libzstd_la_CFLAGS = $(AM_CFLAGS) $(LIBRARY_CFLAGS) +libzstd_la_CPPFLAGS = $(AM_CPPFLAGS) $(LIBZPOOL_CPPFLAGS) + # -fno-tree-vectorize is set for gcc in zstd/common/compiler.h # Set it for other compilers, too. libzstd_la_CFLAGS += -fno-tree-vectorize