> -----Original Message-----
> From: Stephen Hemminger <step...@networkplumber.org>
> Sent: Thursday, February 23, 2023 5:56 AM
> To: dev@dpdk.org
> Cc: Stephen Hemminger <step...@networkplumber.org>; Yipeng Wang
> <yipeng1.w...@intel.com>;
> Sameh Gobriel <sameh.gobr...@intel.com>; Bruce Richardson
> <bruce.richard...@intel.com>;
> Vladimir Medvedkin <vladimir.medved...@intel.com>; Ruifeng Wang
> <ruifeng.w...@arm.com>
> Subject: [PATCH v11 21/22] hash: move rte_hash_set_alg out header
>
> The code for setting algorithm for hash is not at all perf sensitive, and
> doing it inline
> has a couple of problems. First, it means that if multiple files include the
> header, then
> the initialization gets done multiple times. But also, it makes it harder to
> fix usage of
> RTE_LOG().
>
> Despite what the checking script say. This is not an ABI change, the previous
> version
> inlined the same code; therefore both old and new code will work the same.
>
> Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
> ---
> lib/hash/meson.build | 1 +
> lib/hash/rte_crc_arm64.h | 8 ++---
> lib/hash/rte_crc_x86.h | 10 +++---
> lib/hash/rte_hash_crc.c | 68 ++++++++++++++++++++++++++++++++++++++++
> lib/hash/rte_hash_crc.h | 48 ++--------------------------
> lib/hash/version.map | 7 +++++
> 6 files changed, 88 insertions(+), 54 deletions(-) create mode 100644
> lib/hash/rte_hash_crc.c
>
Acked-by: Ruifeng Wang <ruifeng.w...@arm.com>
