On 4/23/19 4:31 PM, Kees Cook wrote:
On Tue, Apr 16, 2019 at 4:21 PM shuah shuah@kernel.org wrote:
On 4/16/19 5:16 PM, Kees Cook wrote:
On Tue, Apr 16, 2019 at 6:11 PM shuah shuah@kernel.org 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.
Right: it's used only by the run_kselftest.sh script:
ALL_SCRIPT := $(INSTALL_PATH)/run_kselftest.sh
install: ... echo "if [ "$$1" = "--summary" ]; then" >> $(ALL_SCRIPT)
So, I think this entire series can land. Is there other feedback I should incorporate? I'd like to see it get some -next testing...
Thanks!
Kees,
Yes I was planning to get this into next and did some testing. I found that the first patch breaks the summary option.
You can see it by running
make summary=1 TARGETS="breakpoints" kselftest
with and without your first patch. Ca you fix the regression and send me revised patches. Sorry, I didn't get back to you sooner. A few bugs I found kept me busy.
While I was testing your patch series, I found test build failures due to circular dependency that needed fixing. In addition, the same headers_install dependency, also broke O= and KBUILD_OUTPUT builds and fixed those as well. Long story short, these fixes might conflict with your work.
thanks, -- Shuah