0.3.0-beta.2

.github/workflows/ci.yml

@@ -0,0 +1,29 @@
+name: ci
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+
+env:
+  CARGO_TERM_COLOR: always
+
+jobs:
+  build-test:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          submodules: recursive
+      - name: Build
+        run: cargo build --verbose
+      - name: Run tests
+        run: cargo test --verbose
+      - name: Run tests in release mode
+        run: cargo test --verbose --release
+      - name: Clippy
+        run: cargo clippy

CHANGELOG.md

@@ -18,6 +18,7 @@ _This changelog documents only changes relevant to users, internal changes might
 - `Image.copy()` has been replaced by `Image.extend()`
 - `Image.add_cached()` now takes a `Rc<SectionCache>` instead of `&mut SectionCache` to ensure that the cache outlives the `Image`. 
 - Many packet/event types methods now take a `&self` instead of consuming `self`
+- Some simple methods are now `const`
 
 ### Removed
 

Cargo.toml

@@ -1,7 +1,10 @@
 [package]
 name = "libipt"
-version = "0.3.0-beta.1"
-authors = ["sum_catnip <catnip@catnip.fyi>", "Marcondiro <cavenatimarco+libiptrs@gmail.com>"]
+version = "0.3.0-beta.2"
+authors = [
+  "sum_catnip <catnip@catnip.fyi>",
+  "Marcondiro <cavenatimarco+libiptrs@gmail.com>",
+]
 edition = "2021"
 license = "MIT"
 description = "The Intel Processor Trace (Intel PT) Decoder Library is Intel's reference implementation for decoding Intel PT."
@@ -14,6 +17,6 @@ rust-version = "1.82.0"
 libipt_master = ["libipt-sys/libipt_master"]
 
 [dependencies]
-libipt-sys = "0.2.1-beta.3"
+libipt-sys = { version = "0.2.1-beta.3", git = "https://github.com/sum-catnip/libipt-sys.git" }
 bitflags = "2.4.1"
 num_enum = "0.7.1"

src/asid.rs

@@ -4,16 +4,26 @@ use libipt_sys::{pt_asid, pt_asid_no_cr3 as NO_CR3, pt_asid_no_vmcs as NO_VMCS};
 ///
 /// This identifies a particular address space when adding file sections or
 /// when reading memory.
-#[derive(Clone, Copy, Debug)]
+#[derive(Clone, Copy, Debug, Eq, PartialEq, Hash)]
+#[repr(transparent)]
 pub struct Asid(pub(crate) pt_asid);
 impl Asid {
     #[inline]
     #[must_use]
-    pub fn new(cr3: Option<u64>, vmcs: Option<u64>) -> Self {
+    pub const fn new(cr3: Option<u64>, vmcs: Option<u64>) -> Self {
+        let cr3_raw = match cr3 {
+            Some(v) => v,
+            None => NO_CR3,
+        };
+        let vmcs_raw = match vmcs {
+            Some(v) => v,
+            None => NO_VMCS,
+        };
+
         Asid(pt_asid {
             size: size_of::<pt_asid>(),
-            cr3: cr3.unwrap_or(NO_CR3),
-            vmcs: vmcs.unwrap_or(NO_VMCS),
+            cr3: cr3_raw,
+            vmcs: vmcs_raw,
         })
     }
 
@@ -56,18 +66,6 @@ impl Default for Asid {
     }
 }
 
-impl From<pt_asid> for Asid {
-    fn from(asid: pt_asid) -> Self {
-        Asid(asid)
-    }
-}
-
-impl PartialEq for Asid {
-    fn eq(&self, other: &Self) -> bool {
-        self.cr3() == other.cr3() && self.vmcs() == other.vmcs()
-    }
-}
-
 #[cfg(test)]
 mod test {
     use super::*;
@@ -126,7 +124,7 @@ mod test {
         assert_eq!(raw.cr3, NO_CR3);
         assert_eq!(raw.vmcs, NO_VMCS);
 
-        let mut asid2 = Asid::from(raw);
+        let mut asid2 = Asid(raw);
         asid2.set_cr3(666);
 
         assert_eq!(asid.cr3(), None);