Re: [PATCH 1/4] hrtimers: provide a hrtimers_late_resume() call

2013-06-21 Thread Thomas Gleixner
On Fri, 21 Jun 2013, David Vrabel wrote: > On 21/06/13 15:32, Thomas Gleixner wrote: > However, since hrtimers require the use of a one-shot ticker and when > one-shot timers are resumed they are armed to fire immediately (see > tick_resume_oneshot()) this interrupt is sufficient to kick the requir

Re: [PATCH 1/4] hrtimers: provide a hrtimers_late_resume() call

2013-06-21 Thread David Vrabel
On 21/06/13 15:32, Thomas Gleixner wrote: > On Fri, 21 Jun 2013, David Vrabel wrote: >> On 21/06/13 08:53, Thomas Gleixner wrote: >>> This is the completely wrong approach. If an architecture does not >>> shut down the non boot cpus on suspend, then this wants to be handled >>> in the core code and

Re: [PATCH 1/4] hrtimers: provide a hrtimers_late_resume() call

2013-06-21 Thread Thomas Gleixner
On Fri, 21 Jun 2013, David Vrabel wrote: > On 21/06/13 08:53, Thomas Gleixner wrote: > > This is the completely wrong approach. If an architecture does not > > shut down the non boot cpus on suspend, then this wants to be handled > > in the core code and not in some random arch specific driver. >

Re: [PATCH 1/4] hrtimers: provide a hrtimers_late_resume() call

2013-06-21 Thread David Vrabel
On 21/06/13 08:53, Thomas Gleixner wrote: > On Thu, 20 Jun 2013, David Vrabel wrote: >> From: David Vrabel >> >> Xen suspends (and resumes) without disabling non-boot CPUs as doing so >> adds considerable delay to live migrations. A 4 VCPU guest had more >> than 200 ms of additional downtime if d

Re: [PATCH 1/4] hrtimers: provide a hrtimers_late_resume() call

2013-06-21 Thread Thomas Gleixner
On Thu, 20 Jun 2013, David Vrabel wrote: > From: David Vrabel > > Xen suspends (and resumes) without disabling non-boot CPUs as doing so > adds considerable delay to live migrations. A 4 VCPU guest had more > than 200 ms of additional downtime if disable_nonboot_cpus() was > called prior to susp

[PATCH 1/4] hrtimers: provide a hrtimers_late_resume() call

2013-06-20 Thread David Vrabel
From: David Vrabel Xen suspends (and resumes) without disabling non-boot CPUs as doing so adds considerable delay to live migrations. A 4 VCPU guest had more than 200 ms of additional downtime if disable_nonboot_cpus() was called prior to suspending. As a consequence, only high resolution timer