On Wed, Mar 24, 2010 at 3:25 AM, Jason Wang <[email protected]> wrote:
> We should also test timedrift for Linux guests especially for guest
> with pvclock. So this patch enable the timedrift for linux guests.
>
> Changes from v1:
> - Correct the wrong name for guest load cleaning
> - Use -no-kvm-pit-reinjection for linux guests and -rtc-td-hack for
> windows guests.

Here I have a little doubt if the test is useful only while running
under these command line options (since we indeed have timedrift
failures without them). Maybe it makes more sense to create variants
with these options, to ensure that this command line will also be
tested. Michael?

> Signed-off-by: Jason Wang <[email protected]>
> ---
>  client/tests/kvm/tests_base.cfg.sample |   14 ++++++++++++--
>  1 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/client/tests/kvm/tests_base.cfg.sample 
> b/client/tests/kvm/tests_base.cfg.sample
> index 8cc83a9..29a2430 100644
> --- a/client/tests/kvm/tests_base.cfg.sample
> +++ b/client/tests/kvm/tests_base.cfg.sample
> @@ -147,7 +147,6 @@ variants:
>         type = linux_s3
>
>     - timedrift:    install setup unattended_install
> -        extra_params += " -rtc-td-hack"
>         variants:
>             - with_load:
>                 type = timedrift
> @@ -330,7 +329,7 @@ variants:
>  variants:
>     # Linux section
>     - @Linux:
> -        no timedrift autoit
> +        no autoit
>         shutdown_command = shutdown -h now
>         reboot_command = shutdown -r now
>         status_test_command = echo $?
> @@ -342,6 +341,16 @@ variants:
>         file_transfer_port = 22
>         mem_chk_cmd = dmidecode -t 17 | awk -F: '/Size/ {print $2}'
>         cpu_chk_cmd = grep -c processor /proc/cpuinfo
> +        timedrift:
> +            extra_params += " -no-kvm-pit-reinjection"
> +            time_command = date +'TIME: %a %m/%d/%Y %H:%M:%S.%N'
> +            time_filter_re = "(?:TIME: \w\w\w )(.{19})(?:\.\d\d)"
> +            time_format = "%m/%d/%Y %H:%M:%S"
> +            guest_load_command = "dd if=/dev/urandom of=/dev/null"
> +            guest_load_instances = 2
> +            guest_load_stop_command = "killall -9 dd"
> +            host_load_command = "bzip2 -c --best /dev/urandom > /dev/null"
> +            host_load_instances = 8
>
>         variants:
>             - Fedora:
> @@ -717,6 +726,7 @@ variants:
>         stress_boot:
>             alive_test_cmd = systeminfo
>         timedrift:
> +            extra_params += " -rtc-td-hack"
>             time_command = "echo TIME: %date% %time%"
>             time_filter_re = "(?<=TIME: \w\w\w ).{19}(?=\.\d\d)"
>             time_format = "%m/%d/%Y %H:%M:%S"
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>



-- 
Lucas
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to