Message ID | 1367436460-10183-3-git-send-email-stefano.stabellini@eu.citrix.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Wed, 2013-05-01 at 20:27 +0100, Stefano Stabellini wrote: > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > index d30042e..8f040a0 100644 > --- a/arch/arm/xen/enlighten.c > +++ b/arch/arm/xen/enlighten.c > @@ -14,6 +14,7 @@ > #include <xen/xen-ops.h> > #include <asm/xen/hypervisor.h> > #include <asm/xen/hypercall.h> > +#include <asm/arch_timer.h> > #include <asm/system_misc.h> > #include <linux/interrupt.h> > #include <linux/irqreturn.h> > @@ -172,6 +173,8 @@ static int __init xen_secondary_init(unsigned int cpu) > later ones fail to. */ > per_cpu(xen_vcpu, cpu) = vcpup; > } > + > + xen_setup_runstate_info(cpu); Is this called for the boot processor too? (do you have a linux-next branch where I can easily find the baseline for this series?) Ian.
On Thu, 2 May 2013, Ian Campbell wrote: > On Wed, 2013-05-01 at 20:27 +0100, Stefano Stabellini wrote: > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > > index d30042e..8f040a0 100644 > > --- a/arch/arm/xen/enlighten.c > > +++ b/arch/arm/xen/enlighten.c > > @@ -14,6 +14,7 @@ > > #include <xen/xen-ops.h> > > #include <asm/xen/hypervisor.h> > > #include <asm/xen/hypercall.h> > > +#include <asm/arch_timer.h> > > #include <asm/system_misc.h> > > #include <linux/interrupt.h> > > #include <linux/irqreturn.h> > > @@ -172,6 +173,8 @@ static int __init xen_secondary_init(unsigned int cpu) > > later ones fail to. */ > > per_cpu(xen_vcpu, cpu) = vcpup; > > } > > + > > + xen_setup_runstate_info(cpu); > > Is this called for the boot processor too? Yep. > (do you have a linux-next branch where I can easily find the baseline > for this series?) git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git lost_ticks_1 It's based on my 3.9-rc3-smp-6 branch currently in linux-next (that is based on 3.9-rc3). However lost_ticks_1 is not in linux-next yet.
On Thu, 2013-05-02 at 11:38 +0100, Stefano Stabellini wrote: > On Thu, 2 May 2013, Ian Campbell wrote: > > On Wed, 2013-05-01 at 20:27 +0100, Stefano Stabellini wrote: > > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > > > index d30042e..8f040a0 100644 > > > --- a/arch/arm/xen/enlighten.c > > > +++ b/arch/arm/xen/enlighten.c > > > @@ -14,6 +14,7 @@ > > > #include <xen/xen-ops.h> > > > #include <asm/xen/hypervisor.h> > > > #include <asm/xen/hypercall.h> > > > +#include <asm/arch_timer.h> > > > #include <asm/system_misc.h> > > > #include <linux/interrupt.h> > > > #include <linux/irqreturn.h> > > > @@ -172,6 +173,8 @@ static int __init xen_secondary_init(unsigned int cpu) > > > later ones fail to. */ > > > per_cpu(xen_vcpu, cpu) = vcpup; > > > } > > > + > > > + xen_setup_runstate_info(cpu); > > > > Is this called for the boot processor too? > > Yep. > > > (do you have a linux-next branch where I can easily find the baseline > > for this series?) > > git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git lost_ticks_1 Is that your Xen tree, I was after the Linux one... > It's based on my 3.9-rc3-smp-6 branch currently in linux-next (that is > based on 3.9-rc3). > However lost_ticks_1 is not in linux-next yet. Which branch is in Linux next? It doesn't seem to be your linux-next (last commit 2011). Ian.
On Thu, 2 May 2013, Ian Campbell wrote: > On Thu, 2013-05-02 at 11:38 +0100, Stefano Stabellini wrote: > > On Thu, 2 May 2013, Ian Campbell wrote: > > > On Wed, 2013-05-01 at 20:27 +0100, Stefano Stabellini wrote: > > > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > > > > index d30042e..8f040a0 100644 > > > > --- a/arch/arm/xen/enlighten.c > > > > +++ b/arch/arm/xen/enlighten.c > > > > @@ -14,6 +14,7 @@ > > > > #include <xen/xen-ops.h> > > > > #include <asm/xen/hypervisor.h> > > > > #include <asm/xen/hypercall.h> > > > > +#include <asm/arch_timer.h> > > > > #include <asm/system_misc.h> > > > > #include <linux/interrupt.h> > > > > #include <linux/irqreturn.h> > > > > @@ -172,6 +173,8 @@ static int __init xen_secondary_init(unsigned int cpu) > > > > later ones fail to. */ > > > > per_cpu(xen_vcpu, cpu) = vcpup; > > > > } > > > > + > > > > + xen_setup_runstate_info(cpu); > > > > > > Is this called for the boot processor too? > > > > Yep. > > > > > (do you have a linux-next branch where I can easily find the baseline > > > for this series?) > > > > git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git lost_ticks_1 > > Is that your Xen tree, I was after the Linux one... nope, that's my Linux tree disguised as a Xen tree to confuse the maintainers ;-) > > It's based on my 3.9-rc3-smp-6 branch currently in linux-next (that is > > based on 3.9-rc3). > > However lost_ticks_1 is not in linux-next yet. > > Which branch is in Linux next? It doesn't seem to be your linux-next > (last commit 2011). that would be linux-next from git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git
On Thu, 2013-05-02 at 11:54 +0100, Stefano Stabellini wrote: > On Thu, 2 May 2013, Ian Campbell wrote: > > On Thu, 2013-05-02 at 11:38 +0100, Stefano Stabellini wrote: > > > On Thu, 2 May 2013, Ian Campbell wrote: > > > > On Wed, 2013-05-01 at 20:27 +0100, Stefano Stabellini wrote: > > > > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > > > > > index d30042e..8f040a0 100644 > > > > > --- a/arch/arm/xen/enlighten.c > > > > > +++ b/arch/arm/xen/enlighten.c > > > > > @@ -14,6 +14,7 @@ > > > > > #include <xen/xen-ops.h> > > > > > #include <asm/xen/hypervisor.h> > > > > > #include <asm/xen/hypercall.h> > > > > > +#include <asm/arch_timer.h> > > > > > #include <asm/system_misc.h> > > > > > #include <linux/interrupt.h> > > > > > #include <linux/irqreturn.h> > > > > > @@ -172,6 +173,8 @@ static int __init xen_secondary_init(unsigned int cpu) > > > > > later ones fail to. */ > > > > > per_cpu(xen_vcpu, cpu) = vcpup; > > > > > } > > > > > + > > > > > + xen_setup_runstate_info(cpu); > > > > > > > > Is this called for the boot processor too? > > > > > > Yep. > > > > > > > (do you have a linux-next branch where I can easily find the baseline > > > > for this series?) > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git lost_ticks_1 > > > > Is that your Xen tree, I was after the Linux one... > > nope, that's my Linux tree disguised as a Xen tree to confuse the > maintainers ;-) ;-) > > > It's based on my 3.9-rc3-smp-6 branch currently in linux-next (that is > > > based on 3.9-rc3). > > > However lost_ticks_1 is not in linux-next yet. > > > > Which branch is in Linux next? It doesn't seem to be your linux-next > > (last commit 2011). > > that would be linux-next from > git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git Ah, I was looking in your xenbits tree -- is that deprecated? Ian.
On Thu, 2 May 2013, Ian Campbell wrote: > > > > It's based on my 3.9-rc3-smp-6 branch currently in linux-next (that is > > > > based on 3.9-rc3). > > > > However lost_ticks_1 is not in linux-next yet. > > > > > > Which branch is in Linux next? It doesn't seem to be your linux-next > > > (last commit 2011). > > > > that would be linux-next from > > git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git > > Ah, I was looking in your xenbits tree -- is that deprecated? I guess it is, now that I have a kernel.org account. I might as well get rid of it.
diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c index d30042e..8f040a0 100644 --- a/arch/arm/xen/enlighten.c +++ b/arch/arm/xen/enlighten.c @@ -14,6 +14,7 @@ #include <xen/xen-ops.h> #include <asm/xen/hypervisor.h> #include <asm/xen/hypercall.h> +#include <asm/arch_timer.h> #include <asm/system_misc.h> #include <linux/interrupt.h> #include <linux/irqreturn.h> @@ -172,6 +173,8 @@ static int __init xen_secondary_init(unsigned int cpu) later ones fail to. */ per_cpu(xen_vcpu, cpu) = vcpup; } + + xen_setup_runstate_info(cpu); return 0; } @@ -272,6 +275,7 @@ static int __init xen_guest_init(void) if (!xen_initial_domain()) xenbus_probe(NULL); + arch_timer_stolen_ticks = xen_stolen_accounting; pm_power_off = xen_power_off; arm_pm_restart = xen_restart;
Register the runstate_memory_area with the hypervisor. Use arch_timer_stolen_ticks and xen_stolen_accounting to account for stolen ticks. Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> --- arch/arm/xen/enlighten.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)