Re: [(RT RFC) PATCH v2 6/9] add a loop counter based timeout mechanism

2008-02-26 Thread Gregory Haskins
>>> On Mon, Feb 25, 2008 at 5:06 PM, in message <[EMAIL PROTECTED]>, Pavel Machek <[EMAIL PROTECTED]> wrote: > > I believe you have _way_ too many config variables. If this can be set > at runtime, does it need a config option, too? Generally speaking, I think until this algorithm has an adapti

Re: [(RT RFC) PATCH v2 6/9] add a loop counter based timeout mechanism

2008-02-25 Thread Sven-Thorsten Dietrich
On Mon, 2008-02-25 at 23:06 +0100, Pavel Machek wrote: > On Mon 2008-02-25 11:01:08, Gregory Haskins wrote: > > From: Sven Dietrich <[EMAIL PROTECTED]> > > Why is this good idea? > The timeout is useful to eliminate excessive CPU utilization when waiting for long-held critical sections. The p

Re: [(RT RFC) PATCH v2 6/9] add a loop counter based timeout mechanism

2008-02-25 Thread Pavel Machek
On Mon 2008-02-25 14:19:47, Greg KH wrote: > On Mon, Feb 25, 2008 at 11:06:01PM +0100, Pavel Machek wrote: > > On Mon 2008-02-25 11:01:08, Gregory Haskins wrote: > > > From: Sven Dietrich <[EMAIL PROTECTED]> > > > > Why is this good idea? > > It preserves the original author when the patch is app

Re: [(RT RFC) PATCH v2 6/9] add a loop counter based timeout mechanism

2008-02-25 Thread Greg KH
On Mon, Feb 25, 2008 at 11:06:01PM +0100, Pavel Machek wrote: > On Mon 2008-02-25 11:01:08, Gregory Haskins wrote: > > From: Sven Dietrich <[EMAIL PROTECTED]> > > Why is this good idea? It preserves the original author when the patch is applied by git. Otherwise the author of the email would be c

Re: [(RT RFC) PATCH v2 6/9] add a loop counter based timeout mechanism

2008-02-25 Thread Pavel Machek
On Mon 2008-02-25 11:01:08, Gregory Haskins wrote: > From: Sven Dietrich <[EMAIL PROTECTED]> Why is this good idea? > Signed-off-by: Sven Dietrich <[EMAIL PROTECTED]> > --- > > +config RTLOCK_DELAY > + int "Default delay (in loops) for adaptive rtlocks" > + range 0 10 > + d

[(RT RFC) PATCH v2 6/9] add a loop counter based timeout mechanism

2008-02-25 Thread Gregory Haskins
From: Sven Dietrich <[EMAIL PROTECTED]> Signed-off-by: Sven Dietrich <[EMAIL PROTECTED]> --- kernel/Kconfig.preempt| 11 +++ kernel/rtmutex.c |4 kernel/rtmutex_adaptive.h | 11 +-- kernel/sysctl.c | 12 4 files changed, 36 inser