Merge #27

27: Add MSRV of 1.63 r=bertptrs a=bertptrs

Fixes #26.

Co-authored-by: Bert Peters <bert@bertptrs.nl>

.github/workflows/ci.yml

@@ -15,6 +15,7 @@ jobs:
     strategy:
       matrix:
         rust:
+          - "1.63" # minimum stable rust version
           - stable
           - beta
           - nightly

CHANGELOG.md

@@ -6,6 +6,21 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+### Added
+
+- The minimum supported Rust version is now defined as 1.63. Previously it was undefined.
+- Wrappers for `std::sync` primitives can now be `const` constructed.
+
+### Breaking
+
+- Update [`parking_lot`][parking_lot] dependency to `0.12`.
+- Restructured the crate to reduce typename verbosity. For details, see: #25.
+
+### Fixed
+
+- Enforce that all internal mutex guards are `!Send`. They already should be according to other
+  reasons, but this adds extra security through the type system.
+
 ## [0.2.1] - 2022-05-23
 
 ### Added

Cargo.toml

@@ -10,6 +10,7 @@ keywords = ["mutex", "rwlock", "once", "thread"]
 description = "Ensure deadlock-free mutexes by allocating in order, or else."
 readme = "README.md"
 repository = "https://github.com/bertptrs/tracing-mutex"
+rust-version = "1.63"
 
 [package.metadata.docs.rs]
 # Build docs for all features so the documentation is more complete
@@ -20,7 +21,7 @@ rustdoc-args = ["--cfg", "docsrs"]
 [dependencies]
 lazy_static = "1"
 lock_api = { version = "0.4", optional = true }
-parking_lot = { version = "0.11", optional = true }
+parking_lot = { version = "0.12", optional = true }
 
 [dev-dependencies]
 criterion = "0.3"

README.md

@@ -23,10 +23,12 @@ tree out of it, and panics if your dependencies would create a cycle. It provide
 existing synchronization primitives with an identical API, and should be a drop-in replacement.
 
 Inspired by [this blogpost][whileydave], which references a similar behaviour implemented by
-[Abseil][abseil-mutex] for their mutexes.
+[Abseil][abseil-mutex] for their mutexes. [This article goes into more depth on the exact
+implementation.][article]
 
 [whileydave]: https://whileydave.com/2020/12/19/dynamic-cycle-detection-for-lock-ordering/
 [abseil-mutex]: https://abseil.io/docs/cpp/guides/synchronization
+[article]: https://bertptrs.nl/2022/06/23/deadlock-free-mutexes-and-directed-acyclic-graphs.html
 
 ## Usage
 
@@ -42,9 +44,9 @@ Replacements for the synchronization primitives in `std::sync` can be found in t
 Support for other synchronization primitives is planned.
 
 ```rust
-use tracing_mutex::stdsync::TracingMutex;
+use tracing_mutex::stdsync::Mutex;
 
-let some_mutex = TracingMutex::new(42);
+let some_mutex = Mutex::new(42);
 *some_mutex.lock().unwrap() += 1;
 println!("{:?}", some_mutex);
 ```
@@ -59,6 +61,9 @@ performance penalty in your production environment, this library also offers deb
 when debug assertions are enabled, and to `Mutex` when they are not. Similar helper types are
 available for other synchronization primitives.
 
+The minimum supported Rust version is 1.63. Increasing this is not considered a breaking change, but
+will be avoided within semver-compatible releases if possible.
+
 ### Features
 
 - Dependency-tracking wrappers for all locking primitives

benches/mutex.rs

@@ -7,7 +7,7 @@ use criterion::BenchmarkId;
 use criterion::Criterion;
 use criterion::Throughput;
 use rand::prelude::*;
-use tracing_mutex::stdsync::TracingMutex;
+use tracing_mutex::stdsync::tracing::Mutex as TracingMutex;
 
 const SAMPLE_SIZES: [usize; 5] = [10, 30, 100, 300, 1000];
 

bors.toml

@@ -1,4 +1,5 @@
 status = [
+    'Rust project (1.63)',
     'Rust project (stable)',
     'Rust project (beta)',
     'Documentation build',

src/lib.rs

@@ -41,7 +41,7 @@
 //!
 //! These operations have been reasonably optimized, but the performance penalty may yet be too much
 //! for production use. In those cases, it may be beneficial to instead use debug-only versions
-//! (such as [`stdsync::DebugMutex`]) which evaluate to a tracing mutex when debug assertions are
+//! (such as [`stdsync::Mutex`]) which evaluate to a tracing mutex when debug assertions are
 //! enabled, and to the underlying mutex when they're not.
 //!
 //! [paper]: https://whileydave.com/publications/pk07_jea/
@@ -53,10 +53,10 @@ use std::marker::PhantomData;
 use std::mem::MaybeUninit;
 use std::ops::Deref;
 use std::ops::DerefMut;
-use std::ptr;
 use std::sync::atomic::AtomicUsize;
 use std::sync::atomic::Ordering;
 use std::sync::Mutex;
+use std::sync::MutexGuard;
 use std::sync::Once;
 use std::sync::PoisonError;
 
@@ -134,7 +134,10 @@ impl MutexId {
     /// This method panics if the new dependency would introduce a cycle.
     pub fn get_borrowed(&self) -> BorrowedMutex {
         self.mark_held();
-        BorrowedMutex(self)
+        BorrowedMutex {
+            id: self,
+            _not_send: PhantomData,
+        }
     }
 
     /// Mark this lock as held for the purposes of dependency tracking.
@@ -267,15 +270,27 @@ impl Drop for LazyMutexId {
             // We have a valid mutex ID and need to drop it
 
             // Safety: we know that this pointer is valid because the initializer has successfully run.
-            let mutex_id = unsafe { ptr::read((*self.inner.get()).as_ptr()) };
-
-            drop(mutex_id);
+            unsafe { (*self.inner.get()).assume_init_drop() };
         }
     }
 }
 
+/// Borrowed mutex ID
+///
+/// This type should be used as part of a mutex guard wrapper. It can be acquired through
+/// [`MutexId::get_borrowed`] and will automatically mark the mutex as not borrowed when it is
+/// dropped.
+///
+/// This type intentionally is [`!Send`](std::marker::Send) because the ownership tracking is based
+/// on a thread-local stack which doesn't work if a guard gets released in a different thread from
+/// where they're acquired.
 #[derive(Debug)]
