more #[derive(Debug)]

Author: Marcondiro

Cargo.toml

@@ -8,6 +8,6 @@ description = "The Intel Processor Trace (Intel PT) Decoder Library is Intel's r
 repository = "https://github.com/sum-catnip/libipt-rs"
 
 [dependencies]
-libipt-sys = "0.2.0-rc.1"
+libipt-sys = { git = "https://github.com/sum-catnip/libipt-sys" }
 bitflags = "2.4.1"
 num_enum = "0.7.1"

src/block/block.rs

@@ -69,7 +69,7 @@ mod test {
 ///
 /// Instructions in this block are executed sequentially but are not necessarily
 /// contiguous in memory.  Users are expected to follow direct branches.
-#[derive(Clone, Copy)]
+#[derive(Debug, Clone, Copy)]
 pub struct Block(pub(super) pt_block);
 impl Block {
     /// The IP of the first instruction in this block.

src/block/decoder.rs

@@ -58,6 +58,7 @@ mod test {
 /// The decoder needs to be synchronized before it can be used.
 ///
 /// * `T` - The Callback Closure Type in the Config
+#[derive(Debug)]
 pub struct BlockDecoder<'a, T>(&'a mut pt_block_decoder, PhantomData<T>);
 impl<T> BlockDecoder<'_, T> {
     /// Allocate an Intel PT block decoder.

src/config/config.rs

@@ -208,6 +208,7 @@ where
 }
 
 /// A helper type to create the libipt Configuration instance
+#[derive(Debug)]
 pub struct ConfigBuilder<'a, T>(pt_config, PhantomData<&'a mut T>);
 impl<'a, T> ConfigBuilder<'a, T> {
     // when theres a bug here, there might be on in `new` too.
@@ -286,6 +287,7 @@ impl<'a> ConfigBuilder<'a, ()> {
 }
 
 /// A libipt configuration
+#[derive(Debug)]
 pub struct Config<'a, C>(pub(crate) Cow<'a, pt_config>, PhantomData<&'a mut C>);
 impl<'a, C> Config<'a, C> {
     /// Gets this configs buffer.

src/config/cpu.rs

@@ -42,6 +42,7 @@ mod test {
 
 bitflags! {
     /// i suppose this is relevant when/if amd finally gets intelpt support?
+    #[derive(Debug)]
     pub struct CpuVendor: u32 {
         const INTEL = pt_cpu_vendor_pcv_intel as u32;
         const UNKNOWN = pt_cpu_vendor_pcv_unknown as u32;

src/config/filter.rs

@@ -43,7 +43,7 @@ pub enum AddrConfig {
 }
 
 /// an address range inside the address filter
-#[derive(Clone, Copy)]
+#[derive(Debug, Clone, Copy)]
 pub struct AddrRange {
     /// This corresponds to the IA32_RTIT_ADDRn_A MSRs
     a: u64,
@@ -93,7 +93,7 @@ impl AddrRange {
 }
 
 /// the address filter configuration
-#[derive(Clone, Copy)]
+#[derive(Debug, Clone, Copy)]
 pub struct AddrFilter(pub(super) pt_conf_addr_filter);
 impl AddrFilter {
     #[inline]
@@ -141,6 +141,7 @@ impl AddrFilter {
     }
 }
 
+#[derive(Debug)]
 pub struct AddrFilterBuilder(pub(super) pt_conf_addr_filter);
 impl Default for AddrFilterBuilder {
     fn default() -> Self {

src/config/flags.rs

@@ -75,6 +75,7 @@ mod test {
 
 bitflags! {
     /// flags for the block decoder
+    #[derive(Debug)]
     pub struct BlockFlags: u8 {
         /// End a block after a call instruction
         const END_ON_CALL        = 0b00000001;
@@ -89,6 +90,7 @@ bitflags! {
 
 bitflags! {
     /// flags for the instruction flow decoder
+    #[derive(Debug)]
     pub struct InsnFlags: u8 {
         /// Enable tick events for timing updates
         const ENABLE_TICK_EVENTS = 0b00000001;
@@ -99,6 +101,7 @@ bitflags! {
 
 bitflags! {
     /// flags for the query decoder
+    #[derive(Debug)]
     pub struct QueryFlags: u8 {
         /// Preserve timing calibration on overflow
         const KEEP_TCAL_ON_OVF = 0b00000001;

src/config/freqency.rs

@@ -23,7 +23,7 @@ mod test {
 }
 
 /// Frequency values used for timing packets
-#[derive(Clone, Copy, Default)]
+#[derive(Debug, Clone, Copy, Default)]
 pub struct Frequency {
     /// The Mini Time Counter (MTC) frequency as defined in IA32_RTIT_CTL.MTCFreq.
     ///

src/event/mod.rs

@@ -154,7 +154,7 @@ impl From<pt_event> for Payload {
     }
 }
 
-#[derive(Clone, Copy)]
+#[derive(Debug, Clone, Copy)]
 pub struct Event(pub(crate) pt_event);
 impl Event {
     /// A flag indicating that the event IP has been suppressed.

src/event/qry.rs

@@ -49,7 +49,7 @@ mod test {
     }
 }
 
-#[derive(Clone, Copy, TryFromPrimitive)]
+#[derive(Debug, Clone, Copy, TryFromPrimitive)]
 #[repr(i32)]
 pub enum CondBranch {
     Taken = 1,
@@ -59,6 +59,7 @@ pub enum CondBranch {
 /// The decoder will work on the buffer defined in the config,
 /// it shall contain raw trace data and remain valid for the lifetime of the decoder.
 /// The decoder needs to be synchronized before it can be used.
+#[derive(Debug)]
 pub struct QueryDecoder<'a, T>(&'a mut pt_query_decoder, PhantomData<T>);
 impl<T> QueryDecoder<'_, T> {
     /// Allocate an Intel PT query decoder.