Removes Config::external_internal_function_hash_collision.

src/program.rs

@@ -139,9 +139,7 @@ impl<T: Copy + PartialEq> FunctionRegistry<T> {
             } else {
                 ebpf::hash_symbol_name(&usize::from(value).to_le_bytes())
             };
-            if config.external_internal_function_hash_collision
-                && loader.get_function_registry().lookup_by_key(hash).is_some()
-            {
+            if loader.get_function_registry().lookup_by_key(hash).is_some() {
                 return Err(ElfError::SymbolHashCollision(hash));
             }
             hash

src/vm.rs

@@ -72,8 +72,6 @@ pub struct Config {
     pub noop_instruction_rate: u32,
     /// Enable disinfection of immediate values and offsets provided by the user in JIT
     pub sanitize_user_provided_values: bool,
-    /// Throw ElfError::SymbolHashCollision when a BPF function collides with a registered syscall
-    pub external_internal_function_hash_collision: bool,
     /// Have the verifier reject "callx r10"
     pub reject_callx_r10: bool,
     /// Avoid copying read only sections when possible
@@ -105,7 +103,6 @@ impl Default for Config {
             reject_broken_elfs: false,
             noop_instruction_rate: 256,
             sanitize_user_provided_values: true,
-            external_internal_function_hash_collision: true,
             reject_callx_r10: true,
             optimize_rodata: true,
             aligned_memory_mapping: true,