From 14e534265aad0796690a5aea429f5bd2877ccff0 Mon Sep 17 00:00:00 2001 From: Richard Henderson Date: Wed, 13 Feb 2013 17:47:40 -0800 Subject: [PATCH] target-i386: Inline bitops_flsl Use clz32 directly. Which makes slightly more sense given that the input is type "int" and not type "long". Signed-off-by: Richard Henderson Reviewed-by: Eric Blake Signed-off-by: Blue Swirl --- target-i386/topology.h | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/target-i386/topology.h b/target-i386/topology.h index 24ed525453..07a6c5fb55 100644 --- a/target-i386/topology.h +++ b/target-i386/topology.h @@ -52,10 +52,8 @@ typedef uint32_t apic_id_t; static unsigned apicid_bitwidth_for_count(unsigned count) { g_assert(count >= 1); - if (count == 1) { - return 0; - } - return bitops_flsl(count - 1) + 1; + count -= 1; + return count ? 32 - clz32(count) : 0; } /* Bit width of the SMT_ID (thread ID) field on the APIC ID -- 2.11.4.GIT