Re: [PATCH v2 07/67] fscache: Implement a hash function

From: Linus Torvalds
Date: Thu Dec 09 2021 - 12:24:42 EST


On Thu, Dec 9, 2021 at 8:54 AM David Howells <dhowells@xxxxxxxxxx> wrote:
>
> Implement a function to generate hashes. It needs to be stable over time
> and endianness-independent as the hashes will appear on disk in future
> patches.

I'm not actually seeing this being endianness-independent.

Is the input just regular 32-bit data in native word order? Because
then it's not endianness-independent, it's purely that there *is* no
endianness to the data at all and it is purely native data.

So the code may be correct, but the explanation is confusing. There is
absolutely nothing here that is about endianness.

Linus