Re: [-mm patch] arch/i386/kernel/apic.c: make modern_apic() static

From: Adrian Bunk
Date: Fri Mar 31 2006 - 10:13:11 EST


On Fri, Mar 31, 2006 at 05:02:19PM +0200, Andi Kleen wrote:
> On Friday 31 March 2006 16:56, Adrian Bunk wrote:
> > This patch makes a nnedlessly global function static.
>
> Disagree. It will be likely used in more code in the future.

If this "likely" case becomes reality at any time in the future,
reverting my patch will be trivial.

OTOH, I have seen too many cases where people have said "I will need
this soon", and one year later it was still unused.

> -Andi

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

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