From: "Jason A. Donenfeld" Subject: Re: Re: [PATCH v2 1/4] siphash: add cryptographically secure hashtable function Date: Thu, 15 Dec 2016 22:16:23 +0100 Message-ID: Reply-To: kernel-hardening@lists.openwall.com Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: "kernel-hardening@lists.openwall.com" , Hannes Frederic Sowa , David Laight , Netdev , Jean-Philippe Aumasson , LKML , Linux Crypto Mailing List , "Daniel J . Bernstein" , Eric Biggers To: Linus Torvalds Return-path: List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: linux-crypto.vger.kernel.org On Thu, Dec 15, 2016 at 10:14 PM, Linus Torvalds wrote: > I think you can/should just use the natural alignment for "u64". > > For architectures that need 8-byte alignment, u64 will already be > properly aligned. For architectures (like x86-32) that only need > 4-byte alignment, you get it. I should have added mention of that with my previous email. For the parameters that are always a multiple of u64 -- namely, the key -- I now do that in v5 of the patchset. So this is already done.