Re: [kvm-unit-tests PATCH 7/7] arm/arm64: Use argv_find() for test names

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
> 



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux