linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH][RESEND] tracing: fix a build error on alpha
@ 2008-10-28  8:43 Frederic Weisbecker
  2008-10-28  8:53 ` Ingo Molnar
  2008-11-13 22:52 ` Alexey Dobriyan
  0 siblings, 2 replies; 5+ messages in thread
From: Frederic Weisbecker @ 2008-10-28  8:43 UTC (permalink / raw)
  To: Ingo Molnar; +Cc: Linux Kernel, Alexey Dobriyan

When tracing is enabled, some arch have included <linux/irqflags.h>
on their <asm/system.h> but others like alpha or m68k don't.

Build error on alpha:

kernel/trace/trace.c: In function 'tracing_cpumask_write':
kernel/trace/trace.c:2145: error: implicit declaration of function 'raw_local_irq_disable'
kernel/trace/trace.c:2162: error: implicit declaration of function 'raw_local_irq_enable'

Tested on Alpha through a cross-compiler (should correct a similar issue on m68k).

Reported-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
---
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c
index 306e433..c09493e 100644
--- a/kernel/trace/trace.c
+++ b/kernel/trace/trace.c
@@ -34,6 +34,7 @@
 
 #include <linux/stacktrace.h>
 #include <linux/ring_buffer.h>
+#include <linux/irqflags.h>
 
 #include "trace.h"
 

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH][RESEND] tracing: fix a build error on alpha
  2008-10-28  8:43 [PATCH][RESEND] tracing: fix a build error on alpha Frederic Weisbecker
@ 2008-10-28  8:53 ` Ingo Molnar
  2008-11-13 22:52 ` Alexey Dobriyan
  1 sibling, 0 replies; 5+ messages in thread
From: Ingo Molnar @ 2008-10-28  8:53 UTC (permalink / raw)
  To: Frederic Weisbecker; +Cc: Linux Kernel, Alexey Dobriyan


* Frederic Weisbecker <fweisbec@gmail.com> wrote:

> When tracing is enabled, some arch have included <linux/irqflags.h>
> on their <asm/system.h> but others like alpha or m68k don't.
> 
> Build error on alpha:
> 
> kernel/trace/trace.c: In function 'tracing_cpumask_write':
> kernel/trace/trace.c:2145: error: implicit declaration of function 'raw_local_irq_disable'
> kernel/trace/trace.c:2162: error: implicit declaration of function 'raw_local_irq_enable'
> 
> Tested on Alpha through a cross-compiler (should correct a similar issue on m68k).
> 
> Reported-by: Alexey Dobriyan <adobriyan@gmail.com>
> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>

applied to tip/tracing/urgent, thanks Frederic!

	Ingo

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH][RESEND] tracing: fix a build error on alpha
  2008-10-28  8:43 [PATCH][RESEND] tracing: fix a build error on alpha Frederic Weisbecker
  2008-10-28  8:53 ` Ingo Molnar
@ 2008-11-13 22:52 ` Alexey Dobriyan
  2008-11-13 23:11   ` Ingo Molnar
  2008-11-13 23:28   ` Frédéric Weisbecker
  1 sibling, 2 replies; 5+ messages in thread
From: Alexey Dobriyan @ 2008-11-13 22:52 UTC (permalink / raw)
  To: Frederic Weisbecker; +Cc: Ingo Molnar, Linux Kernel

On Tue, Oct 28, 2008 at 11:43 AM, Frederic Weisbecker
<fweisbec@gmail.com> wrote:
> +#include <linux/irqflags.h>

Already fixed.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH][RESEND] tracing: fix a build error on alpha
  2008-11-13 22:52 ` Alexey Dobriyan
@ 2008-11-13 23:11   ` Ingo Molnar
  2008-11-13 23:28   ` Frédéric Weisbecker
  1 sibling, 0 replies; 5+ messages in thread
From: Ingo Molnar @ 2008-11-13 23:11 UTC (permalink / raw)
  To: Alexey Dobriyan; +Cc: Frederic Weisbecker, Linux Kernel


* Alexey Dobriyan <adobriyan@gmail.com> wrote:

> On Tue, Oct 28, 2008 at 11:43 AM, Frederic Weisbecker
> <fweisbec@gmail.com> wrote:
> > +#include <linux/irqflags.h>
> 
> Already fixed.

Note that it's already fixed because the patch you are replying to 
(and which was sent 16 days ago) is already upstream:

  21798a8: tracing: fix a build error on alpha

	Ingo

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH][RESEND] tracing: fix a build error on alpha
  2008-11-13 22:52 ` Alexey Dobriyan
  2008-11-13 23:11   ` Ingo Molnar
@ 2008-11-13 23:28   ` Frédéric Weisbecker
  1 sibling, 0 replies; 5+ messages in thread
From: Frédéric Weisbecker @ 2008-11-13 23:28 UTC (permalink / raw)
  To: Alexey Dobriyan; +Cc: Ingo Molnar, Linux Kernel

2008/11/13 Alexey Dobriyan <adobriyan@gmail.com>:
> On Tue, Oct 28, 2008 at 11:43 AM, Frederic Weisbecker
> <fweisbec@gmail.com> wrote:
>> +#include <linux/irqflags.h>
>
> Already fixed.
>

And you were even put on Reported-by tag...

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2008-11-13 23:28 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-10-28  8:43 [PATCH][RESEND] tracing: fix a build error on alpha Frederic Weisbecker
2008-10-28  8:53 ` Ingo Molnar
2008-11-13 22:52 ` Alexey Dobriyan
2008-11-13 23:11   ` Ingo Molnar
2008-11-13 23:28   ` Frédéric Weisbecker

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).