On Thu, Jan 24, 2019 at 11:16:34AM +0000, Alexandru Elisei wrote: > Instead of aborting the test when an unexpected parameter is found, use > argv_find() to search for the desired parameter. On arm and arm64, this > allows kvm-unit-tests to be used with virtual machine monitors like kvmtool > which automatically adds parameters to the kernel command line. > > For example, to run the gicv3-ipi test the following command was used: > > lkvm run -k gic.flat -c 8 -m 410 -p "ipi" --irqchip=gicv3 > > The resulting kernel command line that was passed to kvm-unit-tests was: > > "console=ttyS0 rw rootflags=trans=virtio,version=9p2000.L,cache=loose rootfstype=9p init=/virt/init ip=dhcp ipi" Why not write a patch for kvmtool that provides a new command line parameter which says to not add any kernel parameters other than what is provided with '-p'? Giving the user full control over the kernel command line seems like a feature kvmtool would want anyway. Thanks, drew > > Without the patch, the following error occurs: > "ABORT: gicv3: Unknown subtest 'console=ttyS0'" > > With the patch, the test is able to run. > > Signed-off-by: Alexandru Elisei <alexandru.elisei@xxxxxxx> > --- > arm/gic.c | 7 ++++--- > arm/selftest.c | 19 ++++++++++++------- > 2 files changed, 16 insertions(+), 10 deletions(-) > > diff --git a/arm/gic.c b/arm/gic.c > index ed5642e74f70..f3746de4bfd8 100644 > --- a/arm/gic.c > +++ b/arm/gic.c > @@ -12,6 +12,7 @@ > * This work is licensed under the terms of the GNU LGPL, version 2. > */ > #include <libcflat.h> > +#include <argv.h> > #include <asm/setup.h> > #include <asm/processor.h> > #include <asm/delay.h> > @@ -533,13 +534,13 @@ int main(int argc, char **argv) > if (argc < 2) > report_abort("no test specified"); > > - if (strcmp(argv[1], "ipi") == 0) { > + if (argv_find("ipi", argc, argv) != -1) { > report_prefix_push(argv[1]); > nr_cpu_check(2); > on_cpus(ipi_test, NULL); > - } else if (strcmp(argv[1], "active") == 0) { > + } else if (argv_find("active", argc, argv) != -1) { > run_active_clear_test(); > - } else if (strcmp(argv[1], "mmio") == 0) { > + } else if (argv_find("mmio", argc, argv) != -1) { > report_prefix_push(argv[1]); > gic_test_mmio(); > report_prefix_pop(); > diff --git a/arm/selftest.c b/arm/selftest.c > index ea5101ef7217..44b2f23ca115 100644 > --- a/arm/selftest.c > +++ b/arm/selftest.c > @@ -8,6 +8,7 @@ > #include <libcflat.h> > #include <util.h> > #include <devicetree.h> > +#include <argv.h> > #include <asm/setup.h> > #include <asm/ptrace.h> > #include <asm/asm-offsets.h> > @@ -319,28 +320,32 @@ static void cpu_report(void *data __unused) > > int main(int argc, char **argv) > { > + int pos; > + > report_prefix_push("selftest"); > > if (argc < 2) > report_abort("no test specified"); > > - report_prefix_push(argv[1]); > - > - if (strcmp(argv[1], "setup") == 0) { > + if ((pos = argv_find("setup", argc, argv)) != -1) { > > - check_setup(argc-2, &argv[2]); > + report_prefix_push("setup"); > + check_setup(argc-(pos+1), &argv[pos+1]); > > - } else if (strcmp(argv[1], "vectors-kernel") == 0) { > + } else if (argv_find("vectors-kernel", argc, argv) != -1) { > > + report_prefix_push("vectors-kernel"); > check_vectors(NULL); > > - } else if (strcmp(argv[1], "vectors-user") == 0) { > + } else if (argv_find("vectors-user", argc, argv) != -1) { > > + report_prefix_push("vectors-user"); > start_usr(check_vectors, NULL, > (unsigned long)thread_stack_alloc()); > > - } else if (strcmp(argv[1], "smp") == 0) { > + } else if (argv_find("smp", argc, argv) != -1) { > > + report_prefix_push("smp"); > report("PSCI version", psci_check()); > on_cpus(cpu_report, NULL); > > -- > 2.17.0 >