diff --git a/pageserver/src/tenant/timeline/detach_ancestor.rs b/pageserver/src/tenant/timeline/detach_ancestor.rs
index 148a71305f..f5831a5d8a 100644
--- a/pageserver/src/tenant/timeline/detach_ancestor.rs
+++ b/pageserver/src/tenant/timeline/detach_ancestor.rs
@@ -121,18 +121,11 @@ impl Default for Options {
 ///
 /// Currently this is tracked at tenant level, but it could be moved to be on the roots
 /// of each timeline tree.
+#[derive(Default)]
 pub(crate) struct SharedState {
     inner: std::sync::Mutex<Option<(TimelineId, completion::Barrier)>>,
 }
 
-impl Default for SharedState {
-    fn default() -> Self {
-        SharedState {
-            inner: Default::default(),
-        }
-    }
-}
-
 impl SharedState {
     /// Notify an uninitialized shared state that an attempt to detach timeline ancestor continues
     /// from previous instance.