Thread (1 message) 1 message, 1 author, 2009-02-12

Re: [PATCH] Update jhash.h with the new version of Jenkins' hash

From: Eric W. Biederman <hidden>
Date: 2009-02-12 19:58:23
Also in: lkml, netfilter-devel

Jozsef Kadlecsik [off-list ref] writes:
On Wed, 11 Feb 2009, wli@movementarian.org wrote:
quoted
On Wed, Feb 11, 2009 at 11:19:20AM +0100, Jozsef Kadlecsik wrote:
quoted
 /* The golden ration: an arbitrary value */
-#define JHASH_GOLDEN_RATIO	0x9e3779b9
+#define JHASH_GOLDEN_RATIO	0xdeadbeef
0xdeadbeef is a really bad choice of an arbitrary value.  It is
already used in multiple places.  Only a few of which are currently
listed in linux/poison.h and if I happened to see that number in a
debug trace having to sort through all of the possible sources looks
like a major pain.

I don't really care what we call it but somehow 0xdeadbeef strikes me
as wrong and something that will make debugging harder.

Eric
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help