This is an archive of the discontinued LLVM Phabricator instance.

[BPF] Fix a bug in BTF_KIND_TYPE_TAG generation
ClosedPublic

Authored by yonghong-song on Feb 14 2022, 5:59 PM.

Details

Summary

Kumar Kartikeya Dwivedi reported a bug ([1]) where BTF_KIND_TYPE_TAG types
are not generated.

Currently, BPF backend only generates BTF types which are used by
the program, e.g., global variables, functions and some builtin functions.
For example, suppose we have

struct task_struct {                  
  ... 
  struct task_group               *sched_task_group;
  struct mm_struct                *mm;
  ...
  pid_t                           pid;
  pid_t                           tgid;
  ...
}

If BPF program intends to access task_struct->pid and task_struct->tgid,
there really no need to generate BTF types for struct task_group
and mm_struct.

In BPF backend, during BTF generation, when generating BTF for struct
task_struct, if types for task_group and mm_struct have not been generated
yet, a Fixup structure will be created, which will be reexamined later
to instantiate into either a full type or a forward type.

In current implementation, if we have something like

struct foo {
   struct bar  __tag1    *f;
};

and when generating types for struct foo, struct bar type
has not been generated, the __tag1 will be lost during later
Fixup instantiation. This patch fixed this issue by properly
handling btf_type_tag's during Fixup instantiation stage.

[1] https://lore.kernel.org/bpf/20220210232411.pmhzj7v5uptqby7r@apollo.legion/

Diff Detail

Event Timeline

yonghong-song created this revision.Feb 14 2022, 5:59 PM
yonghong-song requested review of this revision.Feb 14 2022, 5:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 14 2022, 5:59 PM
ast accepted this revision.Feb 14 2022, 6:06 PM
This revision is now accepted and ready to land.Feb 14 2022, 6:06 PM
This revision was automatically updated to reflect the committed changes.