Re: [tip:x86/asm] bitops: Optimise get_order()

From: H. Peter Anvin
Date: Thu Mar 01 2012 - 16:03:10 EST


David, could you make a fix for this?

-hpa

On 02/29/2012 04:31 PM, Stephen Rothwell wrote:
On Wed, 29 Feb 2012 15:29:04 -0500 Paul Gortmaker<paul.gortmaker@xxxxxxxxxxxxx> wrote:

On Mon, Feb 20, 2012 at 6:20 PM, tip-bot for David Howells
<dhowells@xxxxxxxxxx> wrote:
Commit-ID: d66acc39c7cee323733c8503b9de1821a56dff7e
Gitweb: http://git.kernel.org/tip/d66acc39c7cee323733c8503b9de1821a56dff7e
Author: David Howells<dhowells@xxxxxxxxxx>
AuthorDate: Mon, 20 Feb 2012 22:39:29 +0000
Committer: H. Peter Anvin<hpa@xxxxxxxxx>
CommitDate: Mon, 20 Feb 2012 14:47:02 -0800

bitops: Optimise get_order()

This is causing build failures on non-x86 in linux next according to git bisect.

Presumably it needs to include linux/bitops.h (and see below).

+static inline __attribute_const__
+int __get_order(unsigned long size)
+{
+ int order;
+
+ size--;
+ size>>= PAGE_SHIFT;
+#if BITS_PER_LONG == 32
+ order = fls(size);
+#else
+ order = fls64(size);
+#endif

linux/bitops.h has fls_long() that does this size test and calls the right thing.


--
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/