Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759003Ab3EGNeB (ORCPT ); Tue, 7 May 2013 09:34:01 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:43243 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758964Ab3EGNd6 (ORCPT ); Tue, 7 May 2013 09:33:58 -0400 Date: Tue, 7 May 2013 09:33:17 -0400 From: Konrad Rzeszutek Wilk To: Stefano Stabellini Cc: Ian Campbell , "xen-devel@lists.xensource.com" , "marc.zyngier@arm.com" , "will.deacon@arm.com" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" Subject: Re: [Xen-devel] [PATCH RFC v2 4/4] xen/arm: account for stolen ticks Message-ID: <20130507133317.GD31610@phenom.dumpdata.com> References: <1367851878-21629-4-git-send-email-stefano.stabellini@eu.citrix.com> <1367918270.26321.39.camel@zakaz.uk.xensource.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: ucsinet21.oracle.com [156.151.31.93] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3681 Lines: 100 On Tue, May 07, 2013 at 12:58:32PM +0100, Stefano Stabellini wrote: > On Tue, 7 May 2013, Ian Campbell wrote: > > On Mon, 2013-05-06 at 15:51 +0100, Stefano Stabellini wrote: > > > Register the runstate_memory_area with the hypervisor. > > > Use pv_time_ops.steal_clock to account for stolen ticks. > > > > > > Signed-off-by: Stefano Stabellini > > > --- > > > arch/arm/xen/enlighten.c | 23 +++++++++++++++++++++++ > > > 1 files changed, 23 insertions(+), 0 deletions(-) > > > > > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > > > index ee86bfa..2a5cc82 100644 > > > --- a/arch/arm/xen/enlighten.c > > > +++ b/arch/arm/xen/enlighten.c > > > @@ -14,7 +14,10 @@ > > > #include > > > #include > > > #include > > > +#include > > > #include > > > +#include > > > +#include > > > #include > > > #include > > > #include > > > @@ -152,6 +155,20 @@ int xen_unmap_domain_mfn_range(struct vm_area_struct *vma, > > > } > > > EXPORT_SYMBOL_GPL(xen_unmap_domain_mfn_range); > > > > > > +unsigned long long xen_stolen_accounting(int cpu) > > > +{ > > > + struct vcpu_runstate_info state; > > > + > > > + if (cpu != get_cpu()) > > > > get_cpu disables preempt, so you need a matching put_cpu. > > Oops, thanks. > > > > But actually I think you just want smp_processor_id and you probably > > want the BUG_ON form to get unlikely etc. > > > > That said, you don't use cpu for anything else, so why not drop it > > entirely? > > Ah, that's right, legacy of the past. I'll do that. > > > > > + BUG(); > > > + > > > + xen_get_runstate_snapshot(&state); > > > + > > > + WARN_ON(state.state != RUNSTATE_running); > > > + > > > + return state.time[RUNSTATE_runnable] + state.time[RUNSTATE_offline]; > > > +} > > > + > > > static void __init xen_percpu_init(void *unused) > > > { > > > struct vcpu_register_vcpu_info info; > > > @@ -170,6 +187,8 @@ static void __init xen_percpu_init(void *unused) > > > BUG(); > > > per_cpu(xen_vcpu, cpu) = vcpup; > > > > > > + xen_setup_runstate_info(cpu); > > > + > > > enable_percpu_irq(xen_events_irq, 0); > > > } > > > > > > @@ -301,6 +320,10 @@ static int __init xen_init_events(void) > > > > > > on_each_cpu(xen_percpu_init, NULL, 0); > > > > > > + pv_time_ops.steal_clock = xen_stolen_accounting; > > > + static_key_slow_inc(¶virt_steal_enabled); > > > + static_key_slow_inc(¶virt_steal_rq_enabled); > > > > We don't seem to do this on x86 -- is that a bug on x86 on Xen? > > On x86 we do all the accounting in do_stolen_accounting, called from our > own interrupt handler (xen_timer_interrupt). > I don't think we would gain anything by using the common infrastructure, > we would actually loose the idle ticks accounting we do there. > > Speaking of which, I don't think that pv_time_ops.steal_clock would > properly increase CPUTIME_IDLE the way we do in do_stolen_accounting. > > How much of an issue is that? Well, if you look in https://git.kernel.org/cgit/linux/kernel/git/konrad/xen.git/log/?h=devel/pvtime.v1.1 I tried to do use those two asm goto keys but it all started complaining to me during bootup with some WARNs. Never figured out why, but if somebody is interested in taking a look ... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/