Skip to content

Commit

Permalink
ci: Build statically linked bpf-linker in CI
Browse files Browse the repository at this point in the history
- Download LLVM builds from Rust CI.
- Always link libLLVM statically.
- For now, do it only for x86_64.
  • Loading branch information
vadorovsky committed Aug 14, 2024
1 parent 90a5ae4 commit 2b3fc8b
Show file tree
Hide file tree
Showing 2 changed files with 64 additions and 58 deletions.
78 changes: 29 additions & 49 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,9 @@ jobs:
- stable
- beta
- nightly
llvm:
- 19
- source
name: rustc=${{ matrix.rust }} llvm=${{ matrix.llvm }}
target:
- x86_64-unknown-linux-musl
name: rustc=${{ matrix.rust }} target=${{ matrix.target }}
needs: llvm

env:
Expand All @@ -81,7 +80,7 @@ jobs:
toolchain: ${{ matrix.rust }}
components: rust-src
# TODO: Remove this and run the integration tests on the local machine when they pass on 5.15.
targets: aarch64-unknown-linux-musl,x86_64-unknown-linux-musl
targets: ${{ matrix.target }}

- uses: Swatinem/rust-cache@v2

Expand All @@ -91,59 +90,36 @@ jobs:
- name: Build (default features, no system LLVM)
run: cargo build

- name: Install dependencies
if: matrix.rust == 'nightly'
# ubuntu-22.04 comes with clang 14[0] which doesn't include support for signed and 64bit
# enum values which was added in clang 15[1].
#
# gcc-multilib provides at least <asm/types.h> which is referenced by libbpf.
#
# llvm provides llvm-objcopy which is used to build the BTF relocation tests.
#
# [0] https://github.com/actions/runner-images/blob/ubuntu22/20230724.1/images/linux/Ubuntu2204-Readme.md
#
# [1] https://github.com/llvm/llvm-project/commit/dc1c43d
run: |
set -euxo pipefail
wget -qO- https://apt.llvm.org/llvm-snapshot.gpg.key | sudo tee /etc/apt/trusted.gpg.d/apt.llvm.org.asc
echo deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy main | sudo tee /etc/apt/sources.list.d/llvm.list
sudo apt update
sudo apt -y install clang gcc-multilib
- name: Install LLVM
if: matrix.llvm != 'source'
run: |
set -euxo pipefail
wget -qO- https://apt.llvm.org/llvm-snapshot.gpg.key | sudo tee /etc/apt/trusted.gpg.d/apt.llvm.org.asc
echo -e deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy-${{ matrix.llvm }} main | sudo tee /etc/apt/sources.list.d/llvm.list
sudo apt update
# TODO(vadorovsky): Remove the requirement of libpolly.
#
# Packages from apt.llvm.org are being built all at once, with one
# cmake build with superset of options, then different binaries and
# libraries are being included in different packages.
#
# That results in `llvm-config --libname --link-static` mentioning
# libpolly, even if it's not installed. The output of that command is
# being used in build.rs of llvm-sys, so building llvm-sys on such
# system is complaining about lack of libpolly.
#
# Hopefully that nightmare goes away once we switch to binstalls and
# ditch the system LLVM option.
sudo apt -y install llvm-${{ matrix.llvm }}-dev libpolly-${{ matrix.llvm }}-dev
echo /usr/lib/llvm-${{ matrix.llvm }}/bin >> $GITHUB_PATH
# - name: Install dependencies
# if: matrix.rust == 'nightly'
# # ubuntu-22.04 comes with clang 14[0] which doesn't include support for signed and 64bit
# # enum values which was added in clang 15[1].
# #
# # gcc-multilib provides at least <asm/types.h> which is referenced by libbpf.
# #
# # llvm provides llvm-objcopy which is used to build the BTF relocation tests.
# #
# # [0] https://github.com/actions/runner-images/blob/ubuntu22/20230724.1/images/linux/Ubuntu2204-Readme.md
# #
# # [1] https://github.com/llvm/llvm-project/commit/dc1c43d
# run: |
# run: |
# set -euxo pipefail
# wget https://ci-artifacts.rust-lang.org/rustc-builds/${{ env.LLVM_SHA }}/rust-dev-nightly-${{ env.LLVM_TARGET }}.tar.xz
# set -euxo pipefail
# wget -qO- https://apt.llvm.org/llvm-snapshot.gpg.key | sudo tee /etc/apt/trusted.gpg.d/apt.llvm.org.asc
# echo deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy main | sudo tee /etc/apt/sources.list.d/llvm.list
# sudo apt update
# sudo apt -y install clang gcc-multilib

- name: Restore LLVM
if: matrix.llvm == 'source'
uses: actions/cache/restore@v4
with:
path: llvm-install
key: ${{ needs.llvm.outputs.cache-key }}
fail-on-cache-miss: true

