On 4/16/19 5:16 PM, Kees Cook wrote:
On Tue, Apr 16, 2019 at 6:11 PM shuah <shuah@xxxxxxxxxx> wrote:
Hi Kees,
Thanks for the patch.
On 4/9/19 5:55 PM, Kees Cook wrote:
In order to improve the reusability of the kselftest test running logic,
this extracts the single-test logic from lib.mk into kselftest/runner.sh
which lib.mk can call directly. No changes in output.
As part of the change, this removes the unused "summary" Makefile variable
(and tests). However, future merging with the "emit_tests" target needs
to be able to redirect output, so a new "logfile" variable is introduced.
Shouldn't the selftests/Makefile need update for "summary" removal??
I didn't see anything using "summary" except as a --summary argument
to the run_kselftests.sh script. Maybe I missed it?
It is in the selftests/Makefile install target.
Signed-off-by: Kees Cook <keescook@xxxxxxxxxxxx>
---
tools/testing/selftests/.gitignore | 1 -
tools/testing/selftests/kselftest/runner.sh | 31 +++++++++++++++++++
tools/testing/selftests/lib.mk | 33 ++-------------------
3 files changed, 34 insertions(+), 31 deletions(-)
create mode 100644 tools/testing/selftests/kselftest/runner.sh
diff --git a/tools/testing/selftests/.gitignore b/tools/testing/selftests/.gitignore
index 91750352459d..8059ce834247 100644
--- a/tools/testing/selftests/.gitignore
+++ b/tools/testing/selftests/.gitignore
@@ -1,4 +1,3 @@
-kselftest
gpiogpio-event-mon
gpiogpio-hammer
gpioinclude/
Please don't include this .gitignore change here. These are generated
in tools/gpio and this .gitignore isn't the right place for them.
This change is only removing the "kselftest" entry, for which the
target is long gone. Since I was adding a directory by that name, I
needed to remove it from the .gitignore file. I have nothing to do
with the gpio stuff. :)
Yeah my bad! :)
Thanks for looking this over!
thanks,
-- Shuah
