On Wed, 18 Jul 2007 18:04:51 +0200 Arnd Bergmann <arnd at arndb.de> wrote: > + if (spu->number >= CRASH_NUM_SPUS) { > + WARN_ON(1); > + continue; > + } Could be if (WARN_ON(spu->number >= CRASH_NUM_SPUS)) continue;