lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Sun, 27 Jan 2019 09:31:48 -0800
From:   Linus Torvalds <>
To:     Florian La Roche <>
Cc:     Linux List Kernel Mailing <>,
        Crt Mori <>, Joe Perches <>,
        Davidlohr Bueso <>,
        Will Deacon <>,
        Peter Zijlstra <>,
        Florian La Roche <>
Subject: Re: [PATCH 1/1] int_sqrt() adjustments

On Sun, Jan 27, 2019 at 9:06 AM Florian La Roche
<> wrote:
> -       m = 1UL << (__fls(x) & ~1UL);
> +       m = 1UL << ((int)__fls(x) & ~1);

No need to add the cast. It doesn't do anything and just makes the code uglier.

The type of a shift operator is the type of the left side of the shift
(with the usual int-promotion).

The type of the right side simply doesn't matter (the *value* does, of
course, but because the value range is limited, the cast is just


Powered by blists - more mailing lists