On Fri, Apr 15, 2016 at 10:52:42PM +0200, Radim Krčmář wrote: > v2: http://www.spinics.net/lists/kvm/msg125429.html > > mkstandalone is using generic code now, so v3 focuses on the output > again. And the output looks like in v2. Since v2, I've > * removed svm-disabled [3/7] > * reworked summary reporting [8/9] (it's a nice feature, but still ugly) > * turned all cases when QEMU won't run the test into SKIP [6/7] > * preserved check_param (it's not used, though) > * stopped returning 77 on all SKIPs [7/7] (a sacrifice to prevent > another refactoring of our scripts) > > I didn't manage to look at the comment from Lukáš's patches, but I don't > expect that v3 will be the last one ... a lot has changed since v2. Lukas' comment was that the skip due to a test needing KVM (i.e. not possible to run with TCG) should get a more informative summary. He wanted something along the lines of 'kvm test only, and this system doesn't have kvm', rather than just 'kvm only'. Oh, actually those summaries are arm and powerpc run script summaries, not common runtime. They could still be improved with this series :-) Thanks, drew > > > Radim Krčmář (9): > lib/report: allow test skipping > x86/*: report skipped tests > x86/unittests: remove svm-disabled > x86/pmu: expect failure with nmi_watchdog > lib/report: don't print 0 failed tests > scripts/runtime: skip tests that cannot run > scripts/runtime: consolidate summary tags > run_tests: print summary > run_tests: log stderr > > lib/libcflat.h | 1 + > lib/report.c | 49 ++++++++++++++++++++++++++++++++++--------------- > run_tests.sh | 7 ++++--- > scripts/mkstandalone.sh | 4 ++++ > scripts/runtime.bash | 32 +++++++++++++++++++++++++------- > x86/apic.c | 7 +++---- > x86/emulator.c | 2 +- > x86/hyperv_synic.c | 2 +- > x86/pku.c | 2 +- > x86/pmu.c | 11 +++++++++-- > x86/smap.c | 2 +- > x86/svm.c | 2 +- > x86/tsc.c | 2 +- > x86/unittests.cfg | 9 +-------- > 14 files changed, 87 insertions(+), 45 deletions(-) > > -- > 2.8.1 > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
