[PATCH] D102712: BPF: Add more relocation kinds

Yonghong Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 18 17:34:53 PDT 2021


yonghong-song added a comment.

In D102712#2767238 <https://reviews.llvm.org/D102712#2767238>, @MaskRay wrote:

> These ELF flags should be documented somewhere. Even a draft will help.
>
> For example, riscv is using https://github.com/riscv/riscv-elf-psabi-doc/blob/master/riscv-elf.md#relocations , though not as sophisticated as other psABIs (x86-64,aarch64,ppc64 ELFv2)

This is a good suggestion. Currently, the main bpf compiler/instruction set related doc is living in the kernel tree

  https://www.kernel.org/doc/Documentation/networking/filter.rst

I think that is not a good place to put relocation as kernel people really don't care relocations much and
compiler/tool people may not be interested to check the kernel doc.

We need to find a different place to document some bpf compiler/tool internal stuff. I will add a little bit more
documentation in the code and in the commit message at this time. Will consider to put relocation description (maybe others)
in a public accessible document as a followup.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102712/new/

https://reviews.llvm.org/D102712



More information about the llvm-commits mailing list