Re: [PATCH 00/15] cleanups and optimizations

2013-12-17 Thread Thomas Gleixner
On Tue, 17 Dec 2013, Eliezer Tamir wrote: > On 17/12/2013 17:13, Peter Zijlstra wrote: > > On Tue, Dec 17, 2013 at 04:03:58PM +0200, Eliezer Tamir wrote: > >> > >> I'm not sure that subtracting the spin time is the optimal thing to do. > >> > >> The busy poll time is supposed to be limited to somet

Re: [PATCH 00/15] cleanups and optimizations

2013-12-17 Thread Eliezer Tamir
On 17/12/2013 17:13, Peter Zijlstra wrote: > On Tue, Dec 17, 2013 at 04:03:58PM +0200, Eliezer Tamir wrote: >> >> I'm not sure that subtracting the spin time is the optimal thing to do. >> >> The busy poll time is supposed to be limited to something less than 1ms. >> (I'm using 50us in most of my t

Re: [PATCH 00/15] cleanups and optimizations

2013-12-17 Thread Peter Zijlstra
On Tue, Dec 17, 2013 at 04:03:58PM +0200, Eliezer Tamir wrote: > n 17/12/2013 15:32, Peter Zijlstra wrote: > > > > Awesomeness.. you'll work on subtracting the spin time from the sleep > > time? > > Me or someone on our team will work on it. > > I'm not sure that subtracting the spin time is the

Re: [PATCH 00/15] cleanups and optimizations

2013-12-17 Thread Eliezer Tamir
n 17/12/2013 15:32, Peter Zijlstra wrote: > > Awesomeness.. you'll work on subtracting the spin time from the sleep > time? Me or someone on our team will work on it. I'm not sure that subtracting the spin time is the optimal thing to do. The busy poll time is supposed to be limited to somethin

Re: [PATCH 00/15] cleanups and optimizations

2013-12-17 Thread Peter Zijlstra
On Mon, Dec 16, 2013 at 07:48:32PM +0200, Eliezer Tamir wrote: > On 13/12/2013 15:56, Peter Zijlstra wrote: > > On Fri, Dec 13, 2013 at 12:49:01PM +0200, Eliezer Tamir wrote: > >> On 12/12/2013 16:08, Peter Zijlstra wrote: > >>> This series contains the preempt_enable_no_resched() cleanups that inc

Re: [PATCH 00/15] cleanups and optimizations

2013-12-16 Thread Eliezer Tamir
On 13/12/2013 15:56, Peter Zijlstra wrote: > On Fri, Dec 13, 2013 at 12:49:01PM +0200, Eliezer Tamir wrote: >> On 12/12/2013 16:08, Peter Zijlstra wrote: >>> This series contains the preempt_enable_no_resched() cleanups that include >>> spin_lock_bh() optimizations and local_clock() optimizations.

Re: [PATCH 00/15] cleanups and optimizations

2013-12-13 Thread Peter Zijlstra
On Fri, Dec 13, 2013 at 12:49:01PM +0200, Eliezer Tamir wrote: > On 12/12/2013 16:08, Peter Zijlstra wrote: > > This series contains the preempt_enable_no_resched() cleanups that include > > spin_lock_bh() optimizations and local_clock() optimizations. > > I'm trying to test this on tip/master. >

Re: [PATCH 00/15] cleanups and optimizations

2013-12-13 Thread Eliezer Tamir
On 12/12/2013 16:08, Peter Zijlstra wrote: > This series contains the preempt_enable_no_resched() cleanups that include > spin_lock_bh() optimizations and local_clock() optimizations. I'm trying to test this on tip/master. Patch 3 fails to apply on kenrel/softirq.c Thanks, Eliezer -- To unsubscri

Re: [PATCH 00/15] cleanups and optimizations

2013-12-12 Thread Mike Galbraith
On Thu, 2013-12-12 at 15:08 +0100, Peter Zijlstra wrote: > This series contains the preempt_enable_no_resched() cleanups that include > spin_lock_bh() optimizations and local_clock() optimizations. Those clock optimizations look particularly yummy. -Mike -- To unsubscribe from this list: send t

[PATCH 00/15] cleanups and optimizations

2013-12-12 Thread Peter Zijlstra
This series contains the preempt_enable_no_resched() cleanups that include spin_lock_bh() optimizations and local_clock() optimizations. This patch series does not include the generic idle loop stuff used to 'fix' the idle injection crap. Rafael, can we make that happen by simply ignoring pm-qos l