Merge pull request deepfence#50 from deepfence/tomasz/ebpf-build

eBPF: add compiled release mode build

Author: tomaszjonak

docs/gh/development.md

@@ -6,17 +6,16 @@ All commands should be executed from repository/workspace root folder unless not
 
 ## Compilation
 
-First compile ebpf bytecode with the following command. It will be embedded
-in userspace binary using aya.
+Just utilize cargo infrastructure.
 
 ```
-$ cargo xtask build-ebpf
+$ cargo build
 ```
 
-Then userspace code.
+If you make changes to any code under `ebpfguard-ebpf` and/or `ebpfguard-common` make sure to rebuild eBPF objects.
 
 ```
-$ cargo build
+$ cargo xtask build-ebpf
 ```
 
 ## Tests
@@ -41,8 +40,18 @@ Clippy lints.
 $ cargo clippy --workspace -- --deny warnings
 ```
 
-Miri verification. Requires optional dependencies from [miri section](prerequisites.md#miri)
+Miri verification.
 
 ```
 $ cargo +nightly miri test --all-targets
 ```
+
+## Contributing
+
+Before setting up a PR make sure to run
+
+```
+cargo clippy --fix && cargo fmt
+```
+
+And verify/commit any resulting changes.

ebpfguard-ebpf/ebpfguard.debug.obj

@@ -29,6 +29,30 @@ impl std::fmt::Display for Architecture {
     }
 }
 
+#[derive(Debug, Copy, Clone)]
+pub enum BuildType {
+    Debug,
+    Release,
+}
+
+impl From<bool> for BuildType {
+    fn from(value: bool) -> Self {
+        match value {
+            true => Self::Release,
+            false => Self::Debug,
+        }
+    }
+}
+
+impl std::fmt::Display for BuildType {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        f.write_str(match self {
+            Self::Release => "release",
+            Self::Debug => "debug",
+        })
+    }
+}
+
 #[derive(Debug, Parser)]
 pub struct Options {
     /// Set the endianness of the BPF target
@@ -49,7 +73,9 @@ pub fn build_ebpf(opts: Options) -> Result<(), anyhow::Error> {
         "-Z",
         "build-std=core",
     ];
-    if opts.release {
+    let build_type = BuildType::from(opts.release);
+
+    if matches!(build_type, BuildType::Release) {
         args.push("--release")
     }
 
@@ -64,5 +90,12 @@ pub fn build_ebpf(opts: Options) -> Result<(), anyhow::Error> {
         .status()
         .expect("failed to build bpf program");
     assert!(status.success());
+
+    let source = format!("target/{}/{}/ebpfguard", opts.target, build_type);
+    let destination = format!("ebpfguard-ebpf/ebpfguard.{}.obj", build_type);
+
+    std::fs::copy(source, destination)
+        .expect("Couldn't copy compiled eBPFObject to destination path");
+
     Ok(())
 }