On Thu, 10 Sep 2020 at 18:59, Martin KaFai Lau <kafai@xxxxxx> wrote: > [...] > > + > > + if (!btf_struct_ids_match(&env->log, reg->off, reg->btf_id, *btf_id)) { > > + verbose(env, "R%d has incompatible type %s\n", regno, > > + kernel_type_name(reg->btf_id)); > The original log has the expected kernel type also which will be a useful info. > It could be a follow up. > > Acked-by: Martin KaFai Lau <kafai@xxxxxx> Good idea. I'll way for feedback from Daniel or Alexei, and roll it into a v4 or do a follow up. -- Lorenz Bauer | Systems Engineer 6th Floor, County Hall/The Riverside Building, SE1 7PB, UK www.cloudflare.com