Re: [PATCH]: linux/interrupt.h doesn't need sched.h

From: Alexey Dobriyan
Date: Mon Oct 29 2007 - 14:17:07 EST


On Mon, Oct 29, 2007 at 09:12:10PM +0300, wrote:
> On Mon, Oct 29, 2007 at 11:45:39AM -0200, Arnaldo Carvalho de Melo wrote:
> > --- a/include/linux/interrupt.h
> > +++ b/include/linux/interrupt.h
> > @@ -9,7 +9,6 @@
> > #include <linux/cpumask.h>
> > #include <linux/irqreturn.h>
> > #include <linux/hardirq.h>
> > -#include <linux/sched.h>
> > #include <linux/irqflags.h>
> > #include <asm/atomic.h>
> > #include <asm/ptrace.h>
>
> ACK the part that nothing needs sched.h, but it shouldn't change anything,
> because of interrupt.h => hardirq.h => smp_lock.h => sched.h dependency.
>
> [puts patch on cross-compile farm anyway]
> [starts finding scalpels]

Actually, NAK as-is. It immediately breaks on x86_64-allnoconfig et al.
I'll send updated version :)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/