lkml.org 
[lkml]   [2020]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.9 417/757] tools feature: Add missing -lzstd to the fast path feature detection
    Date
    From: Arnaldo Carvalho de Melo <acme@redhat.com>

    [ Upstream commit 6c014694b1d2702cdc736d17b60746e7b95ba664 ]

    We were failing that due to GTK2+ and then for the ZSTD test, which made
    test-all.c, the fast path feature detection file to fail and thus
    trigger building all of the feature tests, slowing down the test.

    Eventually the ZSTD test would be built and would succeed, since it had
    the needed -lzstd, avoiding:

    $ cat /tmp/build/perf/feature/test-all.make.output
    /usr/bin/ld: /tmp/ccRRJQ4u.o: in function `main_test_libzstd':
    /home/acme/git/perf/tools/build/feature/test-libzstd.c:8: undefined reference to `ZSTD_createCStream'
    /usr/bin/ld: /home/acme/git/perf/tools/build/feature/test-libzstd.c:9: undefined reference to `ZSTD_freeCStream'
    collect2: error: ld returned 1 exit status
    $

    Fix it by adding -lzstd to the test-all target.

    Now I need an entry to 'perf test' to make sure that
    /tmp/build/perf/feature/test-all.make.output is empty...

    Fixes: 3b1c5d9659718263 ("tools build: Implement libzstd feature check, LIBZSTD_DIR and NO_LIBZSTD defines")
    Reviewed-by: Alexei Budankov <alexey.budankov@linux.intel.com>
    Acked-by: Namhyung Kim <namhyung@kernel.org>
    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: Ian Rogers <irogers@google.com>
    Cc: Jiri Olsa <jolsa@kernel.org>
    Link: http://lore.kernel.org/lkml/20200904202611.GJ3753976@kernel.org
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    tools/build/feature/Makefile | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/tools/build/feature/Makefile b/tools/build/feature/Makefile
    index 9ee2bcb8d560a..8da2556cdbfac 100644
    --- a/tools/build/feature/Makefile
    +++ b/tools/build/feature/Makefile
    @@ -90,7 +90,7 @@ __BUILDXX = $(CXX) $(CXXFLAGS) -MD -Wall -Werror -o $@ $(patsubst %.bin,%.cpp,$(
    ###############################

    $(OUTPUT)test-all.bin:
    - $(BUILD) -fstack-protector-all -O2 -D_FORTIFY_SOURCE=2 -ldw -lelf -lnuma -lelf -I/usr/include/slang -lslang $(FLAGS_PERL_EMBED) $(FLAGS_PYTHON_EMBED) -DPACKAGE='"perf"' -lbfd -ldl -lz -llzma
    + $(BUILD) -fstack-protector-all -O2 -D_FORTIFY_SOURCE=2 -ldw -lelf -lnuma -lelf -I/usr/include/slang -lslang $(FLAGS_PERL_EMBED) $(FLAGS_PYTHON_EMBED) -DPACKAGE='"perf"' -lbfd -ldl -lz -llzma -lzstd

    $(OUTPUT)test-hello.bin:
    $(BUILD)
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-10-27 17:31    [W:2.388 / U:0.112 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site