Skip to content

Latest commit

 

History

History
81 lines (63 loc) · 2.44 KB

CONTRIBUTING.md

File metadata and controls

81 lines (63 loc) · 2.44 KB
Raw

Contributing to kvm-bindings

Dependencies

Bindgen

The bindings are currently generated using bindgen version 0.59.1:

cargo install bindgen --vers 0.59.1

Linux Kernel

Generating bindings depends on the Linux kernel, so you need to have the repository on your machine:

git clone https://github.com/torvalds/linux.git

Add a new architecture

When adding a new architecture, the bindings must be generated for all existing versions for consistency reasons.

Example for arm64 and version 5.13

For this example we assume that you have both linux and kvm-bindings repositories in your root.

# Step 1: Create a new module using the name of the architecture in src/
cd kvm-bindings
mkdir src/arm64
cd ~

# linux is the repository that you cloned at the previous step.
cd linux
# Step 2: Checkout the version you want to generate the bindings for.
git checkout v5.13

# Step 3: Generate the bindings.
# This will generate the headers for the targeted architecture and place them
# in the user specified directory. In this case, we generate them in the
# arm64_v5_13_headers directory.
make headers_install ARCH=arm64 INSTALL_HDR_PATH=arm64_v5_13_headers
cd arm64_v5_13_headers
bindgen include/linux/kvm.h -o bindings_v5_13_0.rs \
  --with-derive-default \
  --with-derive-partialeq \
  -- -Iinclude
cd ~

# Step 4: Copy the generated file to the arm64 module.
cp linux/arm64_v5_13_headers/bindings_v5_13_0.rs

Steps 2, 3 and 4 must be repeated for each of the existing KVM versions. Don't forget to change the name of the bindings file using the appropriate version.

Now that we have the bindings generated, we can copy the module file from one of the existing modules as this is only changed when a new version is added.

cp arm/mod.rs arm64/

Also, you will need to add the new architecture to kvm-bindings/lib.rs.

Future Improvements

All the above steps are scriptable, so in the next iteration I will add a script to generate the bindings.

Testing

This crate is tested using rust-vmm-ci and Buildkite pipelines. Each new feature added to this crate must be accompanied by Buildkite steps for testing the following:

  • Release builds (using musl/gnu) with the new feature on arm and x86
  • Coverage test as specified in the rust-vmm-ci readme.