lind-perf: add benchmarks for threei.#847
Open
stupendoussuperpowers wants to merge 8 commits intolind-perf-initfrom
Open
lind-perf: add benchmarks for threei.#847stupendoussuperpowers wants to merge 8 commits intolind-perf-initfrom
lind-perf: add benchmarks for threei.#847stupendoussuperpowers wants to merge 8 commits intolind-perf-initfrom
Conversation
…d-wasm into lind-perf-threei
…d-wasm into lind-perf-threei
…d-wasm into lind-perf-threei
…to lind-perf-threei
Contributor
End-to-End Test ReportTest PreviewUnified Test Report grate harness
Cases
wasm harnessTest ReportDeterministic TestsSummary
Test Results by Category
Fail TestsSummary
Test Results by Category
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
rennergade
reviewed
Mar 4, 2026
| arg6: u64, | ||
| arg6_cageid: u64, | ||
| ) -> i32 { | ||
| // Only enable timers for syscalls that are explicitly benchmarking related (2000-3000) |
Contributor
There was a problem hiding this comment.
does this need to be cfg'd?
Contributor
There was a problem hiding this comment.
same for everything in a 3i file
Contributor
Author
There was a problem hiding this comment.
All cfg's are handled within lind-perf, which has no-op shims when the feature is not enabled. This allows us to not have to use cfg's here.
When building without lind_perf, the get_timer!() just returns None, and will not cause overheads.
rennergade
reviewed
Mar 4, 2026
| arg6: u64, | ||
| arg6_cageid: u64, | ||
| ) -> Option<i32> { | ||
| let _timer = lind_perf::get_timer!(perf::CALL_GRATE_FUNC); |
Contributor
There was a problem hiding this comment.
why are others range gated but not this?
Contributor
Author
There was a problem hiding this comment.
_call_grate_func doesn't have access to the syscall_num that was called so we can't gate it.
besides, the rationale for gating make_syscall was to ensure we dont see measurement from other syscalls. _call_grate_func is only called for interposed syscalls, and we can easily have a setup where only the syscall being benchmarked is interposed.
rennergade
reviewed
Mar 4, 2026
Contributor
End-to-End Test ReportTest PreviewUnified Test Report grate harness
Cases
wasm harnessTest ReportDeterministic TestsSummary
Test Results by Category
Fail TestsSummary
Test Results by Category
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
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
2 participants
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.
lind-perf: threei [Based on #846]perf.rsfile for threei, and include its counters in lind-boot/perf_grate_callback_trampolineandtyped_func.call()