- name: Add LLVM to PATH && LD_LIBRARY_PATH
if: matrix.llvm == 'source'
run: |
set -euxo pipefail
echo "${{ github.workspace }}/llvm-install/bin" >> $GITHUB_PATH
Expand Down Expand Up @@ -220,3 +196,7 @@ jobs:
set -euxo pipefail
sudo apt install -y locate qemu-system-{arm,x86}
find test/.tmp -name 'vmlinuz-*' | RUSTFLAGS=-Cdebuginfo=line-directives-only xargs -t cargo xtask integration-test vm
- name: Setup tmate session
if: ${{ failure() }}
uses: mxschmitt/action-tmate@v3
44 changes: 35 additions & 9 deletions .github/workflows/llvm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,33 @@ on:
jobs:
llvm:
runs-on: ubuntu-22.04
strategy:
matrix:
target:
- x86_64-unknown-linux-musl
name: llvm
outputs:
cache-key: ${{ steps.cache-key.outputs.cache-key }}
steps:

# We retrieve the commit hashes of two repositories:
#
# - rust-lang/rust - the Rust monorepo. Rust CI is using commit hashes
# from the monorepo for annotating LLVM tarballs. We use it to download
# the tarball.
# - aya-rs/llvm-project - which is our fork of LLVM, kept in sync with
# rust-lang/llvm-project. We use it in case we can't download LLVM
# toolchain from Rust CI.
- id: ls-remote
run: |
set -euxo pipefail
value=$(git ls-remote https://github.com/aya-rs/llvm-project.git refs/heads/rustc/19.1-2024-07-30 | cut -f1)
echo "sha=$value" >> "$GITHUB_OUTPUT"
rustc_sha=$(git ls-remote https://github.com/rust-lang/rust.git HEAD | cut -f1)
aya_llvm_sha=$(git ls-remote https://github.com/aya-rs/llvm-project.git refs/heads/rustc/19.1-2024-07-30 | cut -f1)
echo "rustc-sha=$rustc_sha" >> "$GITHUB_OUTPUT"
echo "aya-llvm-sha=$aya_llvm_sha" >> "$GITHUB_OUTPUT"
- id: cache-key
run: echo "cache-key=llvm-${{ steps.ls-remote.outputs.sha }}-1" >> "$GITHUB_OUTPUT"
run: echo "cache-key=llvm-${{ matrix.target }}-${{ steps.ls-remote.outputs.rustc-sha }}-1" >> "$GITHUB_OUTPUT"

- name: Cache
id: cache-llvm
Expand All @@ -30,8 +45,19 @@ jobs:
key: ${{ steps.cache-key.outputs.cache-key }}
lookup-only: true

- name: Install Tools
- name: Download LLVM from Rust CI
id: download-llvm
if: steps.cache-llvm.outputs.cache-hit != 'true'
run: |
set -euxo pipefail
mkdir -p llvm-install
wget -q -O - https://ci-artifacts.rust-lang.org/rustc-builds/${{ steps.ls-remote.outputs.rustc-sha }}/rust-dev-nightly-${{ matrix.target }}.tar.xz | \
tar -xJf - --strip-components 1 -C llvm-install && \
echo "success=true" >> "$GITHUB_OUTPUT" || \
echo "success=false" >> "$GITHUB_OUTPUT"
- name: Install Tools
if: steps.cache-llvm.outputs.cache-hit != 'true' && steps.download-llvm.outputs.success != 'true'
run: |
set -euxo pipefail
wget -O - https://apt.kitware.com/keys/kitware-archive-latest.asc 2>/dev/null | \
Expand All @@ -44,15 +70,15 @@ jobs:
sudo apt -y install cmake ninja-build clang lld
- name: Checkout LLVM Source
if: steps.cache-llvm.outputs.cache-hit != 'true'
if: steps.cache-llvm.outputs.cache-hit != 'true' && steps.download-llvm.outputs.success != 'true'
uses: actions/checkout@v4
with:
repository: aya-rs/llvm-project
ref: ${{ steps.ls-remote.outputs.sha }}
ref: ${{ steps.ls-remote.outputs.aya-llvm-sha }}
path: llvm-project

- name: Configure LLVM
if: steps.cache-llvm.outputs.cache-hit != 'true'
if: steps.cache-llvm.outputs.cache-hit != 'true' && steps.download-llvm.outputs.success != 'true'
run: |
set -euxo pipefail
cmake \
Expand All @@ -73,7 +99,7 @@ jobs:
-DLLVM_USE_LINKER=lld
- name: Install LLVM
if: steps.cache-llvm.outputs.cache-hit != 'true'
if: steps.cache-llvm.outputs.cache-hit != 'true' && steps.download-llvm.outputs.success != 'true'
env:
# Create symlinks rather than copies to conserve disk space. At the time of this writing,
# GitHub-hosted runners have 14GB of SSD space
Expand All @@ -85,7 +111,7 @@ jobs:
run: cmake --build llvm-build --target install

- name: Rewrite LLVM Symlinks
if: steps.cache-llvm.outputs.cache-hit != 'true'
if: steps.cache-llvm.outputs.cache-hit != 'true' && steps.download-llvm.outputs.success != 'true'
# Move targets over the symlinks that point to them.
#
# This whole dance would be simpler if CMake supported CMAKE_INSTALL_MODE=MOVE.
Expand Down

0 comments on commit 2b3fc8b

Please sign in to comment.