Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-14 Thread Peter Zijlstra
On Wed, May 13, 2015 at 07:09:40PM -0400, Sasha Levin wrote: > On 05/13/2015 09:43 AM, Peter Zijlstra wrote: > > So I suppose the below (compile tested) patch should fix things, but > > seeing how I've been up since 4am I might just have missed something > > obvious :-) > > I know exactly how

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-14 Thread Peter Zijlstra
On Wed, May 13, 2015 at 07:09:40PM -0400, Sasha Levin wrote: On 05/13/2015 09:43 AM, Peter Zijlstra wrote: So I suppose the below (compile tested) patch should fix things, but seeing how I've been up since 4am I might just have missed something obvious :-) I know exactly how you feel...

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread Sasha Levin
On 05/13/2015 09:43 AM, Peter Zijlstra wrote: > On Tue, May 12, 2015 at 09:52:09AM -0400, Sasha Levin wrote: >> Hey Peter, >> >> I seem to be hitting this warning with the latest -next (2015-05-11): >> >> [3344291.055800] WARNING: CPU: 0 PID: 9422 at kernel/time/hrtimer.c:802 >>

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread bsegall
Peter Zijlstra writes: > @@ -4038,7 +4041,7 @@ void init_cfs_bandwidth(struct cfs_bandwidth *cfs_b) > cfs_b->period = ns_to_ktime(default_cfs_period()); > > INIT_LIST_HEAD(_b->throttled_cfs_rq); > - hrtimer_init(_b->period_timer, CLOCK_MONOTONIC, HRTIMER_MODE_REL); > +

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Tue, May 12, 2015 at 09:52:09AM -0400, Sasha Levin wrote: > > Hey Peter, > > > > I seem to be hitting this warning with the latest -next (2015-05-11): > > > > [3344291.055800] WARNING: CPU: 0 PID: 9422 at kernel/time/hrtimer.c:802 > >

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread Peter Zijlstra
On Tue, May 12, 2015 at 09:52:09AM -0400, Sasha Levin wrote: > Hey Peter, > > I seem to be hitting this warning with the latest -next (2015-05-11): > > [3344291.055800] WARNING: CPU: 0 PID: 9422 at kernel/time/hrtimer.c:802 > hrtimer_forward+0x1f9/0x330() Indeed. So I can't seem to come up

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread Peter Zijlstra
On Tue, May 12, 2015 at 09:52:09AM -0400, Sasha Levin wrote: Hey Peter, I seem to be hitting this warning with the latest -next (2015-05-11): [3344291.055800] WARNING: CPU: 0 PID: 9422 at kernel/time/hrtimer.c:802 hrtimer_forward+0x1f9/0x330() Indeed. So I can't seem to come up with a

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread Ingo Molnar
* Peter Zijlstra pet...@infradead.org wrote: On Tue, May 12, 2015 at 09:52:09AM -0400, Sasha Levin wrote: Hey Peter, I seem to be hitting this warning with the latest -next (2015-05-11): [3344291.055800] WARNING: CPU: 0 PID: 9422 at kernel/time/hrtimer.c:802

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread Sasha Levin
On 05/13/2015 09:43 AM, Peter Zijlstra wrote: On Tue, May 12, 2015 at 09:52:09AM -0400, Sasha Levin wrote: Hey Peter, I seem to be hitting this warning with the latest -next (2015-05-11): [3344291.055800] WARNING: CPU: 0 PID: 9422 at kernel/time/hrtimer.c:802 hrtimer_forward+0x1f9/0x330()

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-13 Thread bsegall
Peter Zijlstra pet...@infradead.org writes: @@ -4038,7 +4041,7 @@ void init_cfs_bandwidth(struct cfs_bandwidth *cfs_b) cfs_b-period = ns_to_ktime(default_cfs_period()); INIT_LIST_HEAD(cfs_b-throttled_cfs_rq); - hrtimer_init(cfs_b-period_timer, CLOCK_MONOTONIC,

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-12 Thread Sasha Levin
On 04/22/2015 03:15 PM, tip-bot for Peter Zijlstra wrote: > Commit-ID: 5de2755c8c8b3a6b8414870e2c284914a2b42e4d > Gitweb: http://git.kernel.org/tip/5de2755c8c8b3a6b8414870e2c284914a2b42e4d > Author: Peter Zijlstra > AuthorDate: Tue, 20 May 2014 15:49:48 +0200 > Committer: Thomas

Re: [tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-05-12 Thread Sasha Levin
On 04/22/2015 03:15 PM, tip-bot for Peter Zijlstra wrote: Commit-ID: 5de2755c8c8b3a6b8414870e2c284914a2b42e4d Gitweb: http://git.kernel.org/tip/5de2755c8c8b3a6b8414870e2c284914a2b42e4d Author: Peter Zijlstra pet...@infradead.org AuthorDate: Tue, 20 May 2014 15:49:48 +0200 Committer:

[tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-04-22 Thread tip-bot for Peter Zijlstra
Commit-ID: 5de2755c8c8b3a6b8414870e2c284914a2b42e4d Gitweb: http://git.kernel.org/tip/5de2755c8c8b3a6b8414870e2c284914a2b42e4d Author: Peter Zijlstra AuthorDate: Tue, 20 May 2014 15:49:48 +0200 Committer: Thomas Gleixner CommitDate: Wed, 22 Apr 2015 17:06:52 +0200 hrtimer: Allow

[tip:timers/core] hrtimer: Allow concurrent hrtimer_start() for self restarting timers

2015-04-22 Thread tip-bot for Peter Zijlstra
Commit-ID: 5de2755c8c8b3a6b8414870e2c284914a2b42e4d Gitweb: http://git.kernel.org/tip/5de2755c8c8b3a6b8414870e2c284914a2b42e4d Author: Peter Zijlstra pet...@infradead.org AuthorDate: Tue, 20 May 2014 15:49:48 +0200 Committer: Thomas Gleixner t...@linutronix.de CommitDate: Wed, 22 Apr