-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Proposal] Bump minimum LLVM version to 11 #3808
Labels
Comments
@rupran @bluca @brendangregg Please let us know if you see any issues with this minimum llvm version 11 requirement. |
no issue from my point of view |
Same from me, libdebuginfod support should work just fine. |
6 tasks
chantra
added a commit
to chantra/gobpf
that referenced
this issue
Jul 5, 2022
This is what BCC is going to be [using as base version in the future](iovisor/bcc#3808), so eventually, the current LLVM 9 will break.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently there is no explicitly-stated minimum version of LLVM that bcc supports linking against. Based on
INSTALL.md
and @yonghong-song's mailing list response from late 2020, which states3.7 seems to be the implicit minimum version. 3.7 is from mid-2015.
I propose that we bump the minimum supported version of LLVM to 11 (released late 2020).
Why?
bcc
is one of the more widely-deployed projects in the BPF ecosystem, the tools specifically are many folks' introduction to BPF. In order to keepbcc
working in as many environments as possible with minimum fuss we've historically tried to support a variety of older LLVMs. There are a few reasons why I think this is no longer the best path forward:bcc
with access to more cutting-edge libbpf features (e.g. global variables).bcc
- are rejected by the verifier, with upgrading to a more recent LLVM fixing the issue. This results in frustration and wasted time on the users' part ("I'm just trying to get this simple thing to work") and maintainers ("Didn't we fix this bad bytecode generation two years ago?").Why Now?
Similar to #3682, I feel that we've reached an inflection point where the drag of continuing to support old LLVMs is no longer worth it. Conversations with upstream BPF folks have convinced me that the second bullet above is more of an issue than I previously thought.
What needs to be done?
We need to update
bcc
packages on various distros, either in-tree (e.g.snap
dir) or not, to list the llvm 11 as the minimum required version.Need to confirm that Github Actions test workflows are testing with 11+ (preferably each major release will be tested). Currently ubuntu test container is using 11, fedora is using 12.
Then can remove LLVM version checks for older versions
The text was updated successfully, but these errors were encountered: