linux-kernel - Re: [PATCH v3] selftests/zram: Adding zram tests

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CA+G9fYvSp3xyn8ZT2-CcgGVwue49REXiTA1tS-dGQm9Hqe_TXw@mail.gmail.com>
Date:	Mon, 31 Aug 2015 17:49:51 +0530
From:	Naresh Kamboju <naresh.kamboju@...aro.org>
To:	Michael Ellerman <mpe@...erman.id.au>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Tyler Baker <tyler.baker@...aro.org>,
	Milosz Wasilewski <milosz.wasilewski@...aro.org>,
	Shuah Khan <shuahkh@....samsung.com>, alexey.kodanev@...cle.com
Subject: Re: [PATCH v3] selftests/zram: Adding zram tests

On 31 August 2015 at 09:37, Michael Ellerman <mpe@...erman.id.au> wrote:
> On Tue, 2015-08-18 at 12:31 +0530, naresh.kamboju@...aro.org wrote:
>> From: Naresh Kamboju <naresh.kamboju@...aro.org>
>>
>> diff --git a/tools/testing/selftests/zram/Makefile b/tools/testing/selftests/zram/Makefile
>> new file mode 100644
>> index 0000000..ec45513
>> --- /dev/null
>> +++ b/tools/testing/selftests/zram/Makefile
>> @@ -0,0 +1,12 @@
>> +all:
>> +
>> +TEST_PROGS := zram.sh
>> +TEST_FILES := zram01.sh zram02.sh zram_lib.sh
>> +
>> +include ../lib.mk
>> +
>> +run_tests:
>> +     @/bin/bash ./zram.sh
>> +
>

Michael,

> This is wrong, it gives me:
>
> $ make TARGETS=zram install
> ...
> Makefile:9: warning: overriding recipe for target 'run_tests'
> ../lib.mk:12: warning: ignoring old recipe for target 'run_tests'
>
>
> If you want to define run_tests you need to do:
>
> override define RUN_TESTS
>         ./zram.sh
> endef
>
>
> BUT, I don't see any reason why you need to override run_tests. The default
> rule will just run TEST_PROGS, which is zram.sh so that should work fine?

Fixed according to your comments.
selftests/zram: Makefile fix - Patch sent for review.

Cheers
Naresh



>
> cheers
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/