Skip to content

Add arc_prefetch script. #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 13, 2020
Merged

Add arc_prefetch script. #34

merged 1 commit into from
Mar 13, 2020

Conversation

brad-lewis
Copy link
Contributor

See DLPX-68394

Here is sample output:
02/12/20 - 17:21:40 UTC

prefetch hit count : 21
prefetch miss count : 17
prefetch read count : 1
microseconds arc read latency
value range count ------------- Distribution -------------
[16, 32) 9 |@@@@@@@@@@@@@@@@@@
[32, 64) 6 |@@@@@@@@@@@@
[64, 128) 3 |@@@@@

microseconds prefetch read latency
value range count ------------- Distribution -------------
[16M, 32M) 1 |@

                            avg latency(us)

arc read latency 39
prefetch read latency 34658844

@brad-lewis brad-lewis force-pushed the arc_prefetch branch 2 times, most recently from a0da28f to ecc250d Compare February 14, 2020 22:04
@brad-lewis
Copy link
Contributor Author

I updated BCCHelper with a couple things. One was to add the BCCMapIndex and the key CUSTOM index type. The purpose of this was to allow ints to be used for the map keys in the C code with strings being substituted for the display. The second BCCPoolCompare using the approach Seb suggested of getting the spa_t pointer first and using that as an input into the main tracing script. Both of these can be used to simplify the existing scripts. Once the final form is reviewed and approved I can do the refactoring.

@sebroy sebroy merged commit 3f0e542 into delphix:master Mar 13, 2020
brad-lewis added a commit to brad-lewis/performance-diagnostics that referenced this pull request Oct 23, 2020
brad-lewis added a commit to brad-lewis/performance-diagnostics that referenced this pull request Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants