Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-18 Thread Thomas Gleixner
On Tue, 18 Jul 2017, dbasehore . wrote: > On Tue, Jul 18, 2017 at 3:22 PM, Thomas Gleixner wrote: > > The HPET consists of several independent comparator units. The kernel uses > > only a limited set of them. We can reserve one or more for that purpose, so > > it does not require any multiplexing

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-18 Thread dbasehore .
On Tue, Jul 18, 2017 at 3:22 PM, Thomas Gleixner wrote: > On Tue, 18 Jul 2017, dbasehore . wrote: >> On Tue, Jul 18, 2017 at 2:53 PM, Thomas Gleixner wrote: >> > On Tue, 18 Jul 2017, dbasehore . wrote: >> >> On Mon, Jul 17, 2017 at 11:40 PM, Thomas Gleixner >> >> wrote: >> >> > On Mon, 17 Jul 2

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-18 Thread Thomas Gleixner
On Tue, 18 Jul 2017, dbasehore . wrote: > On Tue, Jul 18, 2017 at 2:53 PM, Thomas Gleixner wrote: > > On Tue, 18 Jul 2017, dbasehore . wrote: > >> On Mon, Jul 17, 2017 at 11:40 PM, Thomas Gleixner > >> wrote: > >> > On Mon, 17 Jul 2017, dbasehore . wrote: > >> >> On Mon, Jul 17, 2017 at 6:33 PM,

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-18 Thread dbasehore .
On Tue, Jul 18, 2017 at 2:53 PM, Thomas Gleixner wrote: > On Tue, 18 Jul 2017, dbasehore . wrote: >> On Mon, Jul 17, 2017 at 11:40 PM, Thomas Gleixner wrote: >> > On Mon, 17 Jul 2017, dbasehore . wrote: >> >> On Mon, Jul 17, 2017 at 6:33 PM, Rafael J. Wysocki >> >> wrote: >> >> I could make a p

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-18 Thread Thomas Gleixner
On Tue, 18 Jul 2017, dbasehore . wrote: > On Mon, Jul 17, 2017 at 11:40 PM, Thomas Gleixner wrote: > > On Mon, 17 Jul 2017, dbasehore . wrote: > >> On Mon, Jul 17, 2017 at 6:33 PM, Rafael J. Wysocki > >> wrote: > >> I could make a patch to try it out. I would probably add a flag to rtc > >> time

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-18 Thread dbasehore .
On Mon, Jul 17, 2017 at 11:40 PM, Thomas Gleixner wrote: > On Mon, 17 Jul 2017, dbasehore . wrote: >> On Mon, Jul 17, 2017 at 6:33 PM, Rafael J. Wysocki wrote: >> I could make a patch to try it out. I would probably add a flag to rtc >> timers to indicate whether it wakes the system (default true

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-17 Thread Thomas Gleixner
On Mon, 17 Jul 2017, dbasehore . wrote: > On Mon, Jul 17, 2017 at 6:33 PM, Rafael J. Wysocki wrote: > I could make a patch to try it out. I would probably add a flag to rtc > timers to indicate whether it wakes the system (default true). We > would have to add a sync with the rtc irq and the rtc i

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-17 Thread dbasehore .
On Mon, Jul 17, 2017 at 6:33 PM, Rafael J. Wysocki wrote: > On Tue, Jul 18, 2017 at 2:30 AM, dbasehore . wrote: >> On Sat, Jul 15, 2017 at 5:39 AM, Rafael J. Wysocki >> wrote: >>> On Thursday, July 13, 2017 03:58:53 PM dbasehore . wrote: On Thu, Jul 13, 2017 at 8:09 AM, Rafael J. Wysocki

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-17 Thread Rafael J. Wysocki
On Tue, Jul 18, 2017 at 2:30 AM, dbasehore . wrote: > On Sat, Jul 15, 2017 at 5:39 AM, Rafael J. Wysocki wrote: >> On Thursday, July 13, 2017 03:58:53 PM dbasehore . wrote: >>> On Thu, Jul 13, 2017 at 8:09 AM, Rafael J. Wysocki >>> wrote: >>> > On Thu, Jul 13, 2017 at 9:32 AM, Peter Zijlstra

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-17 Thread dbasehore .
On Sat, Jul 15, 2017 at 5:39 AM, Rafael J. Wysocki wrote: > On Thursday, July 13, 2017 03:58:53 PM dbasehore . wrote: >> On Thu, Jul 13, 2017 at 8:09 AM, Rafael J. Wysocki wrote: >> > On Thu, Jul 13, 2017 at 9:32 AM, Peter Zijlstra >> > wrote: >> >> On Fri, Jul 07, 2017 at 05:03:00PM -0700, Der

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-15 Thread Rafael J. Wysocki
On Thursday, July 13, 2017 03:58:53 PM dbasehore . wrote: > On Thu, Jul 13, 2017 at 8:09 AM, Rafael J. Wysocki wrote: > > On Thu, Jul 13, 2017 at 9:32 AM, Peter Zijlstra > > wrote: > >> On Fri, Jul 07, 2017 at 05:03:00PM -0700, Derek Basehore wrote: > >>> Adds a new feature to tick to schedule w

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-13 Thread dbasehore .
On Thu, Jul 13, 2017 at 8:09 AM, Rafael J. Wysocki wrote: > On Thu, Jul 13, 2017 at 9:32 AM, Peter Zijlstra wrote: >> On Fri, Jul 07, 2017 at 05:03:00PM -0700, Derek Basehore wrote: >>> Adds a new feature to tick to schedule wakeups on a CPU during freeze. >>> This won't fully wake up the system

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-13 Thread Rafael J. Wysocki
On Thu, Jul 13, 2017 at 9:32 AM, Peter Zijlstra wrote: > On Fri, Jul 07, 2017 at 05:03:00PM -0700, Derek Basehore wrote: >> Adds a new feature to tick to schedule wakeups on a CPU during freeze. >> This won't fully wake up the system (devices are not resumed), but >> allow simple platform function

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-13 Thread Peter Zijlstra
On Fri, Jul 07, 2017 at 05:03:00PM -0700, Derek Basehore wrote: > Adds a new feature to tick to schedule wakeups on a CPU during freeze. > This won't fully wake up the system (devices are not resumed), but > allow simple platform functionality to be run during freeze with > little power impact. >

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-12 Thread Thomas Gleixner
Derek, On Wed, 12 Jul 2017, dbasehore . wrote: > On Wed, Jul 12, 2017 at 2:25 PM, Thomas Gleixner wrote: > > On Fri, 7 Jul 2017, Derek Basehore wrote: > >> +/* > >> + * Clockevent device may run during freeze > >> + */ > >> +# define CLOCK_EVT_FEAT_FREEZE_NONSTOP 0x000100 > > > > Is that re

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-12 Thread dbasehore .
On Wed, Jul 12, 2017 at 2:25 PM, Thomas Gleixner wrote: > On Fri, 7 Jul 2017, Derek Basehore wrote: >> Adds a new feature to tick to schedule wakeups on a CPU during freeze. >> This won't fully wake up the system (devices are not resumed), but >> allow simple platform functionality to be run durin

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-12 Thread Thomas Gleixner
On Fri, 7 Jul 2017, Derek Basehore wrote: > Adds a new feature to tick to schedule wakeups on a CPU during freeze. > This won't fully wake up the system (devices are not resumed), but > allow simple platform functionality to be run during freeze with > little power impact. > > This implementation

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-10 Thread dbasehore .
On Sat, Jul 8, 2017 at 9:05 AM, Andy Shevchenko wrote: > On Sat, Jul 8, 2017 at 3:03 AM, Derek Basehore wrote: >> Adds a new feature to tick to schedule wakeups on a CPU during freeze. >> This won't fully wake up the system (devices are not resumed), but >> allow simple platform functionality to

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-10 Thread Rafael J. Wysocki
On Friday, July 07, 2017 05:03:00 PM Derek Basehore wrote: > Adds a new feature to tick to schedule wakeups on a CPU during freeze. This is referred to "suspend-to-idle" nowadays. I guess I need to update the code to be more consistent with respect to the terminology. Thanks, Rafael

Re: [PATCH v5 2/5] tick: Add freeze timer events

2017-07-08 Thread Andy Shevchenko
On Sat, Jul 8, 2017 at 3:03 AM, Derek Basehore wrote: > Adds a new feature to tick to schedule wakeups on a CPU during freeze. > This won't fully wake up the system (devices are not resumed), but > allow simple platform functionality to be run during freeze with > little power impact. > > This imp

[PATCH v5 2/5] tick: Add freeze timer events

2017-07-07 Thread Derek Basehore
Adds a new feature to tick to schedule wakeups on a CPU during freeze. This won't fully wake up the system (devices are not resumed), but allow simple platform functionality to be run during freeze with little power impact. This implementation allows an idle driver to setup a timer event with the