public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Jason Wang <jasowang@redhat.com>
To: Michael Goldish <mgoldish@redhat.com>
Cc: autotest@test.kernel.org, kvm@vger.kernel.org
Subject: Re: [PATCH 4/4] KVM test: Enable timedrift for Linux guests
Date: Wed, 24 Mar 2010 11:13:33 +0800	[thread overview]
Message-ID: <4BA9835D.4020303@redhat.com> (raw)
In-Reply-To: <346163263.3898461269258300968.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>

Michael Goldish wrote:
> ----- "Jason Wang" <jasowang@redhat.com> wrote:
>
>   
>> We should also test timedrift for Linux guests especially for guest
>> with pvclock. So this patch enable the timedrift for linux guests.
>>
>> Signed-off-by: Jason Wang <jasowang@redhat.com>
>> ---
>>  client/tests/kvm/tests_base.cfg.sample |   11 ++++++++++-
>>  1 files changed, 10 insertions(+), 1 deletions(-)
>>
>> diff --git a/client/tests/kvm/tests_base.cfg.sample
>> b/client/tests/kvm/tests_base.cfg.sample
>> index 8cc83a9..dcd0dcf 100644
>> --- a/client/tests/kvm/tests_base.cfg.sample
>> +++ b/client/tests/kvm/tests_base.cfg.sample
>> @@ -330,7 +330,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 +342,15 @@ 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:
>> +            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_clean_cmd = "killall -9 dd"
>>     
>
> I think this should be 'guest_load_stop_command'.
> Other than that, looks good to me.
>
>   
Thanks for pointing this, would resend this patch.
>> +            host_load_command = "bzip2 -c --best /dev/urandom >
>> /dev/null"
>> +            host_load_instances = 8
>>  
>>          variants:
>>              - Fedora:
>>
>> _______________________________________________
>> Autotest mailing list
>> Autotest@test.kernel.org
>> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
>>     
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>   

  reply	other threads:[~2010-03-24  3:13 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <223288964.3898351269258161561.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
2010-03-22 11:45 ` [Autotest] [PATCH 4/4] KVM test: Enable timedrift for Linux guests Michael Goldish
2010-03-24  3:13   ` Jason Wang [this message]
2010-03-22  7:44 [PATCH 1/4] KVM test: Add monotonic_time into the guest test Jason Wang
2010-03-22  7:45 ` [PATCH 4/4] KVM test: Enable timedrift for Linux guests Jason Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4BA9835D.4020303@redhat.com \
    --to=jasowang@redhat.com \
    --cc=autotest@test.kernel.org \
    --cc=kvm@vger.kernel.org \
    --cc=mgoldish@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox