Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Perf -8%] Microsoft.Extensions.Logging.LoggingOverhead.TwoArguments_FilteredByLevel #3293

Closed
performanceautofiler bot opened this issue Nov 10, 2020 · 0 comments
Closed

[Perf -8%] Microsoft.Extensions.Logging.LoggingOverhead.TwoArguments_FilteredByLevel #3293

performanceautofiler bot opened this issue Nov 10, 2020 · 0 comments
Labels
Bimodal LLVM=false MonoAOT=false MonoInterpreter=true refs/heads/master Regression RunKind=micro_mono ubuntu 18.04 x64

Comments

@performanceautofiler
Copy link

performanceautofiler bot commented Nov 10, 2020 •
edited
Loading

Run Information

Architecture x64
OS ubuntu 18.04
Baseline 5a4483eb1c0918542298d1864b7233f962952880
Compare a79df14b3cc62ade39382a6e08d3b25871d8ebb6

Regressions in Microsoft.Extensions.Logging.LoggingOverhead

Benchmark Baseline Test Test/Base Baseline IR Compare IR IR Ratio Baseline ETL Compare ETL
TwoArguments_FilteredByLevel 614.36 ns 675.85 ns 1.10
TwoArguments 658.99 ns 720.10 ns 1.09

Related Issue on x64 Windows

[Perf -14%] Microsoft.Extensions.Logging.LoggingOverhead (3)

Related Issue on x86 Windows

[Perf -14%] Microsoft.Extensions.Logging.LoggingOverhead (3)

graph
graph
Historical Data in Reporting System

Repro

git clone https://github.com/dotnet/performance.git
python3 .\performance\scripts\benchmarks_ci.py -f netcoreapp5.0 --filter 'Microsoft.Extensions.Logging.LoggingOverhead*'

Histogram

Microsoft.Extensions.Logging.LoggingOverhead.TwoArguments_FilteredByLevel

[570.232 ; 590.832) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[590.832 ; 606.084) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[606.084 ; 620.233) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[620.233 ; 644.678) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[644.678 ; 663.716) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[663.716 ; 690.809) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[690.809 ; 717.895) | @@@@@@@@

Microsoft.Extensions.Logging.LoggingOverhead.TwoArguments

[608.032 ; 623.685) | @@@@@@@@@@@@@
[623.685 ; 639.251) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[639.251 ; 654.499) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[654.499 ; 670.479) | @@@@@@@@@@@@@@@@@@@@@@@@
[670.479 ; 687.286) | @@@@@@
[687.286 ; 702.674) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[702.674 ; 718.240) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[718.240 ; 734.762) | @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
[734.762 ; 751.217) | @@@@@@@@@@@@@@@@@@@@@@@@@@
[751.217 ; 786.791) | @@@@@@@@@

Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository
@naricc naricc closed this as completed Dec 17, 2020
DrewScoggins pushed a commit that referenced this issue Nov 8, 2023
@mrsharm
Added Improvements to the Features and Messaging to the GC Infra re: …
74c994b 
…ASP.NET Benchmarks (#3293)

* Added significant improvements to the ASP.NET Benchmarks

* Fixed up header for the analysis

* Fix for the downloadOutputOutput field

* Cosmetic improvement and an example of the new params

* Improved messaging from the analysis from

* The create suite command now should reference the entire folder rather than a single corerun.exe

* Fixed linting issues

* One more Lint hydra fix

* Fixed internal references in the markdown output

* Added example of how to use the new config

* Addressed feedback and added contextually named log files if the variables are specified

* Added the name of the file and the log file extension

* Added the analysis notebook

* Adjusted the csv files to be without spaces and modified the Linux and Generic runs

* Updated README to reflect changes

* Updated documentation

* Fixed md liniting issues

* Fixed up the notebook to work for both windows and linux

* Continued work on improving the usability of the analysis

* Added fix for certain cases where the log wasn't being downloaded

* implemented the volatility and difference comparisons

* Incorporated feedback, detection if the process timed out and notebook improvements

* Cleared output

* Updated plot for vol

* Added benchmark filtering and  fix path in the notebook

* Ordering in terms of filters and matches on ends + regexes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bimodal LLVM=false MonoAOT=false MonoInterpreter=true refs/heads/master Regression RunKind=micro_mono ubuntu 18.04 x64
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@naricc