[PATCH v3] hash: add XOR32 hash function

Thomas Monjalon thomas at monjalon.net
Mon Feb 20 18:38:23 CET 2023


20/02/2023 18:21, Bili Dong:
> The naming is following the existing CRC32 hash:
> https://elixir.bootlin.com/dpdk/v22.11.1/source/lib/hash/rte_hash_crc.h#L168.
> I believe all existing hash functions in DPDK are 32 bits, so "32" didn't
> appear in other hash function names. If we add "32" here, we probably
> should also rename rte_hash_crc(). I'm fine with either option.

Why all functions would be 32-bit?
I don't think we need to rename all.
We can just make the right thing when adding a new function.

What maintainers of rte_hash think?




More information about the dev mailing list