From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=57427 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PPbm2-00031d-6b for qemu-devel@nongnu.org; Mon, 06 Dec 2010 09:10:39 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PPbm0-0005LI-SW for qemu-devel@nongnu.org; Mon, 06 Dec 2010 09:10:37 -0500 Received: from mail-qw0-f45.google.com ([209.85.216.45]:57308) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PPbm0-0005LD-Nq for qemu-devel@nongnu.org; Mon, 06 Dec 2010 09:10:36 -0500 Received: by qwf7 with SMTP id 7so917987qwf.4 for ; Mon, 06 Dec 2010 06:10:36 -0800 (PST) Sender: Paolo Bonzini Message-ID: <4CFCEEDC.6040009@redhat.com> Date: Mon, 06 Dec 2010 15:10:36 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1291644227-22923-1-git-send-email-glommer@redhat.com> In-Reply-To: <1291644227-22923-1-git-send-email-glommer@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH v2 0/2] savevm odness related to kvmclock List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Glauber Costa Cc: mtosatti@redhat.com, qemu-devel@nongnu.org, kvm@vger.kernel.org, avi@redhat.com On 12/06/2010 03:03 PM, Glauber Costa wrote: > Some users told me that savevm path is behaving oddly wrt kvmclock. > The first oddness is that a guarantee we never made (AFAIK) is being broken: > two consecutive "savevm" operations, with the machine stopped in between > produces different results, due to the call to KVM_GET_CLOCK ioctl. > I believe the assumption that if the vm does not run, its saveable > state won't change is fairly reasonable. Maybe we should formally > guarantee that? > > Also, this patch deals with the fact that this happens even if > kvmclock is disabled in cpuid: its savevm section is registered > nevertheless. Here, I try to register it only if it's enabled at > machine start. > > v2: improvements suggested by Paolo, and patch reordering. > > Glauber Costa (2): > Do not register kvmclock savevm section if kvmclock is disabled. > make kvmclock value idempotent for stopped machine > > cpus.c | 3 +++ > qemu-kvm-x86.c | 23 +++++++++++++++-------- > qemu-kvm.h | 3 +++ > target-i386/kvm.c | 7 +++++++ > 4 files changed, 28 insertions(+), 8 deletions(-) ACK Paolo