From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gleb Natapov Subject: Re: qemu-kvm: enable get/set vcpu events on reset and migration Date: Thu, 28 Jan 2010 15:55:57 +0200 Message-ID: <20100128135557.GP739@redhat.com> References: <20100127002622.GB8811@amt.cnet> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Kiszka , kvm To: Marcelo Tosatti Return-path: Received: from mx1.redhat.com ([209.132.183.28]:19549 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755566Ab0A1N4A (ORCPT ); Thu, 28 Jan 2010 08:56:00 -0500 Content-Disposition: inline In-Reply-To: <20100127002622.GB8811@amt.cnet> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Jan 26, 2010 at 10:26:22PM -0200, Marcelo Tosatti wrote: > @@ -374,6 +375,7 @@ static int cpu_post_load(void *opaque, int version_id) > > kvm_load_tsc(env); > kvm_load_mpstate(env); > + kvm_put_vcpu_events(env); > } It is confusing to call functions load/save for some state and put/get for others, but the function kvm_put_vcpu_events() is called upstream this way :( Otherwise looks OK. -- Gleb.