Re: [PATCH] interrupt.h needs <asm/system.h>

[Pete Zaitcev <zaitcev@redhat.com>]

><asm/system.h> is needed for smp_mb(). Apparently this definition is pulled in
> some other way on ia32.
> 
> --- linux-2.5.48/include/linux/interrupt.h	Mon Nov 18 10:04:00 2002
> +++ linux-m68k-2.5.48/include/linux/interrupt.h	Mon Nov 18 15:35:14 2002
> @@ -8,6 +8,7 @@
>  #include <asm/hardirq.h>
>  #include <asm/ptrace.h>
>  #include <asm/softirq.h>
> +#include <asm/system.h>
>  
>  struct irqaction {
>  	void (*handler)(int, void *, struct pt_regs *);

Geert's patch looks correct to me.

By the way, I am curious, why do we never comment why a header
was included, like so: "#include <asm/system.h> /* smp_mb */"?
I suspect people are afraid that the comments get stale.

-- Pete