[PATCH v2 3/4] hash: reduce architecture special cases

Stephen Hemminger stephen at networkplumber.org
Tue Aug 26 15:41:49 CEST 2025


On Tue, 26 Aug 2025 08:55:23 +0200
Morten Brørup <mb at smartsharesystems.com> wrote:

> > +static int
> > +rte_hash_k64_cmp_eq(const void *key1, const void *key2, size_t key_len)
> > +{
> > +	return rte_hash_k32_cmp_eq(key1, key2, key_len) |  
> 
> Is the "|" instead of "||", to compare in blocks of 64 bytes instead of 32, intentional?

The cost of the conditional branch is usually higher than the cost of doing
a few more instructions on cached data.


More information about the dev mailing list