-struct BorrowedMutex<'a>(&'a MutexId);
+struct BorrowedMutex<'a> {
+    /// Reference to the mutex we're borrowing from
+    id: &'a MutexId,
+    /// This value serves no purpose but to make the type [`!Send`](std::marker::Send)
+    _not_send: PhantomData<MutexGuard<'static, ()>>,
+}
 
 /// Drop a lock held by the current thread.
 ///
@@ -286,7 +301,7 @@ struct BorrowedMutex<'a>(&'a MutexId);
 impl<'a> Drop for BorrowedMutex<'a> {
     fn drop(&mut self) {
         // Safety: the only way to get a BorrowedMutex is by locking the mutex.
-        unsafe { self.0.mark_released() };
+        unsafe { self.id.mark_released() };
     }
 }
 

src/parkinglot.rs

@@ -1,19 +1,20 @@
 //! Wrapper types and type aliases for tracing [`parking_lot`] mutexes.
 //!
 //! This module provides type aliases that use the [`lockapi`][crate::lockapi] module to provide
-//! tracing variants of the `parking_lot` primitives. Each of the `TracingX` type aliases wraps an
-//! `X` in the `parkint_lot` api with dependency tracking, and a `DebugX` will refer to a `TracingX`
-//! when  `debug_assertions` are enabled and to `X` when they're not. This can be used to aid
-//! debugging in development while enjoying maximum performance in production.
+//! tracing variants of the `parking_lot` primitives. The [`tracing`] module contains type aliases
+//! that use dependency tracking, while the main `parking_lot` primitives are reexported as [`raw`].
+//!
+//! This main module imports from [`tracing`] when `debug_assertions` are enabled, and from [`raw`]
+//! when they're not. Note that primitives for which no tracing wrapper exists are not imported into
+//! the main module.
 //!
 //! # Usage
 //!
 //! ```
 //! # use std::sync::Arc;
 //! # use std::thread;
-//! # use lock_api::Mutex;
-//! # use tracing_mutex::parkinglot::TracingMutex;
-//! let mutex = Arc::new(TracingMutex::new(0));
+//! use tracing_mutex::parkinglot::Mutex;
+//! let mutex = Arc::new(Mutex::new(0));
 //!
 //! let handles: Vec<_> = (0..10).map(|_| {
 //!    let mutex = Arc::clone(&mutex);
@@ -37,141 +38,89 @@
 //! In addition, the mutex guards returned by the tracing wrappers are `!Send`, regardless of
 //! whether `parking_lot` is configured to have `Send` mutex guards. This is a limitation of the
 //! current bookkeeping system.
-use parking_lot::Once;
-use parking_lot::OnceState;
 
-use crate::lockapi::TracingWrapper;
-use crate::LazyMutexId;
+pub use parking_lot as raw;
 
-macro_rules! debug_variant {
-    ($debug_name:ident, $tracing_name:ident, $normal_name:ty) => {
-        type $tracing_name = TracingWrapper<$normal_name>;
+#[cfg(debug_assertions)]
+pub use tracing::{
+    FairMutex, FairMutexGuard, MappedFairMutexGuard, MappedMutexGuard, MappedReentrantMutexGuard,
+    MappedRwLockReadGuard, MappedRwLockWriteGuard, Mutex, MutexGuard, Once, OnceState,
+    ReentrantMutex, ReentrantMutexGuard, RwLock, RwLockReadGuard, RwLockUpgradableReadGuard,
+    RwLockWriteGuard,
+};
 
-        #[cfg(debug_assertions)]
-        type $debug_name = TracingWrapper<$normal_name>;
-        #[cfg(not(debug_assertions))]
-        type $debug_name = $normal_name;
-    };
-}
+#[cfg(not(debug_assertions))]
+pub use parking_lot::{
+    FairMutex, FairMutexGuard, MappedFairMutexGuard, MappedMutexGuard, MappedReentrantMutexGuard,
+    MappedRwLockReadGuard, MappedRwLockWriteGuard, Mutex, MutexGuard, Once, OnceState,
+    ReentrantMutex, ReentrantMutexGuard, RwLock, RwLockReadGuard, RwLockUpgradableReadGuard,
+    RwLockWriteGuard,
+};
 
