Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751785AbdF1Ogg (ORCPT ); Wed, 28 Jun 2017 10:36:36 -0400 Received: from mail-wm0-f51.google.com ([74.125.82.51]:36453 "EHLO mail-wm0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751696AbdF1Oga (ORCPT ); Wed, 28 Jun 2017 10:36:30 -0400 MIME-Version: 1.0 In-Reply-To: References: <20170628111958.22405-1-fathi.boudra@linaro.org> From: Fathi Boudra Date: Wed, 28 Jun 2017 17:36:28 +0300 Message-ID: Subject: Re: [PATCH] selftests: net: use LDLIBS instead of LDFLAGS to link against libnuma To: Shuah Khan Cc: linux-kselftest@vger.kernel.org, "linux-kernel@vger.kernel.org" , SeongJae Park Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2083 Lines: 46 On 28 June 2017 at 17:19, Shuah Khan wrote: > On 06/28/2017 05:19 AM, Fathi Boudra wrote: >> reuseport_bpf_numa fails to build with undefined reference: >> >> aarch64-linaro-linux-gcc >> --sysroot=/build/tmp-rpb-glibc/sysroots/hikey -Wall >> -Wl,--no-as-needed -O2 -g -I../../../../usr/include/ -Wl,-O1 >> -Wl,--hash-style=gnu -Wl,--as-needed -lnuma reuseport_bpf_numa.c >> -o >> /build/tmp-rpb-glibc/work/hikey-linaro-linux/kselftests/4.12-r0/linux-4.12-rc7/tools/testing/selftests/net/reuseport_bpf_numa >> /tmp/ccfUuExT.o: In function `send_from_node': >> /build/tmp-rpb-glibc/work/hikey-linaro-linux/kselftests/4.12-r0/linux-4.12-rc7/tools/testing/selftests/net/reuseport_bpf_numa.c:138: >> undefined reference to `numa_run_on_node' >> /tmp/ccfUuExT.o: In function `main': >> /build/tmp-rpb-glibc/work/hikey-linaro-linux/kselftests/4.12-r0/linux-4.12-rc7/tools/testing/selftests/net/reuseport_bpf_numa.c:230: >> undefined reference to `numa_available' >> /build/tmp-rpb-glibc/work/hikey-linaro-linux/kselftests/4.12-r0/linux-4.12-rc7/tools/testing/selftests/net/reuseport_bpf_numa.c:233: >> undefined reference to `numa_max_node' >> >> Use LDLIBS instead of LDFLAGS to properly link against libnuma. >> >> Signed-off-by: Fathi Boudra > > Did you try this on linux-ksefltest next? It has the fix for this. yes, I tried linux-ksefltest next and can reproduce. > thanks, > -- Shuah >> --- >> tools/testing/selftests/net/Makefile | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tools/testing/selftests/net/Makefile b/tools/testing/selftests/net/Makefile >> index 35cbb4cba410..17c732a19c4c 100644 >> --- a/tools/testing/selftests/net/Makefile >> +++ b/tools/testing/selftests/net/Makefile >> @@ -3,7 +3,7 @@ >> CFLAGS = -Wall -Wl,--no-as-needed -O2 -g >> CFLAGS += -I../../../../usr/include/ >> >> -reuseport_bpf_numa: LDFLAGS += -lnuma >> +reuseport_bpf_numa: LDLIBS += -lnuma >> >> TEST_PROGS := run_netsocktests run_afpackettests test_bpf.sh netdevice.sh >> TEST_GEN_FILES = socket