Clean up Tofino dynamic hash library. (#5059)

Signed-off-by: fruffy <[email protected]>
p4lang · Dec 17, 2024 · 8a91796 · 8a91796
1 parent 5dc7f87
commit 8a91796
Show file tree

Hide file tree

Showing 13 changed files with 22 additions and 1,130 deletions.
diff --git a/backends/tofino/CMakeLists.txt b/backends/tofino/CMakeLists.txt
@@ -229,8 +229,8 @@ set (BF_P4C_IR_SRCS
   bf-p4c/parde/match_register.cpp
   bf-p4c/parde/clot/clot.cpp
   bf-p4c/phv/phv.cpp
-  bf-utils/src/dynamic_hash/dynamic_hash.cpp
-  bf-utils/src/dynamic_hash/bfn_hash_algorithm.cpp
+  bf-utils/dynamic_hash/dynamic_hash.cpp
+  bf-utils/dynamic_hash/bfn_hash_algorithm.cpp
   )
 
 set(QUAL_BF_P4C_IR_SRCS)

diff --git a/backends/tofino/bf-p4c/control-plane/bfruntime_ext.h b/backends/tofino/bf-p4c/control-plane/bfruntime_ext.h
@@ -25,7 +25,7 @@
 #include "barefoot/p4info.pb.h"
 #pragma GCC diagnostic pop
 #include "backends/tofino/bf-p4c/device.h"
-#include "backends/tofino/bf-utils/include/dynamic_hash/bfn_hash_algorithm.h"
+#include "backends/tofino/bf-utils/dynamic_hash/bfn_hash_algorithm.h"
 #include "bfruntime.h"
 
 namespace BFN {

diff --git a/backends/tofino/bf-p4c/mau/hash_function.cpp b/backends/tofino/bf-p4c/mau/hash_function.cpp
@@ -23,7 +23,7 @@
 #include "lib/hex.h"
 #include "lib/big_int_util.h"
 #include "backends/tofino/bf-p4c/mau/hash_function.h"
-#include "backends/tofino/bf-utils/include/dynamic_hash/dynamic_hash.h"
+#include "backends/tofino/bf-utils/dynamic_hash/dynamic_hash.h"
 
 using namespace P4::literals;
 

diff --git a/backends/tofino/bf-p4c/mau/input_xbar.cpp b/backends/tofino/bf-p4c/mau/input_xbar.cpp
@@ -26,7 +26,7 @@
 #include "backends/tofino/bf-p4c/mau/resource_estimate.h"
 #include "backends/tofino/bf-p4c/mau/tofino/input_xbar.h"
 #include "backends/tofino/bf-p4c/phv/phv_fields.h"
-#include "backends/tofino/bf-utils/include/dynamic_hash/dynamic_hash.h"
+#include "backends/tofino/bf-utils/dynamic_hash/dynamic_hash.h"
 #include "lib/algorithm.h"
 #include "lib/bitops.h"
 #include "lib/bitrange.h"

diff --git a/backends/tofino/bf-p4c/mau/tofino/input_xbar.cpp b/backends/tofino/bf-p4c/mau/tofino/input_xbar.cpp
@@ -27,7 +27,7 @@
 #include "backends/tofino/bf-p4c/mau/resource.h"
 #include "backends/tofino/bf-p4c/mau/resource_estimate.h"
 #include "backends/tofino/bf-p4c/phv/phv_fields.h"
-#include "backends/tofino/bf-utils/include/dynamic_hash/dynamic_hash.h"
+#include "backends/tofino/bf-utils/dynamic_hash/dynamic_hash.h"
 #include "lib/algorithm.h"
 #include "lib/bitvec.h"
 #include "lib/bitops.h"

diff --git a/backends/tofino/bf-p4c/midend/fold_constant_hashes.h b/backends/tofino/bf-p4c/midend/fold_constant_hashes.h
@@ -23,7 +23,7 @@
 #include "ir/ir.h"
 #include "backends/tofino/bf-p4c/midend/type_checker.h"
 #include "backends/tofino/bf-p4c/mau/hash_function.h"
-#include "backends/tofino/bf-utils/include/dynamic_hash/dynamic_hash.h"
+#include "backends/tofino/bf-utils/dynamic_hash/dynamic_hash.h"
 
 namespace BFN {
 

diff --git a/.../bf-utils/src/dynamic_hash/CMakeLists.txt → ...fino/bf-utils/dynamic_hash/CMakeLists.txt b/.../bf-utils/src/dynamic_hash/CMakeLists.txt → ...fino/bf-utils/dynamic_hash/CMakeLists.txt
@@ -1,7 +1,9 @@
 project(libdynhash VERSION 0.1 LANGUAGES C)
 add_library(dynhash_o OBJECT
-  dynamic_hash.c
-  bfn_hash_algorithm.c)
+  dynamic_hash.cpp
+  bfn_hash_algorithm.cpp
+)
+
 add_library(dynhash SHARED EXCLUDE_FROM_ALL $<TARGET_OBJECTS:dynhash_o>)
 
 # static lib for compiler with no syslibs

diff --git a/...s/src/dynamic_hash/bfn_hash_algorithm.cpp → ...utils/dynamic_hash/bfn_hash_algorithm.cpp b/...s/src/dynamic_hash/bfn_hash_algorithm.cpp → ...utils/dynamic_hash/bfn_hash_algorithm.cpp
@@ -1,11 +1,13 @@
-#include "backends/tofino/bf-utils/include/dynamic_hash/bfn_hash_algorithm.h"
+#include "backends/tofino/bf-utils/dynamic_hash/bfn_hash_algorithm.h"
 
 #include <errno.h>
 #include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
+#include <vector>
+
 /**
  * These values are based on the crc calculations provided at this url:
  *     http://reveng.sourceforge.net/crc-catalogue/all.htm

diff --git a/...include/dynamic_hash/bfn_hash_algorithm.h → ...f-utils/dynamic_hash/bfn_hash_algorithm.h b/...include/dynamic_hash/bfn_hash_algorithm.h → ...f-utils/dynamic_hash/bfn_hash_algorithm.h
@@ -1,5 +1,5 @@
-#ifndef BACKENDS_TOFINO_BF_UTILS_INCLUDE_DYNAMIC_HASH_BFN_HASH_ALGORITHM_H_
-#define BACKENDS_TOFINO_BF_UTILS_INCLUDE_DYNAMIC_HASH_BFN_HASH_ALGORITHM_H_
+#ifndef BACKENDS_TOFINO_BF_UTILS_DYNAMIC_HASH_BFN_HASH_ALGORITHM_H_
+#define BACKENDS_TOFINO_BF_UTILS_DYNAMIC_HASH_BFN_HASH_ALGORITHM_H_
 
 #ifndef __KERNEL__
 #include <stdbool.h>
@@ -419,4 +419,4 @@ void initialize_crc_matrix(bfn_hash_algorithm_t *alg);
 void calculate_crc(bfn_hash_algorithm_t *alg, uint32_t hash_output_bits, uint8_t *stream,
                    uint32_t stream_len, uint8_t *crc);
 
-#endif /* BACKENDS_TOFINO_BF_UTILS_INCLUDE_DYNAMIC_HASH_BFN_HASH_ALGORITHM_H_ */
+#endif /* BACKENDS_TOFINO_BF_UTILS_DYNAMIC_HASH_BFN_HASH_ALGORITHM_H_ */
diff --git a/...f-utils/src/dynamic_hash/dynamic_hash.cpp → ...no/bf-utils/dynamic_hash/dynamic_hash.cpp b/...f-utils/src/dynamic_hash/dynamic_hash.cpp → ...no/bf-utils/dynamic_hash/dynamic_hash.cpp
@@ -1,4 +1,4 @@
-#include "backends/tofino/bf-utils/include/dynamic_hash/dynamic_hash.h"
+#include "backends/tofino/bf-utils/dynamic_hash/dynamic_hash.h"
 
 #include <assert.h>
 #include <inttypes.h>

diff --git a/...utils/include/dynamic_hash/dynamic_hash.h → ...fino/bf-utils/dynamic_hash/dynamic_hash.h b/...utils/include/dynamic_hash/dynamic_hash.h → ...fino/bf-utils/dynamic_hash/dynamic_hash.h
@@ -1,10 +1,10 @@
-#ifndef BACKENDS_TOFINO_BF_UTILS_INCLUDE_DYNAMIC_HASH_DYNAMIC_HASH_H_
-#define BACKENDS_TOFINO_BF_UTILS_INCLUDE_DYNAMIC_HASH_DYNAMIC_HASH_H_
+#ifndef BACKENDS_TOFINO_BF_UTILS_DYNAMIC_HASH_DYNAMIC_HASH_H_
+#define BACKENDS_TOFINO_BF_UTILS_DYNAMIC_HASH_DYNAMIC_HASH_H_
 
 #include <stdbool.h>
 #include <stdint.h>
 
-#include "backends/tofino/bf-utils/include/dynamic_hash/bfn_hash_algorithm.h"
+#include "backends/tofino/bf-utils/dynamic_hash/bfn_hash_algorithm.h"
 
 #ifdef __cplusplus
 extern "C" {
@@ -213,4 +213,4 @@ void determine_tofino_regs(const ixbar_init_t *ixbar_init, const ixbar_input_t *
 }
 #endif
 
-#endif /* BACKENDS_TOFINO_BF_UTILS_INCLUDE_DYNAMIC_HASH_DYNAMIC_HASH_H_ */
+#endif /* BACKENDS_TOFINO_BF_UTILS_DYNAMIC_HASH_DYNAMIC_HASH_H_ */