Re: [PATCH v3 1/3] siphash: add cryptographically secure hashtable function
From: Linus Torvalds
Date: Wed Dec 14 2016 - 19:26:57 EST
On Wed, Dec 14, 2016 at 3:34 PM, Jason A. Donenfeld <Jason@xxxxxxxxx> wrote:
>
> Or does your reasonable dislike of "word" still allow for the use of
> dword and qword, so that the current function names of:
dword really is confusing to people.
If you have a MIPS background, it means 64 bits. While to people with
Windows programming backgrounds it means 32 bits.
Please try to avoid using it.
As mentioned, I think almost everybody agrees on the "q" part being 64
bits, but that may just be me not having seen it in any other context.
And before anybody points it out - yes, we already have lots of uses
of "dword" in various places. But they tend to be mostly
hardware-specific - either architectures or drivers.
So I'd _prefer_ to try to keep "word" and "dword" away from generic
helper routines. But it's not like anything is really black and white.
Linus