chrismanning / psi

Rust library for Linux PSI

PSI (Pressure Stall Information)

Utilities for working with PSI on Linux 4.20+.

Usage

[dependencies]
psi = "0.1"

License

Copyright (c) Chris Manning

This project is licensed under the MPL-2.0 license.

psi's Issues

Make epoll an optional dependency

I'm using your crate to implement a prometheus exporter for PSI data. The only thing I really need from the crate itself is PSI file parsing: psi::Psi with FromStr + psi::PsiLine.

Your crate depends on epoll for monitoring of PSI events, but really in my case I don't need that part. It wouldn't matter as much if it compiled out of the box on macOS, where you obviously do not have neither epoll, nor real PSI.

It would be nice if it was possible to make monitor as an optional, but enabled by default, feature that pulls epoll.

Crate should enable "std" feature in log dependency

The psi crate will not compile if the "std" feature is not enabled in the "log" dependency. This breaks builds in case you disable default features in more recent log versions:

   Compiling psi v0.1.2
error[E0277]: the trait bound `SetLoggerError: std::error::Error` is not satisfied
  --> /.cargo/registry/src/index.crates.io-6f17d22bba15001f/psi-0.1.2/src/error.rs:53:41
   |
53 |             LoggingInitError(e) => Some(e),
   |                                         ^ the trait `std::error::Error` is not implemented for `SetLoggerError`
   |
   = note: required for the cast from `&SetLoggerError` to `&dyn std::error::Error`

I suppose the crate should explicitly enable the "std" feature in Cargo.toml.