From: Joerg Roedel <joro@8bytes.org>
To: Avi Kivity <avi@redhat.com>
Cc: Joerg Roedel <joerg.roedel@amd.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: Re: [Qemu-devel] [PATCH 2/3] qemu-x86: Add tsc_freq option to -cpu
Date: Tue, 19 Jul 2011 16:14:59 +0200 [thread overview]
Message-ID: <20110719141458.GH24072@8bytes.org> (raw)
In-Reply-To: <4E258CE9.4080203@redhat.com>
On Tue, Jul 19, 2011 at 04:55:53PM +0300, Avi Kivity wrote:
> On 07/19/2011 04:54 PM, Avi Kivity wrote:
>> On 07/19/2011 04:30 PM, Joerg Roedel wrote:
>>> Hmm, I planned to do the VMSTATE thing in a follow-on patch-set. The
>>> plan is to read the VCPU tsc_freq at guest start time on !tsc-scale
>>> hosts and migrate it over so that the destination host can set the
>>> tsc-freq if it supports tsc-scaling.
>>
>> This can be done by a management tool if desired.
>>
>
> Although, if we do this unconditionally (that is, also for tsc-scale
> hosts) then we get stable tsc even without supplying a tsc frequency
> argument... need to think about this.
It has the advantage that it "just works", without the need to extend
management tools and the like. And it makes migration more transparent
to the guests.
Joerg
next prev parent reply other threads:[~2011-07-19 14:15 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-07 14:13 [Qemu-devel] [PATCH 0/3][uq/master] Basic TSC-Scaling support v2 Joerg Roedel
2011-07-07 14:13 ` [Qemu-devel] [PATCH 1/3] qemu: Add strtosz_suffix_unit function Joerg Roedel
2011-07-08 8:00 ` Markus Armbruster
2011-07-07 14:13 ` [Qemu-devel] [PATCH 2/3] qemu-x86: Add tsc_freq option to -cpu Joerg Roedel
2011-07-19 11:46 ` Marcelo Tosatti
2011-07-19 12:20 ` Avi Kivity
2011-07-19 12:56 ` Marcelo Tosatti
2011-07-19 13:30 ` Joerg Roedel
2011-07-19 13:54 ` Avi Kivity
2011-07-19 13:55 ` Avi Kivity
2011-07-19 14:14 ` Joerg Roedel [this message]
2011-07-19 15:55 ` Avi Kivity
2011-07-07 14:13 ` [Qemu-devel] [PATCH 3/3] qemu-x86: Set tsc_khz in kvm when supported Joerg Roedel
2011-07-19 11:48 ` Marcelo Tosatti
2011-07-19 17:03 ` Jan Kiszka
2011-07-19 13:04 ` [Qemu-devel] [PATCH 0/3][uq/master] Basic TSC-Scaling support v2 Marcelo Tosatti
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=20110719141458.GH24072@8bytes.org \
--to=joro@8bytes.org \
--cc=avi@redhat.com \
--cc=joerg.roedel@amd.com \
--cc=kvm@vger.kernel.org \
--cc=mtosatti@redhat.com \
--cc=qemu-devel@nongnu.org \
/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;
as well as URLs for NNTP newsgroup(s).