lkml.org 
[lkml]   [2008]   [Feb]   [19]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateTue, 19 Feb 2008 11:27:00 +0100
FromIngo Molnar <>
SubjectRe: Linux 2.6.25-rc2
* Pekka Enberg <penberg@cs.helsinki.fi> wrote:

> Mathieu, Christoph is on vacation and I'm not at all that familiar 
> with this cmpxchg_local() optimization, so if you could take a peek at 
> this bug report to see if you can spot something obviously wrong with 
> it, I would much appreciate that.

hm, it's bad for at least one other reason as well (which is probably 
unrelated to this crash):

 /*
  * Currently fastpath is not supported if preemption is enabled.
  */
 #if defined(CONFIG_FAST_CMPXCHG_LOCAL) && !defined(CONFIG_PREEMPT)
 #define SLUB_FASTPATH
 #endif
such !PREEMPT exceptions tend to show "i didnt want to think too hard 
about the preemptible case so just turn it off" thinking.

Also, why isnt this "SLUB_FASTPATH" flag done in the Kconfig space?

	Ingo
--
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: 2008-02-19 11:29    [from the cache]
©2003-2008