Skip to content
This repository was archived by the owner on Nov 8, 2026. It is now read-only.

Compositor: faster DRM hotplug (250 ms to 40 ms debounce) + dock latency breakdown - #388

Merged
techmore merged 2 commits into
devfrom
claude/hotplug-fast-debounce
Oct 21, 2026
Merged

techmore merged 2 commits into
devfrom
claude/hotplug-fast-debounce

Conversation

@techmore

Copy link
Copy Markdown
Owner

What

  • `EVENT_DEBOUNCE` 250 ms → 40 ms in `hotplug.rs` (tests now derive from the constants; burst test uses the measured 9 ms gap).
  • Adds `DOCK-LATENCY-BREAKDOWN.md`: per-stage plug-to-mapped timing from the journal across 8 plug cycles.

Why

Of ~4.5 s from partner attach to a mapped output, the compositor only owns ~0.5 s: a 250 ms debounce plus a ~245 ms synchronous DCP modeset. The DCP driver emits its connector uevents ~9 ms apart with modes already populated, so the debounce was pure latency on connect and disconnect. The +1 s follow-up reconcile is unchanged.

The other ~4 s is below the compositor (ACIO firmware AT-register timeout 2.36 s, TB enumeration, boltd authorization before the DP tunnel); see the doc.

Verified

  • `cargo test -p pantheon-compositor --features native-drm hotplug`: 7 passed; `cargo clippy` clean.
  • Not yet run on hardware: the compositor loads at next login (no restart done).

🤖 Generated with Claude Code

techmore and others added 2 commits October 21, 2026 11:40
Journal timing of eight WD22TB4 plug cycles on the J314S shows the DCP driver
sends its connector uevents ~9 ms apart with modes already populated, so the
250 ms wait was pure latency on connect and disconnect. The +1 s follow-up
reconcile still covers late-settling connectors.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@techmore
techmore merged commit 77b17df into dev Oct 21, 2026
16 checks passed
@techmore
techmore deleted the claude/hotplug-fast-debounce branch October 28, 2026 16:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant