[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1191457909.8268.17.camel@localhost.localdomain>
Date: Thu, 04 Oct 2007 10:31:49 +1000
From: Rusty Russell <rusty@...tcorp.com.au>
To: davids@...master.com
Cc: Ingo Molnar <mingo@...e.hu>, linux-kernel@...r.kernel.org
Subject: RE: Network slowdown due to CFS
On Mon, 2007-10-01 at 09:49 -0700, David Schwartz wrote:
> > * Jarek Poplawski <jarkao2@...pl> wrote:
> >
> > > BTW, it looks like risky to criticise sched_yield too much: some
> > > people can misinterpret such discussions and stop using this at all,
> > > even where it's right.
>
> > Really, i have never seen a _single_ mainstream app where the use of
> > sched_yield() was the right choice.
>
> It can occasionally be an optimization. You may have a case where you can do
> something very efficiently if a lock is not held, but you cannot afford to
> wait for the lock to be released. So you check the lock, if it's held, you
> yield and then check again. If that fails, you do it the less optimal way
> (for example, dispatching it to a thread that *can* afford to wait).
This used to be true, and still is if you want to be portable. But the
point of futexes was precisely to attack this use case: whereas
sched_yield() says "I'm waiting for something, but I won't tell you
what" the futex ops tells the kernel what you're waiting for.
While the time to do a futex op is slightly slower than sched_yield(),
futexes win in so many cases that we haven't found a benchmark where
yield wins. Yield-lose cases include:
1) There are other unrelated process that yield() ends up queueing
behind.
2) The process you're waiting for doesn't conveniently sleep as soon as
it releases the lock, so you wait for longer than intended,
3) You race between the yield and the lock being dropped.
In summary: spin N times & futex seems optimal. The value of N depends
on the number of CPUs in the machine and other factors, but N=1 has
shown itself pretty reasonable.
Hope that helps,
Rusty.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists