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

From: Pavel Machek
Date: Mon Feb 25 2008 - 17:05:46 EST


On Mon 2008-02-25 11:01:08, Gregory Haskins wrote:
> From: Sven Dietrich <sdietrich@xxxxxxxxxx>

Why is this good idea?

> Signed-off-by: Sven Dietrich <sdietrich@xxxxxxxxxx>
> ---

>
> +config RTLOCK_DELAY
> + int "Default delay (in loops) for adaptive rtlocks"
> + range 0 1000000000
> + depends on ADAPTIVE_RTLOCK
> + default "10000"
> + help
> + This allows you to specify the maximum attempts a task will spin
> + attempting to acquire an rtlock before sleeping. The value is
> + tunable at runtime via a sysctl. A setting of 0 (zero) disables
> + the adaptive algorithm entirely.
> +

I believe you have _way_ too many config variables. If this can be set
at runtime, does it need a config option, too?

Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/