patina_boot: Add benchmark presentation - #184
Draft
kat-perez wants to merge 1 commit into
Draft
Conversation
kat-perez
requested review from
RobertZ2011,
dymk,
maxperga-msft and
philgweber
August 11, 2026 21:08
4 tasks
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a self-contained, versioned performance artifact for patina_boot helper microbenchmarks (presentation + raw results), and links it from the crate README so stakeholders can review the current scaling and repeatability findings tied to Issues #108 and #183.
Changes:
- Add a 10-slide standalone HTML deck summarizing
connect_all/expand_device_pathmicrobenchmark scaling results and limitations. - Add the source CSV containing three same-host runs plus computed median/min/max and range.
- Link both artifacts from
uefi/crates/patina_boot/README.md.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| uefi/crates/patina_boot/README.md | Adds a Performance section linking to the new presentation and CSV data. |
| uefi/crates/patina_boot/docs/benchmark-results-2026-08.csv | Adds the benchmark dataset (3 runs + summary columns) used by the presentation. |
| uefi/crates/patina_boot/docs/benchmark-presentation-2026-08.html | Adds a self-contained HTML presentation (CSS/JS inline) rendering the results and guidance. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+39
to
+40
| - [August 2026 microbenchmark presentation](docs/benchmark-presentation-2026-08.html) | ||
| - [Presentation benchmark data](docs/benchmark-results-2026-08.csv) |
kat-perez
marked this pull request as draft
August 11, 2026 21:19
kat-perez
force-pushed
the
issue-183-patina-boot-presentation
branch
from
August 11, 2026 21:37
813fed4 to
72bbae2
Compare
Publish a self-contained HTML deck and its source CSV for the merged connect_all and expand_device_path cycle benchmarks. Summarize three-run scaling, repeatability limits, supported claims, and the work required before enabling CI regression gates. Link the artifacts from the crate README. Refs OpenDevicePartnership#108 Refs OpenDevicePartnership#183 Assisted-by: GitHub Copilot:gpt-5.6-sol Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: b47abdb1-d791-40a7-8b97-be1c3f8b7eaf
kat-perez
force-pushed
the
issue-183-patina-boot-presentation
branch
from
August 11, 2026 21:48
72bbae2 to
5466743
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
patina_boothelper benchmarkspatina_bootREADMEKey result
Both helpers scale approximately linearly through 512 synthetic handles. However, six of eight benchmark cases showed more than 10% observed range across repeated runs, so the deck recommends hardening TSC measurement and runner isolation before enabling the planned regression gate.
Validation
cargo bench --bench connect_all --bench expand_device_path -- --noplotthree times from currentmainRefs #108
Refs #183