-debug_variant!(
-    DebugRawFairMutex,
-    TracingRawFairMutex,
-    parking_lot::RawFairMutex
-);
-debug_variant!(DebugRawMutex, TracingRawMutex, parking_lot::RawMutex);
-debug_variant!(DebugRawRwLock, TracingRawRwLock, parking_lot::RawRwLock);
+/// Dependency tracing wrappers for [`parking_lot`].
+pub mod tracing {
+    pub use parking_lot::OnceState;
 
-/// Dependency tracking fair mutex. See: [`parking_lot::FairMutex`].
-pub type TracingFairMutex<T> = lock_api::Mutex<TracingRawFairMutex, T>;
-/// Mutex guard for [`TracingFairMutex`].
-pub type TracingFairMutexGuard<'a, T> = lock_api::MutexGuard<'a, TracingRawFairMutex, T>;
-/// RAII guard for `TracingFairMutexGuard::map`.
-pub type TracingMappedFairMutexGuard<'a, T> =
-    lock_api::MappedMutexGuard<'a, TracingRawFairMutex, T>;
-/// Debug-only dependency tracking fair mutex.
-///
-/// If debug assertions are enabled this resolves to [`TracingFairMutex`] and to
-/// [`parking_lot::FairMutex`] otherwise.
-pub type DebugFairMutex<T> = lock_api::Mutex<DebugRawFairMutex, T>;
-/// Mutex guard for [`DebugFairMutex`].
-pub type DebugFairMutexGuard<'a, T> = lock_api::MutexGuard<'a, DebugRawFairMutex, T>;
-/// RAII guard for `DebugFairMutexGuard::map`.
-pub type DebugMappedFairMutexGuard<'a, T> = lock_api::MappedMutexGuard<'a, DebugRawFairMutex, T>;
+    use crate::lockapi::TracingWrapper;
+    use crate::LazyMutexId;
 
-/// Dependency tracking mutex. See: [`parking_lot::Mutex`].
-pub type TracingMutex<T> = lock_api::Mutex<TracingRawMutex, T>;
-/// Mutex guard for [`TracingMutex`].
-pub type TracingMutexGuard<'a, T> = lock_api::MutexGuard<'a, TracingRawMutex, T>;
-/// RAII guard for `TracingMutexGuard::map`.
-pub type TracingMappedMutexGuard<'a, T> = lock_api::MappedMutexGuard<'a, TracingRawMutex, T>;
-/// Debug-only dependency tracking mutex.
-///
-/// If debug assertions are enabled this resolves to [`TracingMutex`] and to [`parking_lot::Mutex`]
-/// otherwise.
-pub type DebugMutex<T> = lock_api::Mutex<DebugRawMutex, T>;
-/// Mutex guard for [`DebugMutex`].
-pub type DebugMutexGuard<'a, T> = lock_api::MutexGuard<'a, DebugRawMutex, T>;
-/// RAII guard for `TracingMutexGuard::map`.
-pub type DebugMappedMutexGuard<'a, T> = lock_api::MappedMutexGuard<'a, DebugRawMutex, T>;
+    type RawFairMutex = TracingWrapper<parking_lot::RawFairMutex>;
+    type RawMutex = TracingWrapper<parking_lot::RawMutex>;
+    type RawRwLock = TracingWrapper<parking_lot::RawRwLock>;
 
-/// Dependency tracking reentrant mutex. See: [`parking_lot::ReentrantMutex`].
-///
-/// **Note:** due to the way dependencies are tracked, this mutex can only be acquired directly
-/// after itself. Acquiring any other mutex in between introduces a dependency cycle, and will
-/// therefore be rejected.
-pub type TracingReentrantMutex<T> =
-    lock_api::ReentrantMutex<TracingWrapper<parking_lot::RawMutex>, parking_lot::RawThreadId, T>;
-/// Mutex guard for [`TracingReentrantMutex`].
-pub type TracingReentrantMutexGuard<'a, T> = lock_api::ReentrantMutexGuard<
-    'a,
-    TracingWrapper<parking_lot::RawMutex>,
-    parking_lot::RawThreadId,
-    T,
->;
-/// RAII guard for `TracingReentrantMutexGuard::map`.
-pub type TracingMappedReentrantMutexGuard<'a, T> =
-    lock_api::MappedReentrantMutexGuard<'a, TracingRawMutex, parking_lot::RawThreadId, T>;
+    /// Dependency tracking fair mutex. See: [`parking_lot::FairMutex`].
+    pub type FairMutex<T> = lock_api::Mutex<RawFairMutex, T>;
+    /// Mutex guard for [`FairMutex`].
+    pub type FairMutexGuard<'a, T> = lock_api::MutexGuard<'a, RawFairMutex, T>;
+    /// RAII guard for [`FairMutexGuard::map`].
+    pub type MappedFairMutexGuard<'a, T> = lock_api::MappedMutexGuard<'a, RawFairMutex, T>;
 
-/// Debug-only dependency tracking reentrant mutex.
-///
-/// If debug assertions are enabled this resolves to [`TracingReentrantMutex`] and to
-/// [`parking_lot::ReentrantMutex`] otherwise.
-pub type DebugReentrantMutex<T> =
-    lock_api::ReentrantMutex<DebugRawMutex, parking_lot::RawThreadId, T>;
-/// Mutex guard for [`DebugReentrantMutex`].
-pub type DebugReentrantMutexGuard<'a, T> =
-    lock_api::ReentrantMutexGuard<'a, DebugRawMutex, parking_lot::RawThreadId, T>;
-/// RAII guard for `DebugReentrantMutexGuard::map`.
-pub type DebugMappedReentrantMutexGuard<'a, T> =
-    lock_api::MappedReentrantMutexGuard<'a, DebugRawMutex, parking_lot::RawThreadId, T>;
+    /// Dependency tracking mutex. See: [`parking_lot::Mutex`].
+    pub type Mutex<T> = lock_api::Mutex<RawMutex, T>;
+    /// Mutex guard for [`Mutex`].
+    pub type MutexGuard<'a, T> = lock_api::MutexGuard<'a, RawMutex, T>;
+    /// RAII guard for [`MutexGuard::map`].
+    pub type MappedMutexGuard<'a, T> = lock_api::MappedMutexGuard<'a, RawMutex, T>;
 
-/// Dependency tracking RwLock. See: [`parking_lot::RwLock`].
-pub type TracingRwLock<T> = lock_api::RwLock<TracingRawRwLock, T>;
-/// Read guard for [`TracingRwLock`].
-pub type TracingRwLockReadGuard<'a, T> = lock_api::RwLockReadGuard<'a, TracingRawRwLock, T>;
-/// Upgradable Read guard for [`TracingRwLock`].
-pub type TracingRwLockUpgradableReadGuard<'a, T> =
-    lock_api::RwLockUpgradableReadGuard<'a, TracingRawRwLock, T>;
-/// Write guard for [`TracingRwLock`].
-pub type TracingRwLockWriteGuard<'a, T> = lock_api::RwLockWriteGuard<'a, TracingRawRwLock, T>;
-/// RAII guard for `TracingRwLockReadGuard::map`.
-pub type TracingMappedRwLockReadGuard<'a, T> =
-    lock_api::MappedRwLockReadGuard<'a, TracingRawRwLock, T>;
-/// RAII guard for `TracingRwLockWriteGuard::map`.
-pub type TracingMappedRwLockWriteGuard<'a, T> =
-    lock_api::MappedRwLockWriteGuard<'a, TracingRawRwLock, T>;
+    /// Dependency tracking reentrant mutex. See: [`parking_lot::ReentrantMutex`].
+    ///
+    /// **Note:** due to the way dependencies are tracked, this mutex can only be acquired directly
+    /// after itself. Acquiring any other mutex in between introduces a dependency cycle, and will
+    /// therefore be rejected.
+    pub type ReentrantMutex<T> = lock_api::ReentrantMutex<RawMutex, parking_lot::RawThreadId, T>;
+    /// Mutex guard for [`ReentrantMutex`].
+    pub type ReentrantMutexGuard<'a, T> =
+        lock_api::ReentrantMutexGuard<'a, RawMutex, parking_lot::RawThreadId, T>;
+    /// RAII guard for `ReentrantMutexGuard::map`.
+    pub type MappedReentrantMutexGuard<'a, T> =
+        lock_api::MappedReentrantMutexGuard<'a, RawMutex, parking_lot::RawThreadId, T>;
 
-/// Debug-only dependency tracking RwLock.
-///
-/// If debug assertions are enabled this resolved to [`TracingRwLock`] and to
-/// [`parking_lot::RwLock`] otherwise.
-pub type DebugRwLock<T> = lock_api::RwLock<DebugRawRwLock, T>;
-/// Read guard for [`TracingRwLock`].
-pub type DebugRwLockReadGuard<'a, T> = lock_api::RwLockReadGuard<'a, DebugRawRwLock, T>;
-/// Upgradable Read guard for [`TracingRwLock`].
-pub type DebugRwLockUpgradableReadGuard<'a, T> =
-    lock_api::RwLockUpgradableReadGuard<'a, DebugRawRwLock, T>;
-/// Write guard for [`TracingRwLock`].
-pub type DebugRwLockWriteGuard<'a, T> = lock_api::RwLockWriteGuard<'a, DebugRawRwLock, T>;
-/// RAII guard for `DebugRwLockReadGuard::map`.
-pub type DebugMappedRwLockReadGuard<'a, T> = lock_api::MappedRwLockReadGuard<'a, DebugRawRwLock, T>;
-/// RAII guard for `DebugRwLockWriteGuard::map`.
-pub type DebugMappedRwLockWriteGuard<'a, T> =
-    lock_api::MappedRwLockWriteGuard<'a, DebugRawRwLock, T>;
+    /// Dependency tracking RwLock. See: [`parking_lot::RwLock`].
+    pub type RwLock<T> = lock_api::RwLock<RawRwLock, T>;
+    /// Read guard for [`RwLock`].
+    pub type RwLockReadGuard<'a, T> = lock_api::RwLockReadGuard<'a, RawRwLock, T>;
+    /// Upgradable Read guard for [`RwLock`].
+    pub type RwLockUpgradableReadGuard<'a, T> =
+        lock_api::RwLockUpgradableReadGuard<'a, RawRwLock, T>;
+    /// Write guard for [`RwLock`].
+    pub type RwLockWriteGuard<'a, T> = lock_api::RwLockWriteGuard<'a, RawRwLock, T>;
+    /// RAII guard for `RwLockReadGuard::map`.
+    pub type MappedRwLockReadGuard<'a, T> = lock_api::MappedRwLockReadGuard<'a, RawRwLock, T>;
+    /// RAII guard for `RwLockWriteGuard::map`.
+    pub type MappedRwLockWriteGuard<'a, T> = lock_api::MappedRwLockWriteGuard<'a, RawRwLock, T>;
 
-/// A dependency-tracking wrapper for [`parking_lot::Once`].
-#[derive(Debug, Default)]
-pub struct TracingOnce {
-    inner: Once,
+    /// A dependency-tracking wrapper for [`parking_lot::Once`].
+    #[derive(Debug, Default)]
+    pub struct Once {
+        inner: parking_lot::Once,
         id: LazyMutexId,
-}
+    }
 
-impl TracingOnce {
-    /// Create a new `TracingOnce` value.
+    impl Once {
+        /// Create a new `Once` value.
         pub const fn new() -> Self {
             Self {
-            inner: Once::new(),
+                inner: parking_lot::Once::new(),
                 id: LazyMutexId::new(),
             }
         }
@@ -181,8 +130,7 @@ impl TracingOnce {
             self.inner.state()
         }
 
-    ///
-    /// This call is considered as "locking this `TracingOnce`" and it participates in dependency
+        /// This call is considered as "locking this `Once`" and it participates in dependency
         /// tracking as such.
         ///
         /// # Panics
@@ -196,32 +144,24 @@ impl TracingOnce {
 
         /// Performs the given initialization routine once and only once.
         ///
-    /// This method is identical to [`TracingOnce::call_once`] except it ignores poisoning.
+        /// This method is identical to [`Once::call_once`] except it ignores poisoning.
         pub fn call_once_force(&self, f: impl FnOnce(OnceState)) {
             let _borrow = self.id.get_borrowed();
             self.inner.call_once_force(f);
         }
+    }
 }
 
-/// Debug-only `Once`.
-///
-/// If debug assertions are enabled this resolves to [`TracingOnce`] and to [`parking_lot::Once`]
-/// otherwise.
-#[cfg(debug_assertions)]
-pub type DebugOnce = TracingOnce;
-#[cfg(not(debug_assertions))]
-pub type DebugOnce = Once;
-
 #[cfg(test)]
 mod tests {
     use std::sync::Arc;
     use std::thread;
 
-    use super::*;
+    use super::tracing;
 
     #[test]
     fn test_mutex_usage() {
-        let mutex = Arc::new(TracingMutex::new(()));
+        let mutex = Arc::new(tracing::Mutex::new(()));
         let local_lock = mutex.lock();
         drop(local_lock);
 
@@ -236,9 +176,9 @@ mod tests {
     #[should_panic]
     fn test_mutex_conflict() {
         let mutexes = [
-            TracingMutex::new(()),
-            TracingMutex::new(()),
-            TracingMutex::new(()),
+            tracing::Mutex::new(()),
+            tracing::Mutex::new(()),
+            tracing::Mutex::new(()),
         ];
 
         for i in 0..3 {
@@ -249,7 +189,7 @@ mod tests {
 
     #[test]
     fn test_rwlock_usage() {
-        let lock = Arc::new(TracingRwLock::new(()));
+        let lock = Arc::new(tracing::RwLock::new(()));
         let lock2 = Arc::clone(&lock);
 
         let _read_lock = lock.read();
@@ -264,19 +204,19 @@ mod tests {
 
     #[test]
     fn test_rwlock_upgradable_read_usage() {
-        let lock = TracingRwLock::new(());
+        let lock = tracing::RwLock::new(());
 
         // Should be able to acquire an upgradable read lock.
-        let upgradable_guard: TracingRwLockUpgradableReadGuard<'_, _> = lock.upgradable_read();
+        let upgradable_guard: tracing::RwLockUpgradableReadGuard<'_, _> = lock.upgradable_read();
 
         // Should be able to upgrade the guard.
-        let _write_guard: TracingRwLockWriteGuard<'_, _> =
-            TracingRwLockUpgradableReadGuard::upgrade(upgradable_guard);
+        let _write_guard: tracing::RwLockWriteGuard<'_, _> =
+            tracing::RwLockUpgradableReadGuard::upgrade(upgradable_guard);
     }
 
     #[test]
     fn test_once_usage() {
-        let once = Arc::new(TracingOnce::new());
+        let once = Arc::new(tracing::Once::new());
         let once_clone = once.clone();
 
         assert!(!once_clone.state().done());

src/stdsync.rs

@@ -1,130 +1,82 @@
 //! Tracing mutex wrappers for locks found in `std::sync`.
 //!
 //! This module provides wrappers for `std::sync` primitives with exactly the same API and
-//! functionality as their counterparts, with the exception that their acquisition order is
-//! tracked.
+//! functionality as their counterparts, with the exception that their acquisition order is tracked.
+//!
+//! Dedicated wrappers that provide the dependency tracing can be found in the [`tracing`] module.
+//! The original primitives are available from [`std::sync`], imported as [`raw`] for convenience.
+//!
+//! If debug assertions are enabled, this module imports the primitives from [`tracing`], otherwise
+//! it will import from [`raw`].
 //!
 //! ```rust
-//! # use tracing_mutex::stdsync::TracingMutex;
-//! # use tracing_mutex::stdsync::TracingRwLock;
-//! let mutex = TracingMutex::new(());
+//! # use tracing_mutex::stdsync::tracing::Mutex;
+//! # use tracing_mutex::stdsync::tracing::RwLock;
+//! let mutex = Mutex::new(());
 //! mutex.lock().unwrap();
 //!
-//! let rwlock = TracingRwLock::new(());
+//! let rwlock = RwLock::new(());
 //! rwlock.read().unwrap();
 //! ```
-use std::fmt;
-use std::ops::Deref;
-use std::ops::DerefMut;
-use std::sync::Condvar;
-use std::sync::LockResult;
-use std::sync::Mutex;
-use std::sync::MutexGuard;
-use std::sync::Once;
-use std::sync::OnceState;
-use std::sync::PoisonError;
-use std::sync::RwLock;
-use std::sync::RwLockReadGuard;
-use std::sync::RwLockWriteGuard;
-use std::sync::TryLockError;
-use std::sync::TryLockResult;
-use std::sync::WaitTimeoutResult;
-use std::time::Duration;
+pub use std::sync as raw;
 
-use crate::BorrowedMutex;
-use crate::LazyMutexId;
-use crate::MutexId;
-
-/// Debug-only tracing `Mutex`.
-///
-/// Type alias that resolves to [`TracingMutex`] when debug assertions are enabled and to
-/// [`std::sync::Mutex`] when they're not. Use this if you want to have the benefits of cycle
-/// detection in development but do not want to pay the performance penalty in release.
-#[cfg(debug_assertions)]
-pub type DebugMutex<T> = TracingMutex<T>;
 #[cfg(not(debug_assertions))]
-pub type DebugMutex<T> = Mutex<T>;
+pub use std::sync::{Condvar, Mutex, MutexGuard, Once, RwLock, RwLockReadGuard, RwLockWriteGuard};
 
-/// Mutex guard for [`DebugMutex`].
 #[cfg(debug_assertions)]
-pub type DebugMutexGuard<'a, T> = TracingMutexGuard<'a, T>;
-#[cfg(not(debug_assertions))]
-pub type DebugMutexGuard<'a, T> = MutexGuard<'a, T>;
+pub use tracing::{Condvar, Mutex, MutexGuard, Once, RwLock, RwLockReadGuard, RwLockWriteGuard};
 
-/// Debug-only `Condvar`
-///
-/// Type alias that accepts the mutex guard emitted from [`DebugMutex`].
-#[cfg(debug_assertions)]
-pub type DebugCondvar = TracingCondvar;
-#[cfg(not(debug_assertions))]
-pub type DebugCondvar = Condvar;
+/// Dependency tracing versions of [`std::sync`].
+pub mod tracing {
+    use std::fmt;
+    use std::ops::Deref;
+    use std::ops::DerefMut;
+    use std::sync;
+    use std::sync::LockResult;
+    use std::sync::OnceState;
+    use std::sync::PoisonError;
+    use std::sync::TryLockError;
+    use std::sync::TryLockResult;
+    use std::sync::WaitTimeoutResult;
+    use std::time::Duration;
 
-/// Debug-only tracing `RwLock`.
-///
-/// Type alias that resolves to [`TracingRwLock`] when debug assertions are enabled and to
-/// [`std::sync::RwLock`] when they're not. Use this if you want to have the benefits of cycle
-/// detection in development but do not want to pay the performance penalty in release.
-#[cfg(debug_assertions)]
-pub type DebugRwLock<T> = TracingRwLock<T>;
-#[cfg(not(debug_assertions))]
-pub type DebugRwLock<T> = RwLock<T>;
+    use crate::BorrowedMutex;
+    use crate::LazyMutexId;
 
-/// Read guard for [`DebugRwLock`].
-#[cfg(debug_assertions)]
-pub type DebugReadGuard<'a, T> = TracingReadGuard<'a, T>;
-#[cfg(not(debug_assertions))]
-pub type DebugReadGuard<'a, T> = RwLockReadGuard<'a, T>;
+    /// Wrapper for [`std::sync::Mutex`].
+    ///
+    /// Refer to the [crate-level][`crate`] documentation for the differences between this struct and
+    /// the one it wraps.
+    #[derive(Debug, Default)]
+    pub struct Mutex<T> {
+        inner: sync::Mutex<T>,
+        id: LazyMutexId,
+    }
 
-/// Write guard for [`DebugRwLock`].
-#[cfg(debug_assertions)]
-pub type DebugWriteGuard<'a, T> = TracingWriteGuard<'a, T>;
-#[cfg(not(debug_assertions))]
-pub type DebugWriteGuard<'a, T> = RwLockWriteGuard<'a, T>;
-
-/// Debug-only tracing `Once`.
-///
-/// Type alias that resolves to [`TracingOnce`] when debug assertions are enabled and to
-/// [`std::sync::Once`] when they're not. Use this if you want to have the benefits of cycle
-/// detection in development but do not want to pay the performance penalty in release.
-#[cfg(debug_assertions)]
-pub type DebugOnce = TracingOnce;
-#[cfg(not(debug_assertions))]
-pub type DebugOnce = Once;
-
-/// Wrapper for [`std::sync::Mutex`].
-///
-/// Refer to the [crate-level][`crate`] documentaiton for the differences between this struct and
-/// the one it wraps.
-#[derive(Debug, Default)]
-pub struct TracingMutex<T> {
-    inner: Mutex<T>,
-    id: MutexId,
-}
-
-/// Wrapper for [`std::sync::MutexGuard`].
-///
-/// Refer to the [crate-level][`crate`] documentaiton for the differences between this struct and
-/// the one it wraps.
-#[derive(Debug)]
-pub struct TracingMutexGuard<'a, T> {
-    inner: MutexGuard<'a, T>,
+    /// Wrapper for [`std::sync::MutexGuard`].
+    ///
+    /// Refer to the [crate-level][`crate`] documentation for the differences between this struct and
+    /// the one it wraps.
+    #[derive(Debug)]
+    pub struct MutexGuard<'a, T> {
+        inner: sync::MutexGuard<'a, T>,
         _mutex: BorrowedMutex<'a>,
-}
+    }
 
-fn map_lockresult<T, I, F>(result: LockResult<I>, mapper: F) -> LockResult<T>
-where
+    fn map_lockresult<T, I, F>(result: LockResult<I>, mapper: F) -> LockResult<T>
+    where
         F: FnOnce(I) -> T,
-{
+    {
         match result {
             Ok(inner) => Ok(mapper(inner)),
             Err(poisoned) => Err(PoisonError::new(mapper(poisoned.into_inner()))),
         }
-}
+    }
 
-fn map_trylockresult<T, I, F>(result: TryLockResult<I>, mapper: F) -> TryLockResult<T>
-where
+    fn map_trylockresult<T, I, F>(result: TryLockResult<I>, mapper: F) -> TryLockResult<T>
+    where
         F: FnOnce(I) -> T,
-{
+    {
         match result {
             Ok(inner) => Ok(mapper(inner)),
             Err(TryLockError::WouldBlock) => Err(TryLockError::WouldBlock),
@@ -132,14 +84,14 @@ where
                 Err(PoisonError::new(mapper(poisoned.into_inner())).into())
             }
         }
-}
+    }
 
-impl<T> TracingMutex<T> {
+    impl<T> Mutex<T> {
         /// Create a new tracing mutex with the provided value.
-    pub fn new(t: T) -> Self {
+        pub const fn new(t: T) -> Self {
             Self {
-            inner: Mutex::new(t),
-            id: MutexId::new(),
+                inner: sync::Mutex::new(t),
+                id: LazyMutexId::new(),
             }
         }
 
@@ -150,11 +102,11 @@ impl<T> TracingMutex<T> {
         /// This method participates in lock dependency tracking. If acquiring this lock introduces a
         /// dependency cycle, this method will panic.
         #[track_caller]
-    pub fn lock(&self) -> LockResult<TracingMutexGuard<T>> {
+        pub fn lock(&self) -> LockResult<MutexGuard<T>> {
             let mutex = self.id.get_borrowed();
             let result = self.inner.lock();
 
-        let mapper = |guard| TracingMutexGuard {
+            let mapper = |guard| MutexGuard {
                 _mutex: mutex,
                 inner: guard,
             };
@@ -169,11 +121,11 @@ impl<T> TracingMutex<T> {
         /// This method participates in lock dependency tracking. If acquiring this lock introduces a
         /// dependency cycle, this method will panic.
         #[track_caller]
-    pub fn try_lock(&self) -> TryLockResult<TracingMutexGuard<T>> {
+        pub fn try_lock(&self) -> TryLockResult<MutexGuard<T>> {
             let mutex = self.id.get_borrowed();
             let result = self.inner.try_lock();
 
-        let mapper = |guard| TracingMutexGuard {
+            let mapper = |guard| MutexGuard {
                 _mutex: mutex,
                 inner: guard,
             };
@@ -197,137 +149,132 @@ impl<T> TracingMutex<T> {
         pub fn into_inner(self) -> LockResult<T> {
             self.inner.into_inner()
         }
-}
+    }
 
-impl<T> From<T> for TracingMutex<T> {
+    impl<T> From<T> for Mutex<T> {
         fn from(t: T) -> Self {
             Self::new(t)
         }
-}
+    }
 
-impl<'a, T> Deref for TracingMutexGuard<'a, T> {
+    impl<'a, T> Deref for MutexGuard<'a, T> {
         type Target = T;
 
         fn deref(&self) -> &Self::Target {
             &self.inner
         }
-}
+    }
 
-impl<'a, T> DerefMut for TracingMutexGuard<'a, T> {
+    impl<'a, T> DerefMut for MutexGuard<'a, T> {
         fn deref_mut(&mut self) -> &mut Self::Target {
             &mut self.inner
         }
-}
+    }
 
-impl<'a, T: fmt::Display> fmt::Display for TracingMutexGuard<'a, T> {
+    impl<'a, T: fmt::Display> fmt::Display for MutexGuard<'a, T> {
         fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
             self.inner.fmt(f)
         }
-}
+    }
 
-/// Wrapper around [`std::sync::Condvar`].
-///
-/// Allows `TracingMutexGuard` to be used with a `Condvar`. Unlike other structs in this module,
-/// this wrapper does not add any additional dependency tracking or other overhead on top of the
-/// primitive it wraps. All dependency tracking happens through the mutexes itself.
-///
-/// # Panics
-///
-/// This struct does not add any panics over the base implementation of `Condvar`, but panics due to
-/// dependency tracking may poison associated mutexes.
-///
-/// # Examples
-///
-/// ```
-/// use std::sync::Arc;
-/// use std::thread;
-///
-/// use tracing_mutex::stdsync::{TracingCondvar, TracingMutex};
-///
-/// let pair = Arc::new((TracingMutex::new(false), TracingCondvar::new()));
-/// let pair2 = Arc::clone(&pair);
-///
-/// // Spawn a thread that will unlock the condvar
-/// thread::spawn(move || {
-///     let (lock, condvar) = &*pair2;
-///     *lock.lock().unwrap() = true;
-///     condvar.notify_one();
-/// });
-///
-/// // Wait until the thread unlocks the condvar
-/// let (lock, condvar) = &*pair;
-/// let guard = lock.lock().unwrap();
-/// let guard = condvar.wait_while(guard, |started| !*started).unwrap();
-///
-/// // Guard should read true now
-/// assert!(*guard);
-/// ```
-#[derive(Debug, Default)]
-pub struct TracingCondvar(Condvar);
+    /// Wrapper around [`std::sync::Condvar`].
+    ///
+    /// Allows `TracingMutexGuard` to be used with a `Condvar`. Unlike other structs in this module,
+    /// this wrapper does not add any additional dependency tracking or other overhead on top of the
+    /// primitive it wraps. All dependency tracking happens through the mutexes itself.
+    ///
+    /// # Panics
+    ///
+    /// This struct does not add any panics over the base implementation of `Condvar`, but panics due to
+    /// dependency tracking may poison associated mutexes.
+    ///
+    /// # Examples
+    ///
+    /// ```
+    /// use std::sync::Arc;
+    /// use std::thread;
+    ///
+    /// use tracing_mutex::stdsync::tracing::{Condvar, Mutex};
+    ///
+    /// let pair = Arc::new((Mutex::new(false), Condvar::new()));
+    /// let pair2 = Arc::clone(&pair);
+    ///
+    /// // Spawn a thread that will unlock the condvar
+    /// thread::spawn(move || {
+    ///     let (lock, condvar) = &*pair2;
+    ///     *lock.lock().unwrap() = true;
+    ///     condvar.notify_one();
+    /// });
+    ///
+    /// // Wait until the thread unlocks the condvar
+    /// let (lock, condvar) = &*pair;
+    /// let guard = lock.lock().unwrap();
+    /// let guard = condvar.wait_while(guard, |started| !*started).unwrap();
+    ///
+    /// // Guard should read true now
+    /// assert!(*guard);
+    /// ```
+    #[derive(Debug, Default)]
+    pub struct Condvar(sync::Condvar);
 
-impl TracingCondvar {
+    impl Condvar {
         /// Creates a new condition variable which is ready to be waited on and notified.
-    pub fn new() -> Self {
-        Default::default()
+        pub const fn new() -> Self {
+            Self(sync::Condvar::new())
         }
 
         /// Wrapper for [`std::sync::Condvar::wait`].
-    pub fn wait<'a, T>(
-        &self,
-        guard: TracingMutexGuard<'a, T>,
-    ) -> LockResult<TracingMutexGuard<'a, T>> {
-        let TracingMutexGuard { _mutex, inner } = guard;
+        pub fn wait<'a, T>(&self, guard: MutexGuard<'a, T>) -> LockResult<MutexGuard<'a, T>> {
+            let MutexGuard { _mutex, inner } = guard;
 
-        map_lockresult(self.0.wait(inner), |inner| TracingMutexGuard {
-            _mutex,
-            inner,
-        })
+            map_lockresult(self.0.wait(inner), |inner| MutexGuard { _mutex, inner })
         }
 
         /// Wrapper for [`std::sync::Condvar::wait_while`].
         pub fn wait_while<'a, T, F>(
             &self,
-        guard: TracingMutexGuard<'a, T>,
+            guard: MutexGuard<'a, T>,
             condition: F,
-    ) -> LockResult<TracingMutexGuard<'a, T>>
+        ) -> LockResult<MutexGuard<'a, T>>
         where
             F: FnMut(&mut T) -> bool,
         {
-        let TracingMutexGuard { _mutex, inner } = guard;
+            let MutexGuard { _mutex, inner } = guard;
 
-        map_lockresult(self.0.wait_while(inner, condition), |inner| {
-            TracingMutexGuard { _mutex, inner }
+            map_lockresult(self.0.wait_while(inner, condition), |inner| MutexGuard {
+                _mutex,
+                inner,
             })
         }
 
         /// Wrapper for [`std::sync::Condvar::wait_timeout`].
         pub fn wait_timeout<'a, T>(
             &self,
-        guard: TracingMutexGuard<'a, T>,
+            guard: MutexGuard<'a, T>,
             dur: Duration,
-    ) -> LockResult<(TracingMutexGuard<'a, T>, WaitTimeoutResult)> {
-        let TracingMutexGuard { _mutex, inner } = guard;
+        ) -> LockResult<(MutexGuard<'a, T>, WaitTimeoutResult)> {
+            let MutexGuard { _mutex, inner } = guard;
 
             map_lockresult(self.0.wait_timeout(inner, dur), |(inner, result)| {
-            (TracingMutexGuard { _mutex, inner }, result)
+                (MutexGuard { _mutex, inner }, result)
             })
         }
 
         /// Wrapper for [`std::sync::Condvar::wait_timeout_while`].
         pub fn wait_timeout_while<'a, T, F>(
             &self,
-        guard: TracingMutexGuard<'a, T>,
+            guard: MutexGuard<'a, T>,
             dur: Duration,
             condition: F,
-    ) -> LockResult<(TracingMutexGuard<'a, T>, WaitTimeoutResult)>
+        ) -> LockResult<(MutexGuard<'a, T>, WaitTimeoutResult)>
         where
             F: FnMut(&mut T) -> bool,
         {
-        let TracingMutexGuard { _mutex, inner } = guard;
+            let MutexGuard { _mutex, inner } = guard;
 
             map_lockresult(
                 self.0.wait_timeout_while(inner, dur, condition),
-            |(inner, result)| (TracingMutexGuard { _mutex, inner }, result),
+                |(inner, result)| (MutexGuard { _mutex, inner }, result),
             )
         }
 
@@ -340,34 +287,34 @@ impl TracingCondvar {
         pub fn notify_all(&self) {
             self.0.notify_all();
         }
-}
+    }
 
-/// Wrapper for [`std::sync::RwLock`].
-#[derive(Debug, Default)]
-pub struct TracingRwLock<T> {
-    inner: RwLock<T>,
-    id: MutexId,
-}
+    /// Wrapper for [`std::sync::RwLock`].
+    #[derive(Debug, Default)]
+    pub struct RwLock<T> {
+        inner: sync::RwLock<T>,
+        id: LazyMutexId,
+    }
 
-/// Hybrid wrapper for both [`std::sync::RwLockReadGuard`] and [`std::sync::RwLockWriteGuard`].
-///
-/// Please refer to [`TracingReadGuard`] and [`TracingWriteGuard`] for usable types.
-#[derive(Debug)]
-pub struct TracingRwLockGuard<'a, L> {
+    /// Hybrid wrapper for both [`std::sync::RwLockReadGuard`] and [`std::sync::RwLockWriteGuard`].
+    ///
+    /// Please refer to [`RwLockReadGuard`] and [`RwLockWriteGuard`] for usable types.
+    #[derive(Debug)]
+    pub struct TracingRwLockGuard<'a, L> {
         inner: L,
         _mutex: BorrowedMutex<'a>,
-}
+    }
 
-/// Wrapper around [`std::sync::RwLockReadGuard`].
-pub type TracingReadGuard<'a, T> = TracingRwLockGuard<'a, RwLockReadGuard<'a, T>>;
-/// Wrapper around [`std::sync::RwLockWriteGuard`].
-pub type TracingWriteGuard<'a, T> = TracingRwLockGuard<'a, RwLockWriteGuard<'a, T>>;
+    /// Wrapper around [`std::sync::RwLockReadGuard`].
+    pub type RwLockReadGuard<'a, T> = TracingRwLockGuard<'a, sync::RwLockReadGuard<'a, T>>;
+    /// Wrapper around [`std::sync::RwLockWriteGuard`].
+    pub type RwLockWriteGuard<'a, T> = TracingRwLockGuard<'a, sync::RwLockWriteGuard<'a, T>>;
 
-impl<T> TracingRwLock<T> {
-    pub fn new(t: T) -> Self {
+    impl<T> RwLock<T> {
+        pub const fn new(t: T) -> Self {
             Self {
-            inner: RwLock::new(t),
-            id: MutexId::new(),
+                inner: sync::RwLock::new(t),
+                id: LazyMutexId::new(),
             }
         }
 
@@ -378,7 +325,7 @@ impl<T> TracingRwLock<T> {
         /// This method participates in lock dependency tracking. If acquiring this lock introduces a
         /// dependency cycle, this method will panic.
         #[track_caller]
-    pub fn read(&self) -> LockResult<TracingReadGuard<T>> {
+        pub fn read(&self) -> LockResult<RwLockReadGuard<T>> {
             let mutex = self.id.get_borrowed();
             let result = self.inner.read();
 
@@ -395,7 +342,7 @@ impl<T> TracingRwLock<T> {
         /// This method participates in lock dependency tracking. If acquiring this lock introduces a
         /// dependency cycle, this method will panic.
         #[track_caller]
-    pub fn write(&self) -> LockResult<TracingWriteGuard<T>> {
+        pub fn write(&self) -> LockResult<RwLockWriteGuard<T>> {
             let mutex = self.id.get_borrowed();
             let result = self.inner.write();
 
@@ -412,7 +359,7 @@ impl<T> TracingRwLock<T> {
         /// This method participates in lock dependency tracking. If acquiring this lock introduces a
         /// dependency cycle, this method will panic.
         #[track_caller]
-    pub fn try_read(&self) -> TryLockResult<TracingReadGuard<T>> {
+        pub fn try_read(&self) -> TryLockResult<RwLockReadGuard<T>> {
             let mutex = self.id.get_borrowed();
             let result = self.inner.try_read();
 
@@ -429,7 +376,7 @@ impl<T> TracingRwLock<T> {
         /// This method participates in lock dependency tracking. If acquiring this lock introduces a
         /// dependency cycle, this method will panic.
         #[track_caller]
-    pub fn try_write(&self) -> TryLockResult<TracingWriteGuard<T>> {
+        pub fn try_write(&self) -> TryLockResult<RwLockWriteGuard<T>> {
             let mutex = self.id.get_borrowed();
             let result = self.inner.try_write();
 
@@ -450,49 +397,49 @@ impl<T> TracingRwLock<T> {
         pub fn into_inner(self) -> LockResult<T> {
             self.inner.into_inner()
         }
-}
+    }
 
-impl<T> From<T> for TracingRwLock<T> {
+    impl<T> From<T> for RwLock<T> {
         fn from(t: T) -> Self {
             Self::new(t)
         }
-}
+    }
 
-impl<'a, L, T> Deref for TracingRwLockGuard<'a, L>
-where
+    impl<'a, L, T> Deref for TracingRwLockGuard<'a, L>
+    where
         L: Deref<Target = T>,
-{
+    {
         type Target = T;
 
         fn deref(&self) -> &Self::Target {
             self.inner.deref()
         }
-}
+    }
 
-impl<'a, T, L> DerefMut for TracingRwLockGuard<'a, L>
-where
+    impl<'a, T, L> DerefMut for TracingRwLockGuard<'a, L>
+    where
         L: Deref<Target = T> + DerefMut,
-{
+    {
         fn deref_mut(&mut self) -> &mut Self::Target {
             self.inner.deref_mut()
         }
-}
+    }
 
-/// Wrapper around [`std::sync::Once`].
-///
-/// Refer to the [crate-level][`crate`] documentaiton for the differences between this struct and
-/// the one it wraps.
-#[derive(Debug)]
-pub struct TracingOnce {
-    inner: Once,
+    /// Wrapper around [`std::sync::Once`].
+    ///
+    /// Refer to the [crate-level][`crate`] documentaiton for the differences between this struct and
+    /// the one it wraps.
+    #[derive(Debug)]
+    pub struct Once {
+        inner: sync::Once,
         mutex_id: LazyMutexId,
-}
+    }
 
-impl TracingOnce {
+    impl Once {
         /// Create a new `Once` value.
         pub const fn new() -> Self {
             Self {
-            inner: Once::new(),
+                inner: sync::Once::new(),
                 mutex_id: LazyMutexId::new(),
             }
         }
@@ -511,7 +458,7 @@ impl TracingOnce {
             self.inner.call_once(f);
         }
 
-    /// Performs the same operation as [`call_once`][TracingOnce::call_once] except it ignores
+        /// Performs the same operation as [`call_once`][Once::call_once] except it ignores
         /// poisoning.
         ///
         /// # Panics
@@ -530,10 +477,10 @@ impl TracingOnce {
         pub fn is_completed(&self) -> bool {
             self.inner.is_completed()
         }
-}
+    }
 
-#[cfg(test)]
-mod tests {
+    #[cfg(test)]
+    mod tests {
         use std::sync::Arc;
         use std::thread;
 
@@ -541,7 +488,7 @@ mod tests {
 
         #[test]
         fn test_mutex_usage() {
-        let mutex = Arc::new(TracingMutex::new(0));
+            let mutex = Arc::new(Mutex::new(0));
 
             assert_eq!(*mutex.lock().unwrap(), 0);
             *mutex.lock().unwrap() = 1;
@@ -563,7 +510,7 @@ mod tests {
 
         #[test]
         fn test_rwlock_usage() {
-        let rwlock = Arc::new(TracingRwLock::new(0));
+            let rwlock = Arc::new(RwLock::new(0));
 
             assert_eq!(*rwlock.read().unwrap(), 0);
             assert_eq!(*rwlock.write().unwrap(), 0);
@@ -590,7 +537,7 @@ mod tests {
 
         #[test]
         fn test_once_usage() {
-        let once = Arc::new(TracingOnce::new());
+            let once = Arc::new(Once::new());
             let once_clone = once.clone();
 
             assert!(!once.is_completed());
@@ -611,8 +558,8 @@ mod tests {
         #[test]
         #[should_panic(expected = "Mutex order graph should not have cycles")]
         fn test_detect_cycle() {
-        let a = TracingMutex::new(());
-        let b = TracingMutex::new(());
+            let a = Mutex::new(());
+            let b = Mutex::new(());
 
             let hold_a = a.lock().unwrap();
             let _ = b.lock();
@@ -622,4 +569,5 @@ mod tests {
             let _hold_b = b.lock().unwrap();
             let _ = a.lock();
         }
+    }
 }