Skip to content

GCC BPF: use pre-built release#337

Merged
theihor merged 1 commit intokernel-patches:masterfrom
theihor:gcc-bpf-release
Mar 4, 2025
Merged

GCC BPF: use pre-built release#337
theihor merged 1 commit intokernel-patches:masterfrom
theihor:gcc-bpf-release

Conversation

@theihor
Copy link
Contributor

@theihor theihor commented Feb 3, 2025

See the commit message.

@theihor
Copy link
Contributor Author

theihor commented Feb 5, 2025

The test was successful, but I don't see a good reason to use this approach for now.

@theihor theihor closed this Feb 5, 2025
@theihor
Copy link
Contributor Author

theihor commented Mar 4, 2025

Ok, actually we might want to use this in BPF CI.

The problem with current implementation is that many copies of the same GCC build are cached, because caches between pull requests are separate. So using a pre-built release is much more reasonable.

I am going to restore and update this change to use latest release at https://github.com/theihor/gcc-bpf

@theihor theihor reopened this Mar 4, 2025
Running libbpf/ci/build-bpf-gcc action builds GCC from latest source
snapshot and caches the output with actions/cache.

Because of actions/cache rules, caches between pull requests are
independent. This results in cache being full of identical GCC builds,
which interferes with caching actually useful things.

To avoid this, and also to not re-build GCC all the time, download a
pre-built GCC and use it. The "build release" job is set up in an
independent repository, publishing artifacts as a release tag.

Signed-off-by: Ihor Solodrai <ihor.solodrai@linux.dev>
@theihor theihor changed the title Test downloading pre-built GCC BPF GCC BPF: use pre-built release Mar 4, 2025
@theihor theihor marked this pull request as ready for review March 4, 2025 19:13
@theihor theihor merged commit 81846c8 into kernel-patches:master Mar 4, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant