On 10/9/19 1:51 PM, Martin Lau wrote: >> err = btf_check_all_metas(env); >> + if (err) >> + goto errout; >> + > Considering btf_vmlinux is already safe, any concern in making an extra > call to btf_check_all_types()? Only concern is additional memory resolved_* arrays will take. The usual cpu vs memory trade off. For this set I think extra while loop to skip btf modifiers is better than extra array. Yours could be different. And that's ok.