Commit 0756ea3e authored by Alexei Starovoitov's avatar Alexei Starovoitov Committed by David S. Miller
Browse files

bpf: allow networking programs to use bpf_trace_printk() for debugging 



bpf_trace_printk() is a helper function used to debug eBPF programs.
Let socket and TC programs use it as well.
Note, it's DEBUG ONLY helper. If it's used in the program,
the kernel will print warning banner to make sure users don't use
it in production.

Signed-off-by: default avatarAlexei Starovoitov <ast@plumgrid.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ffeedafb

include/linux/bpf.h

+1 −0
Original line number Diff line number Diff line
@@ -150,6 +150,7 @@ struct bpf_array { 
u64 bpf_tail_call(u64 ctx, u64 r2, u64 index, u64 r4, u64 r5);

void bpf_prog_array_map_clear(struct bpf_map *map);

bool bpf_prog_array_compatible(struct bpf_array *array, const struct bpf_prog *fp);

const struct bpf_func_proto *bpf_get_trace_printk_proto(void);



#ifdef CONFIG_BPF_SYSCALL

void bpf_register_prog_type(struct bpf_prog_type_list *tl);

kernel/bpf/core.c

+4 −0
Original line number Diff line number Diff line
@@ -733,6 +733,10 @@ const struct bpf_func_proto bpf_ktime_get_ns_proto __weak; 
const struct bpf_func_proto bpf_get_current_pid_tgid_proto __weak;

const struct bpf_func_proto bpf_get_current_uid_gid_proto __weak;

const struct bpf_func_proto bpf_get_current_comm_proto __weak;

const struct bpf_func_proto * __weak bpf_get_trace_printk_proto(void)

{

	return NULL;

}



/* Always built-in helper functions. */

const struct bpf_func_proto bpf_tail_call_proto = {

kernel/trace/bpf_trace.c

+12 −8
Original line number Diff line number Diff line
@@ -147,6 +147,17 @@ static const struct bpf_func_proto bpf_trace_printk_proto = { 
	.arg2_type	= ARG_CONST_STACK_SIZE,

};



const struct bpf_func_proto *bpf_get_trace_printk_proto(void)

{

	/*

	 * this program might be calling bpf_trace_printk,

	 * so allocate per-cpu printk buffers

	 */

	trace_printk_init_buffers();



	return &bpf_trace_printk_proto;

}



static const struct bpf_func_proto *kprobe_prog_func_proto(enum bpf_func_id func_id)

{

	switch (func_id) {
@@ -168,15 +179,8 @@ static const struct bpf_func_proto *kprobe_prog_func_proto(enum bpf_func_id func 
		return &bpf_get_current_uid_gid_proto;

	case BPF_FUNC_get_current_comm:

		return &bpf_get_current_comm_proto;



	case BPF_FUNC_trace_printk:

		/*

		 * this program might be calling bpf_trace_printk,

		 * so allocate per-cpu printk buffers

		 */

		trace_printk_init_buffers();



		return &bpf_trace_printk_proto;

		return bpf_get_trace_printk_proto();

	default:

		return NULL;

	}

net/core/filter.c

+2 −0
Original line number Diff line number Diff line
@@ -1442,6 +1442,8 @@ sk_filter_func_proto(enum bpf_func_id func_id) 
		return &bpf_tail_call_proto;

	case BPF_FUNC_ktime_get_ns:

		return &bpf_ktime_get_ns_proto;

	case BPF_FUNC_trace_printk:

		return bpf_get_trace_printk_proto();

	default:

		return NULL;

	}

CRA Git | Maintained and supported by SUSTech CRA and CCSE