From: Sasha Levin Subject: Re: linux/bitops.h Date: Fri, 6 May 2016 16:08:27 -0400 Message-ID: <572CF9BB.3060902@oracle.com> References: <1462170413-7164-1-git-send-email-tytso@mit.edu> <1462170413-7164-2-git-send-email-tytso@mit.edu> <20160504174901.GC3901@thunk.org> <20160504190723.GD3901@thunk.org> <572A6CDD.10503@av8n.com> <572A724C.6010704@av8n.com> <572A9437.4020208@zytor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: John Denker , "Theodore Ts'o" , noloader@gmail.com, Linux Kernel Mailing List , Stephan Mueller , Herbert Xu , Andi Kleen , Sandy Harris , cryptography@lakedaemon.net, Linux Crypto Mailing List To: Linus Torvalds , "H. Peter Anvin" Return-path: Received: from aserp1040.oracle.com ([141.146.126.69]:48120 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758232AbcEFUI6 (ORCPT ); Fri, 6 May 2016 16:08:58 -0400 In-Reply-To: Sender: linux-crypto-owner@vger.kernel.org List-ID: On 05/04/2016 08:48 PM, Linus Torvalds wrote: > That said, the fact that the other cases weren't changed > (rol64/ror64/ror32) does make that argument less interesting. Unless > there was some particular code that actively ended up using > "rol32(..0)" but not the other cases. Right, the others seemed wrong as well but I couldn't find any code that triggers that, and preferred to fix just the one I was hitting. I can go fix the rest if that's something we want to do? Thanks, Sasha