]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "[PATCH] LOG2: Alter get_order() so that it can make use of ilog2() on a constant"
authorLinus Torvalds <torvalds@woody.linux-foundation.org>
Wed, 7 Mar 2007 03:38:01 +0000 (19:38 -0800)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Wed, 7 Mar 2007 03:38:01 +0000 (19:38 -0800)
commit34caa741bee465b999a34b4034968a194518ad3c
treea4614cbaf4880b3f782b74df91bbfcac8c3308f2
parent339c6c26f0d08da2d0a36b7826b53e620e916a53
Revert "[PATCH] LOG2: Alter get_order() so that it can make use of ilog2() on a constant"

This reverts commit 947cb8a87756f148c840704467064f38d9e3d854.

The commit was buggy in multiple ways:
 - the conversion to ilog2() was incorrect to begin with
 - it tested the wrong #defines, so on all architectures but FRV you'd
   never see the bug except for constant arguments.
 - the new "get_order()" macro used its arguments multiple times, and
   didn't even parenthesize them properly
 - despite the comments, it was not true that you could use it for
   constant initializers, since not all architectures even use the
   generic page.h header file.

All of the problems are individually fixable, but it all boils down to:
better just revert it, and re-do it from scratch.

Cc: David Howells <dhowells@redhat.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
include/asm-generic/page.h