[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH v2 1/2] Do not register kvmclock savevm section
From: |
Marcelo Tosatti |
Subject: |
[Qemu-devel] Re: [PATCH v2 1/2] Do not register kvmclock savevm section if kvmclock is disabled. |
Date: |
Mon, 6 Dec 2010 19:04:01 -0200 |
User-agent: |
Mutt/1.5.20 (2009-08-17) |
On Mon, Dec 06, 2010 at 09:03:46AM -0500, Glauber Costa wrote:
> Usually nobody usually thinks about that scenario (me included and specially),
> but kvmclock can be actually disabled in the host.
>
> It happens in two scenarios:
> 1. host too old.
> 2. we passed -kvmclock to our -cpu parameter.
>
> In both cases, we should not register kvmclock savevm section. This patch
> achives that by registering this section only if kvmclock is actually
> currently enabled in cpuid.
>
> The only caveat is that we have to register the savevm section a little bit
> later, since we won't know the final kvmclock state before cpuid gets parsed.
What is the problem of registering the section? Restoring the value if
the host does not support it returns an error?
Can't you ignore the error if kvmclock is not reported in cpuid, in the
restore handler?
Also, please generate against uq/master.
[Qemu-devel] [PATCH v2 2/2] make kvmclock value idempotent for stopped machine, Glauber Costa, 2010/12/06
[Qemu-devel] Re: [PATCH v2 0/2] savevm odness related to kvmclock, Paolo Bonzini, 2010/12/06