lkml.org 
[lkml]   [2007]   [Feb]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: BUG in 2.6.20-rt8
On Sat, Feb 24, 2007 at 10:37:44PM -0800, David Miller wrote:
> From: Ingo Molnar <mingo@elte.hu>
> Date: Sun, 25 Feb 2007 07:27:47 +0100
>
> >
> > * Paul E. McKenney <paulmck@linux.vnet.ibm.com> wrote:
> >
> > > I got the following running stock 2.6.20-rt8 on an 4-CPU 1.8GHz
> > > Opteron box. The machine continued to run a few rounds of kernbench
> > > and LTP. Looks a bit scary -- a tasklet was "stolen" from
> > > __tasklet_action().
> > >
> > > Thoughts? In the meantime, kicking it off again to see if it repeats.
> >
> > > BUG: at kernel/softirq.c:559 __tasklet_action()
> >
> > this seems to happen very sporadically. Seems to happen more likely on
> > hyperthreading CPUs. It is very likely caused by the
> > redesign-tasklet-locking-to-be-sane patch below - which is a quick hack
> > of mine from early -rt days. Can you see any obvious bug in it? The
> > cmpxchg logic is certainly a bit ... tricky, locking-wise.
>
> Ingo, please don't use cmpxchg() in generic code, we support several
> processors that simply cannot do it.

OK, I will bite...

Why doesn't the traditional hash table of locks work here? Use the
cache-line address as input to the hash function, take the corresponding
lock, do the compare-and-exchange by hand, and then release the lock.
What am I missing here? Address aliasing do to memory being mapped into
multiple locations or something? (In that case, use only the portion
of the address within the page, right?)

I will agree that cmpxchg() has been abused pretty thoroughly in
some venues, but it does have legitimate uses.

Thanx, Paul

> Instead of saying "it's just something special in -rt for now", take
> it out now so that what you do eventually push upstream does get
> tested.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-02-26 02:55    [W:3.222 / U:0.720 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site