| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1618170495.42951.1508248216596.JavaMail.zimbra@efficios.com>
Date: Tue, 17 Oct 2017 13:50:16 +0000 (UTC)
From: Mathieu Desnoyers <mathieu.desnoyers@...icios.com>
To: Michael Ellerman <mpe@...erman.id.au>
Cc: "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>,
Boqun Feng <boqun.feng@...il.com>,
Peter Zijlstra <peterz@...radead.org>,
Paul Turner <pjt@...gle.com>, Andrew Hunter <ahh@...gle.com>,
Andy Lutomirski <luto@...capital.net>,
Dave Watson <davejwatson@...com>,
Josh Triplett <josh@...htriplett.org>,
Will Deacon <will.deacon@....com>,
linux-kernel <linux-kernel@...r.kernel.org>,
Russell King <linux@....linux.org.uk>,
Catalin Marinas <catalin.marinas@....com>,
Thomas Gleixner <tglx@...utronix.de>,
Andi Kleen <andi@...stfloor.org>, Chris Lameter <cl@...ux.com>,
Ingo Molnar <mingo@...hat.com>,
"H. Peter Anvin" <hpa@...or.com>, Ben Maurer <bmaurer@...com>,
rostedt <rostedt@...dmis.org>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Andrew Morton <akpm@...ux-foundation.org>,
shuah <shuah@...nel.org>,
linux-kselftest <linux-kselftest@...r.kernel.org>,
linux-api <linux-api@...r.kernel.org>
Subject: Re: [RFC PATCH for 4.15 14/14] Restartable sequences: Provide
self-tests
----- On Oct 17, 2017, at 6:36 AM, Michael Ellerman mpe@...erman.id.au wrote:
> Mathieu Desnoyers <mathieu.desnoyers@...icios.com> writes:
>
>> Hi Michael,
>>
>> With your changes integrated, both rseq and cpu-opv selftests fail to
>> build if I pass e.g. -j32 to make.
>>
>> cd tools/testing/selftests/cpu-opv
>>
>> efficios@...pudjdev:~/git/linux-percpu-dev/tools/testing/selftests/cpu-opv$ make
>> clean; make
>> rm -f -r
>> /home/efficios/git/linux-percpu-dev/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
>> gcc -O2 -Wall -g -I./ -I../../../../usr/include/ basic_cpu_opv_test.c
>> cpu-op.c cpu-op.h -o basic_cpu_opv_test
>>
>> efficios@...pudjdev:~/git/linux-percpu-dev/tools/testing/selftests/cpu-opv$ make
>> clean; make -j32
>> rm -f -r
>> /home/efficios/git/linux-percpu-dev/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
>> gcc -O2 -Wall -g -I./ -I../../../../usr/include/ basic_cpu_opv_test.c
>> cpu-op.c cpu-op.h -o basic_cpu_opv_test
>> gcc -O2 -Wall -g -I./ -I../../../../usr/include/ basic_cpu_opv_test.c -o
>> /home/efficios/git/linux-percpu-dev/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
>> /tmp/ccDthnqM.o: In function `test_memcpy_op':
>> /home/efficios/git/linux-percpu-dev/tools/testing/selftests/cpu-opv/basic_cpu_opv_test.c:364:
>> undefined reference to `cpu_op_get_current_cpu'
>> /home/efficios/git/linux-percpu-dev/tools/testing/selftests/cpu-opv/basic_cpu_opv_test.c:365:
>> undefined reference to `cpu_opv'
> ...
>> make: ***
>> [/home/efficios/git/linux-percpu-dev/tools/testing/selftests/cpu-opv/basic_cpu_opv_test]
>> Error 1
>> make: *** Waiting for unfinished jobs....
>>
>> Any idea what is going on here ?
>
> Ugh sorry, yes.
>
> New patch below should fix it. Tested with -j:
Perfect, folding it into my series.
I see that the "all" target was redundant here.
Thanks,
Mathieu
>
> ~/linux/tools/testing/selftests$ make TARGETS="rseq cpu-opv" -j
> make[1]: Entering directory '/home/michael/linux/tools/testing/selftests/rseq'
> gcc -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/ basic_test.c
> rseq.h rseq-ppc.h rseq-x86.h rseq-arm.h rseq.c ../cpu-opv/cpu-op.c
> ../cpu-opv/cpu-op.h -lpthread -o
> /home/michael/linux/tools/testing/selftests/rseq/basic_test
> gcc -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/
> basic_percpu_ops_test.c rseq.h rseq-ppc.h rseq-x86.h rseq-arm.h rseq.c
> ../cpu-opv/cpu-op.c ../cpu-opv/cpu-op.h -lpthread -o
> /home/michael/linux/tools/testing/selftests/rseq/basic_percpu_ops_test
> gcc -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/ param_test.c
> rseq.h rseq-ppc.h rseq-x86.h rseq-arm.h rseq.c ../cpu-opv/cpu-op.c
> ../cpu-opv/cpu-op.h -lpthread -o
> /home/michael/linux/tools/testing/selftests/rseq/param_test
> make[1]: Leaving directory '/home/michael/linux/tools/testing/selftests/rseq'
> make[1]: Entering directory
> '/home/michael/linux/tools/testing/selftests/cpu-opv'
> gcc -O2 -Wall -g -I./ -I../../../../usr/include/ basic_cpu_opv_test.c
> cpu-op.c cpu-op.h -o
> /home/michael/linux/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
> make[1]: Leaving directory '/home/michael/linux/tools/testing/selftests/cpu-opv'
> ~/linux/tools/testing/selftests$ make TARGETS="rseq cpu-opv" clean
> make[1]: Entering directory '/home/michael/linux/tools/testing/selftests/rseq'
> rm -f -r /home/michael/linux/tools/testing/selftests/rseq/basic_test
> /home/michael/linux/tools/testing/selftests/rseq/basic_percpu_ops_test
> /home/michael/linux/tools/testing/selftests/rseq/param_test
> make[1]: Leaving directory '/home/michael/linux/tools/testing/selftests/rseq'
> make[1]: Entering directory
> '/home/michael/linux/tools/testing/selftests/cpu-opv'
> rm -f -r /home/michael/linux/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
> make[1]: Leaving directory '/home/michael/linux/tools/testing/selftests/cpu-opv'
> ~/linux/tools/testing/selftests$ cd cpu-opv/
> ~/linux/tools/testing/selftests/cpu-opv$ make -j
> gcc -O2 -Wall -g -I./ -I../../../../usr/include/ basic_cpu_opv_test.c
> cpu-op.c cpu-op.h -o
> /home/michael/linux/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
> ~/linux/tools/testing/selftests/cpu-opv$ make clean
> rm -f -r /home/michael/linux/tools/testing/selftests/cpu-opv/basic_cpu_opv_test
> ~/linux/tools/testing/selftests/cpu-opv$ cd ../rseq/
> ~/linux/tools/testing/selftests/rseq$ make -j
> gcc -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/ basic_test.c
> rseq.h rseq-ppc.h rseq-x86.h rseq-arm.h rseq.c ../cpu-opv/cpu-op.c
> ../cpu-opv/cpu-op.h -lpthread -o
> /home/michael/linux/tools/testing/selftests/rseq/basic_test
> gcc -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/
> basic_percpu_ops_test.c rseq.h rseq-ppc.h rseq-x86.h rseq-arm.h rseq.c
> ../cpu-opv/cpu-op.c ../cpu-opv/cpu-op.h -lpthread -o
> /home/michael/linux/tools/testing/selftests/rseq/basic_percpu_ops_test
> gcc -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/ param_test.c
> rseq.h rseq-ppc.h rseq-x86.h rseq-arm.h rseq.c ../cpu-opv/cpu-op.c
> ../cpu-opv/cpu-op.h -lpthread -o
> /home/michael/linux/tools/testing/selftests/rseq/param_test
> ~/linux/tools/testing/selftests/rseq$ make clean
> rm -f -r /home/michael/linux/tools/testing/selftests/rseq/basic_test
> /home/michael/linux/tools/testing/selftests/rseq/basic_percpu_ops_test
> /home/michael/linux/tools/testing/selftests/rseq/param_test
>
>
>
> cheers
>
> diff --git a/tools/testing/selftests/cpu-opv/Makefile
> b/tools/testing/selftests/cpu-opv/Makefile
> index 81d0596824ee..d27bd0f74deb 100644
> --- a/tools/testing/selftests/cpu-opv/Makefile
> +++ b/tools/testing/selftests/cpu-opv/Makefile
> @@ -1,13 +1,7 @@
> CFLAGS += -O2 -Wall -g -I./ -I../../../../usr/include/
> -LDFLAGS += -lpthread
>
> -TESTS = basic_cpu_opv_test
> -
> -all: $(TESTS)
> -%: %.c cpu-op.c cpu-op.h
> - $(CC) $(CFLAGS) -o $@ $^ $(LDFLAGS)
> +TEST_GEN_PROGS = basic_cpu_opv_test
>
> include ../lib.mk
>
> -clean:
> - $(RM) $(TESTS)
> +$(TEST_GEN_PROGS): cpu-op.c cpu-op.h
> diff --git a/tools/testing/selftests/rseq/Makefile
> b/tools/testing/selftests/rseq/Makefile
> index 7f0153556b80..7f625147b7fe 100644
> --- a/tools/testing/selftests/rseq/Makefile
> +++ b/tools/testing/selftests/rseq/Makefile
> @@ -1,13 +1,8 @@
> CFLAGS += -O2 -Wall -g -I./ -I../cpu-opv/ -I../../../../usr/include/
> -LDFLAGS += -lpthread
> +LDLIBS += -lpthread
>
> -TESTS = basic_test basic_percpu_ops_test param_test
> -
> -all: $(TESTS)
> -%: %.c rseq.h rseq-*.h rseq.c ../cpu-opv/cpu-op.c ../cpu-opv/cpu-op.h
> - $(CC) $(CFLAGS) -o $@ $^ $(LDFLAGS)
> +TEST_GEN_PROGS = basic_test basic_percpu_ops_test param_test
>
> include ../lib.mk
>
> -clean:
> - $(RM) $(TESTS)
> +$(TEST_GEN_PROGS): rseq.h rseq-*.h rseq.c ../cpu-opv/cpu-op.c
> ../cpu-opv/cpu-op.h
--
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com
Powered by blists - more mailing lists