lkml.org 
[lkml]   [2008]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH [RT] 08/14] add a loop counter based timeout mechanism
    On Thu, Feb 21, 2008 at 05:41:09PM +0100, Andi Kleen wrote:
    >
    > > +config RTLOCK_DELAY
    > > + int "Default delay (in loops) for adaptive rtlocks"
    > > + range 0 1000000000
    > > + depends on ADAPTIVE_RTLOCK
    >
    > I must say I'm not a big fan of putting such subtle configurable numbers
    > into Kconfig. Compilation is usually the wrong place to configure
    > such a thing. Just having it as a sysctl only should be good enough.
    >
    > > + default "10000"
    >
    > Perhaps you can expand how you came up with that default number?
    > It looks suspiciously round and worse the actual spin time depends a lot on the
    > CPU frequency (so e.g. a 3Ghz CPU will likely behave quite
    > differently from a 2Ghz CPU) Did you experiment with other spin times?
    > Should it be scaled with number of CPUs? And at what point is real
    > time behaviour visibly impacted?
    >
    > Most likely it would be better to switch to something that is more
    > absolute time, like checking RDTSC every few iteration similar to what
    > udelay does. That would be at least constant time.

    One approach would be to set the RTLOCK_DELAY parameter to something like
    -1 for default, and to set it to the number of cycles required for about
    10 cache misses at boot time. This would automatically scale with CPU
    frequency and memory latency.

    Thanx, Paul


    \
     
     \ /
      Last update: 2008-02-22 20:11    [W:4.301 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site