crush: remove redundant local variable
authorIlya Dryomov <idryomov@gmail.com>
Tue, 27 Sep 2016 10:35:55 +0000 (12:35 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Wed, 5 Oct 2016 21:02:10 +0000 (23:02 +0200)
Remove extra x1 variable, it's just temporary placeholder that
clutters the code unnecessarily.

Reflects ceph.git commit 0d19408d91dd747340d70287b4ef9efd89e95c6b.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
net/ceph/crush/mapper.c

index 511ade9..a421e90 100644 (file)
@@ -245,7 +245,7 @@ static int bucket_straw_choose(struct crush_bucket_straw *bucket,
 /* compute 2^44*log2(input+1) */
 static __u64 crush_ln(unsigned int xin)
 {
 /* compute 2^44*log2(input+1) */
 static __u64 crush_ln(unsigned int xin)
 {
-       unsigned int x = xin, x1;
+       unsigned int x = xin;
        int iexpon, index1, index2;
        __u64 RH, LH, LL, xl64, result;
 
        int iexpon, index1, index2;
        __u64 RH, LH, LL, xl64, result;
 
@@ -273,12 +273,11 @@ static __u64 crush_ln(unsigned int xin)
        /* RH*x ~ 2^48 * (2^15 + xf), xf<2^8 */
        xl64 = (__s64)x * RH;
        xl64 >>= 48;
        /* RH*x ~ 2^48 * (2^15 + xf), xf<2^8 */
        xl64 = (__s64)x * RH;
        xl64 >>= 48;
-       x1 = xl64;
 
        result = iexpon;
        result <<= (12 + 32);
 
 
        result = iexpon;
        result <<= (12 + 32);
 
-       index2 = x1 & 0xff;
+       index2 = xl64 & 0xff;
        /* LL ~ 2^48*log2(1.0+index2/2^15) */
        LL = __LL_tbl[index2];
 
        /* LL ~ 2^48*log2(1.0+index2/2^15) */
        LL = __LL_tbl[index2];