On Thu, Aug 19, 2010 at 02:23:09PM +0200, Andi Kleen wrote: > Namhyung Kim <namhyung@xxxxxxxxx> writes: > > > When I compiled allyesconfig'ed kernel with C=1, I got 1519 lines of > > following message: > > > > include/linux/mm.h:599:16: warning: potentially expensive pointer subtraction > > > > which is around 10% of total warnings. this was caused by page_to_pfn() macro > > so I think it's worth to remove it by calculating pointer subtraction > > manually. > > IMHO it would be better to simply disable the warning in sparse instead > of uglying the code just to work around sparse bogosity. It doesnt' seem > to make much sense. A subtraction followed by a shift is not expensive. "Bogosity" in question is quite real and proposed "fix" actually makes the things even worse. At least gcc bothers to optimize the division in there... Basically, sparse warns about pointer subtractions that divide by constants that are not powers of two. Which _does_ have non-trivial price, even with optimizations done by gcc. Posted patch is complete crap, since all it does is a) confuse sparse into not noticing the operation b) confuse gcc into not trying to optimize the thing, which actually makes the situation *worse*. Sheesh... -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
