From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rik van Riel Subject: Re: [PATCH 14/16] KVM-GST: KVM Steal time registration Date: Mon, 24 Jan 2011 20:26:59 -0500 Message-ID: <4D3E26E3.3000209@redhat.com> References: <1295892397-11354-1-git-send-email-glommer@redhat.com> <1295892397-11354-15-git-send-email-glommer@redhat.com> <4D3E0BB9.5070805@redhat.com> <1295918712.15920.16.camel@mothafucka.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, aliguori@us.ibm.com, Jeremy Fitzhardinge , Peter Zijlstra , Avi Kivity To: Glauber Costa Return-path: In-Reply-To: <1295918712.15920.16.camel@mothafucka.localdomain> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 01/24/2011 08:25 PM, Glauber Costa wrote: > On Mon, 2011-01-24 at 18:31 -0500, Rik van Riel wrote: >> On 01/24/2011 01:06 PM, Glauber Costa wrote: >>> Register steal time within KVM. Everytime we sample the steal time >>> information, we update a local variable that tells what was the >>> last time read. We then account the difference. >>> >>> Signed-off-by: Glauber Costa >>> CC: Rik van Riel >>> CC: Jeremy Fitzhardinge >>> CC: Peter Zijlstra >>> CC: Avi Kivity >> >> On second thought - how does this deal with cpu hotplug and >> hot unplug? >> >> Do you allocate a new one of these structs every time a cpu >> is hot unplugged and then hotplugged, leaking the old one? >> >> Will leaving the old value around confuse the steal time >> calculation? > > If you look closely, there are no allocations happening at all, > it's all static. In that case, does the per-cpu steal area need to be reinitialized at hotplug time? -- All rights reversed