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? > > > > 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. :) Thanks for looking this over! -- Kees Cook