Skip to content

Commit 51595f9

Browse files
committed
Add types returns to bpf helpers
1 parent 4cf284a commit 51595f9

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

pythonbpf/bpf_helper_handler.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ def bpf_ktime_get_ns_emitter(call, map_ptr, module, builder, func, local_sym_tab
1313
fn_ptr_type = ir.PointerType(fn_type)
1414
fn_ptr = builder.inttoptr(helper_id, fn_ptr_type)
1515
result = builder.call(fn_ptr, [], tail=False)
16-
return result
16+
return result, ir.IntType(64)
1717

1818

1919
def bpf_map_lookup_elem_emitter(call, map_ptr, module, builder, func, local_sym_tab=None, struct_sym_tab=None, local_var_metadata=None):
@@ -60,7 +60,7 @@ def bpf_map_lookup_elem_emitter(call, map_ptr, module, builder, func, local_sym_
6060

6161
result = builder.call(fn_ptr, [map_void_ptr, key_ptr], tail=False)
6262

63-
return result
63+
return result, ir.PointerType()
6464

6565

6666
def bpf_printk_emitter(call, map_ptr, module, builder, func, local_sym_tab=None, local_var_metadata=None):
@@ -87,8 +87,8 @@ def bpf_printk_emitter(call, map_ptr, module, builder, func, local_sym_tab=None,
8787
elif isinstance(value, ast.FormattedValue):
8888
print("Formatted value:", ast.dump(value))
8989
# Assume int for now
90-
fmt_parts.append("%lld")
9190
if isinstance(value.value, ast.Name):
91+
fmt_parts.append("%lld")
9292
exprs.append(value.value)
9393
else:
9494
raise NotImplementedError(
@@ -266,7 +266,7 @@ def bpf_map_update_elem_emitter(call, map_ptr, module, builder, func, local_sym_
266266
result = builder.call(
267267
fn_ptr, [map_void_ptr, key_ptr, value_ptr, flags_const], tail=False)
268268

269-
return result
269+
return result, None
270270

271271

272272
def bpf_map_delete_elem_emitter(call, map_ptr, module, builder, func, local_sym_tab=None, struct_sym_tab=None, local_var_metadata=None):
@@ -321,7 +321,7 @@ def bpf_map_delete_elem_emitter(call, map_ptr, module, builder, func, local_sym_
321321
# Call the helper function
322322
result = builder.call(fn_ptr, [map_void_ptr, key_ptr], tail=False)
323323

324-
return result
324+
return result, None
325325

326326

327327
def bpf_get_current_pid_tgid_emitter(call, map_ptr, module, builder, func, local_sym_tab=None, local_var_metadata=None):
@@ -338,7 +338,7 @@ def bpf_get_current_pid_tgid_emitter(call, map_ptr, module, builder, func, local
338338
# Extract the lower 32 bits (PID) using bitwise AND with 0xFFFFFFFF
339339
mask = ir.Constant(ir.IntType(64), 0xFFFFFFFF)
340340
pid = builder.and_(result, mask)
341-
return pid
341+
return pid, ir.IntType(64)
342342

343343

344344
def bpf_perf_event_output_handler(call, map_ptr, module, builder, func, local_sym_tab=None, struct_sym_tab=None, local_var_metadata=None):
@@ -387,7 +387,7 @@ def bpf_perf_event_output_handler(call, map_ptr, module, builder, func, local_sy
387387

388388
result = builder.call(
389389
fn_ptr, [ctx_ptr, map_void_ptr, flags_val, data_void_ptr, size_val], tail=False)
390-
return result
390+
return result, None
391391
else:
392392
raise NotImplementedError(
393393
"Only simple object names are supported as data in perf event output.")

0 commit comments

Comments
 (0)