linux-kernel - Re: [PATCH v4 0/7] selftests: fixes for installation and cross compilation

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <CANMBJr53a0cdJ5yjBu3ADv=x-==X0HXVQDELmuSuLLNCKLzviw@mail.gmail.com>
Date:	Wed, 6 May 2015 07:49:27 -0700
From:	Tyler Baker <tyler.baker@...aro.org>
To:	Shuah Khan <shuahkh@....samsung.com>
Cc:	Kevin Hilman <khilman@...nel.org>,
	John Stultz <john.stultz@...aro.org>,
	Darren Hart <dvhart@...radead.org>,
	David Herrmann <dh.herrmann@...il.com>,
	Michael Ellerman <mpe@...erman.id.au>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
	Tyler Baker <tyler.baker@...aro.org>
Subject: Re: [PATCH v4 0/7] selftests: fixes for installation and cross compilation

Hi Shuah,

On 21 April 2015 at 15:49, Tyler Baker <tyler.baker@...aro.org> wrote:
> This patch set fixes various issues observed when cross building and
> installing selftests.
>
> As I began investigating improving the test output format, I performed an
> audit of the current tests to ensure all tests were able to execute on various
> target architectures. I found that some tests did not install their binaries
> and others required directories to be installed to execute properly. There
> were also cases in which tests were being installed when they were never built.
> With this series applied all tests compile when appropriate and install their
> output properly.
>
> I have tested this series by building, installing and deploying all selftests
> to x86, arm and arm64 targets.
>
> Changes v3 -> v4:
> * Planning to send a followup series to address the lib.mk issue so I've
> dropped the breakpoints and x86 installation workaround.
> * x86: Removed redundant else logic block.
> * Applied Andy Lutomirski's ACK to patches 4 and 5.
> * Rebased onto next-20150421.

I believe all comments have been addressed, is there anything else you
would like to see?

>
> Changes v2 -> v3:
> * 5/8: Removed duplicate all_32.
> * 6/8: Ensure the default rule is all in every case.
> * 6/8: Fixed commit message wording for better clarity.
> * Applied Michael Ellerman's ACK to patches 1-4, and 8 as they have not changed.
> * Rebased onto next-20150420.
>
> Changes v1 -> v2:
> * Have no dependency on all when CROSS_COMPILE is set. (Andy Lutomirski)
> * Added Andy on CC for all x86 test patches.
> * Split up the x86 patches for better clarity.
> * Rebased onto next-20150415.
>
> This series is based on next-20150421.
>
> Tyler Baker (7):
>   selftests: copy TEST_DIRS to INSTALL_PATH
>   selftests/ftrace: install test.d
>   selftests/kdbus: install kdbus-test
>   selftest/x86: build both bitnesses
>   selftest/x86: have no dependency on all when cross building
>   selftests/x86: install tests
>   selftests/exec: do not install subdir as it is already created
>
>  tools/testing/selftests/exec/Makefile   |  2 +-
>  tools/testing/selftests/ftrace/Makefile |  1 +
>  tools/testing/selftests/kdbus/Makefile  |  2 ++
>  tools/testing/selftests/lib.mk          |  3 +++
>  tools/testing/selftests/x86/Makefile    | 15 ++++++++++++---
>  5 files changed, 19 insertions(+), 4 deletions(-)
>
> --
> 2.1.4
>

Thanks,

Tyler
--
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/