![]() | ||||||||||
Messages in this thread |
* 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 | ||||||||||