linux-kernel - Re: Revert "irq: Add tracepoint to softirq_raise"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 09 Mar 2011 12:12:28 -0500
From:	Steven Rostedt <rostedt@...dmis.org>
To:	monstr@...str.eu
Cc:	Frederic Weisbecker <fweisbec@...il.com>,
	LKML <linux-kernel@...r.kernel.org>, Ingo Molnar <mingo@...e.hu>
Subject: Re: Revert "irq: Add tracepoint to softirq_raise" - microblaze
 build

On Wed, 2011-03-09 at 18:01 +0100, Michal Simek wrote:
> Steven Rostedt wrote:
> > On Wed, 2011-03-09 at 13:20 +0100, Michal Simek wrote:
> >> Hi Frederic,
> >>
> >> I have compiled ftrace support for Microblaze and I have reached a compilation failure.
> >>
> >> I bisected it and I found that
> >> "irq: Add tracepoint to softirq_raise"
> >> (sha1:2bf2160d8805de64308e2e7c3cd97813cb58ed2f)
> >> caused it.
> >>
> >> I also found one thread which could be related.
> >> http://kerneltrap.org/mailarchive/linux-kernel/2010/9/8/4616941
> >>
> >> It is likely any missing header or so.
> >>
> >> Do you know what can be wrong?
> >>
> >> Thanks,
> >> Michal
> >>
> >>
> >> [ALL  ]   CC      kernel/trace/power-traces.o
> >> [ALL  ] In file included from 
> >> /mnt/projects/petalinux-v1.00-devel/software/linux-2.6.x/arch/microblaze/include/asm/irq.h:15,
> > 
> > Is there a reason that arch/microblaze/include/asm/irq.h includes
> > "linux/interrupt.h"?
> 
> Yep, it is. I tried to add some headers not remove them.


Do you mean it is not needed? And removing it fixes the problem?

> 
> Thanks,
> Michal
> 
> P.S.: Do you want to create proper patch? If not, I will do it.

I rather not add patches to systems that I can't build, boot and test.

If removing the header works, just add a:

Suggested-by: Steven Rostedt <rostedt@...dmis.org>

Thanks,

-- Steve



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/