2008/10/23 Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>: > If kernel/trace/trace.c calls irqs_disabled_flags(), it should include > the include that defines irqs_disabled_flags(). You should not add it to > some other random include. > > It's also happening on m68k: > http://kisskb.ellerman.id.au/kisskb/buildresult/50641/ The fact is that other archs include this header into their asm/system.h. It seems that's because they need some non-traced irq_save/restore . I wanted to stay in the same approach because future use of raw_local_irq could be used elsewhere and seem to work perfectly whithout adding special headers on most arch. But there will be some bug report for each future use of these functions for alpha.... (and even m68k as it seems). What do you think? Should I let this patch as is or send a new one (and one other for m68k). -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html