On Thu, Oct 24, 2019 at 5:18 PM Martin KaFai Lau <kafai@xxxxxx> wrote: > > This patch makes a few changes to btf_ctx_access() to prepare > it for non raw_tp use case where the attach_btf_id is not > necessary a BTF_KIND_TYPEDEF. > > It moves the "btf_trace_" prefix check and typedef-follow logic to a new > function "check_attach_btf_id()" which is called only once during > bpf_check(). btf_ctx_access() only operates on a BTF_KIND_FUNC_PROTO > type now. That should also be more efficient since it is done only > one instead of every-time check_ctx_access() is called. > > "check_attach_btf_id()" needs to find the func_proto type from > the attach_btf_id. It needs to store the result into the > newly added prog->aux->attach_func_proto. func_proto > btf type has no name, so a proper name should be stored into > "attach_func_name" also. > > v2: > - Move the "btf_trace_" check to an earlier verifier phase (Alexei) > > Signed-off-by: Martin KaFai Lau <kafai@xxxxxx> Thank you for sending it early. It's right in the place that I'm hacking as well. I'll refactor my work to base on this. Applied. Thanks.