From mboxrd@z Thu Jan  1 00:00:00 1970
From: tglx@linutronix.de (Thomas Gleixner)
Date: Tue, 17 Mar 2009 09:29:35 +0100 (CET)
Subject: [ltt-dev] [PATCH] kernel/irq/handle.c fix kstat_irqs_cpu and
 irq_to_desc symbols v2
In-Reply-To: <20090317000859.GB19848@Krystal>
References: <20090316182227.GA2444@yoda.jdub.homelinux.org>
	<20090316194459.GD11878@Krystal>
	<20090316200115.GA22538@yoda.jdub.homelinux.org>
	<20090316202500.GA13872@Krystal>
	<20090316205829.GB22538@yoda.jdub.homelinux.org>
	<20090316215337.GB15119@Krystal>
	<20090316220518.GD22538@yoda.jdub.homelinux.org>
	<20090316231122.GA18113@Krystal>
	<alpine.LFD.2.00.0903170048120.29264@localhost.localdomain>
	<20090317000859.GB19848@Krystal>
Message-ID: <alpine.LFD.2.00.0903170919310.29264@localhost.localdomain>

On Mon, 16 Mar 2009, Mathieu Desnoyers wrote:

> * Thomas Gleixner (tglx at linutronix.de) wrote:
> > Is this a contribution to the "most useless patch of the week"
> > contest ?
> > 
> > You have my vote.
> > 
> 
> Count mine too ! :) Actually, this was not what it should look like.
> Here is my version. I am not totally convinced that the struct irq_desc
> is absolutely required, but in some include orders, it might matter.

I have not seen a compile failure report yet.

> However, the #endif around the EXPORT_SYMBOL is definitely needed.

Right.

>  	struct irq_desc *desc = irq_to_desc(irq);
>  	return desc ? desc->kstat_irqs[cpu] : 0;
>  }
> -#endif
>  EXPORT_SYMBOL(kstat_irqs_cpu);
> +#endif

Thanks,

	tglx