* [PATCH][RESEND] Add back in <asm/system.h> and <linux/linkage.h> to <linux/interrupt.h>
@ 2002-11-22 15:21 Tom Rini
2002-11-22 15:55 ` Russell King
0 siblings, 1 reply; 3+ messages in thread
From: Tom Rini @ 2002-11-22 15:21 UTC (permalink / raw)
To: Linus Torvalds, Kernel Mailing List
The following trivial patch adds back <asm/system.h> and
<linux/kernel.h> to <linux/interrupt.h>. Without it,
<linux/interrupt.h> is relying on <asm/system.h> to be implicitly
included for smb_mb to be defined, and <linux/linkage.h> to be implicitly
included for asmlinkage/FASTCALL/etc.
Apparently RMK sent a similar patch, which did not add in
<linux/linkage.h>. That patch is incomplete since <linux/interrupt.h>
directly uses FASTCALL, asmlinkage, etc, and on some arches
<linux/linkage.h> will not be implicitly included.
--
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
===== include/linux/interrupt.h 1.17 vs edited =====
--- 1.17/include/linux/interrupt.h Sun Nov 17 09:23:25 2002
+++ edited/include/linux/interrupt.h Tue Nov 19 11:35:47 2002
@@ -3,11 +3,13 @@
#define _LINUX_INTERRUPT_H
#include <linux/config.h>
+#include <linux/linkage.h>
#include <linux/bitops.h>
#include <asm/atomic.h>
#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 *);
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH][RESEND] Add back in <asm/system.h> and <linux/linkage.h> to <linux/interrupt.h>
2002-11-22 15:21 [PATCH][RESEND] Add back in <asm/system.h> and <linux/linkage.h> to <linux/interrupt.h> Tom Rini
@ 2002-11-22 15:55 ` Russell King
2002-11-22 16:02 ` Tom Rini
0 siblings, 1 reply; 3+ messages in thread
From: Russell King @ 2002-11-22 15:55 UTC (permalink / raw)
To: Tom Rini; +Cc: Linus Torvalds, Kernel Mailing List
On Fri, Nov 22, 2002 at 08:21:32AM -0700, Tom Rini wrote:
> The following trivial patch adds back <asm/system.h> and
> <linux/kernel.h> to <linux/interrupt.h>. Without it,
> <linux/interrupt.h> is relying on <asm/system.h> to be implicitly
> included for smb_mb to be defined, and <linux/linkage.h> to be implicitly
> included for asmlinkage/FASTCALL/etc.
>
> Apparently RMK sent a similar patch, which did not add in
This bit is a myth.
--
Russell King (rmk@arm.linux.org.uk) The developer of ARM Linux
http://www.arm.linux.org.uk/personal/aboutme.html
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH][RESEND] Add back in <asm/system.h> and <linux/linkage.h> to <linux/interrupt.h>
2002-11-22 15:55 ` Russell King
@ 2002-11-22 16:02 ` Tom Rini
0 siblings, 0 replies; 3+ messages in thread
From: Tom Rini @ 2002-11-22 16:02 UTC (permalink / raw)
To: Linus Torvalds, Kernel Mailing List
On Fri, Nov 22, 2002 at 03:55:52PM +0000, Russell King wrote:
> On Fri, Nov 22, 2002 at 08:21:32AM -0700, Tom Rini wrote:
> > The following trivial patch adds back <asm/system.h> and
> > <linux/kernel.h> to <linux/interrupt.h>. Without it,
> > <linux/interrupt.h> is relying on <asm/system.h> to be implicitly
> > included for smb_mb to be defined, and <linux/linkage.h> to be implicitly
> > included for asmlinkage/FASTCALL/etc.
> >
> > Apparently RMK sent a similar patch, which did not add in
>
> This bit is a myth.
Well I blame Pete then. :)
--
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2002-11-22 15:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-11-22 15:21 [PATCH][RESEND] Add back in <asm/system.h> and <linux/linkage.h> to <linux/interrupt.h> Tom Rini
2002-11-22 15:55 ` Russell King
2002-11-22 16:02 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.