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.

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

[Perf] Windows/x64: 16 Regressions on 8/28/2024 6:34:28 PM #107302

Closed
performanceautofiler bot opened this issue Sep 3, 2024 · 6 comments
Closed

[Perf] Windows/x64: 16 Regressions on 8/28/2024 6:34:28 PM #107302

performanceautofiler bot opened this issue Sep 3, 2024 · 6 comments
Labels
arch-x64 area-System.IO.Compression os-windows runtime-coreclr specific to the CoreCLR runtime tracking-external-issue The issue is caused by external problem (e.g. OS) - nothing we can do to fix it directly
Milestone

Comments

@performanceautofiler
Copy link

Run Information

Name Value
Architecture x64
OS Windows 10.0.22621
Queue TigerWindows
Baseline 7ff684af49cdf008bebbe10efcc10a2649795d65
Compare 063ae05c63307becef004eefbc8704ccb03c7f04
Diff Diff
Configs CompilationMode:tiered, RunKind:micro

Regressions in System.IO.Compression.Brotli

Benchmark Baseline Test Test/Base Test Quality Edge Detector Baseline IR Compare IR IR Ratio
514.85 μs 574.60 μs 1.12 0.00 False
170.38 μs 185.91 μs 1.09 0.00 False
158.93 μs 174.27 μs 1.10 0.00 False
347.69 μs 509.89 μs 1.47 0.01 True
158.30 μs 173.88 μs 1.10 0.00 False
429.39 μs 594.83 μs 1.39 0.01 False
600.56 μs 677.33 μs 1.13 0.01 False
606.75 μs 678.74 μs 1.12 0.00 False
516.26 μs 574.34 μs 1.11 0.00 False
370.87 μs 544.47 μs 1.47 0.01 False
399.41 μs 554.34 μs 1.39 0.01 True
574.80 μs 632.68 μs 1.10 0.00 False
662.45 μs 733.76 μs 1.11 0.00 False
371.45 μs 544.75 μs 1.47 0.01 False
343.18 μs 509.99 μs 1.49 0.01 True

graph
graph
graph
graph
graph
graph
graph
graph
graph
graph
graph
graph
graph
graph
graph
Test Report

Repro

General Docs link: https://github.com/dotnet/performance/blob/main/docs/benchmarking-workflow-dotnet-runtime.md

git clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net8.0 --filter 'System.IO.Compression.Brotli*'

System.IO.Compression.Brotli.Decompress_WithoutState(level: Optimal, file: "alice29.txt")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress(level: Fastest, file: "sum")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithState(level: Fastest, file: "sum")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithoutState(level: Fastest, file: "TestDocument.pdf")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithoutState(level: Fastest, file: "sum")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress(level: Optimal, file: "TestDocument.pdf")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithoutState(level: Fastest, file: "alice29.txt")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithState(level: Fastest, file: "alice29.txt")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithState(level: Optimal, file: "alice29.txt")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithoutState(level: Optimal, file: "TestDocument.pdf")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress(level: Fastest, file: "TestDocument.pdf")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress(level: Optimal, file: "alice29.txt")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress(level: Fastest, file: "alice29.txt")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithState(level: Optimal, file: "TestDocument.pdf")

ETL Files

Histogram

JIT Disasms

System.IO.Compression.Brotli.Decompress_WithState(level: Fastest, file: "TestDocument.pdf")

ETL Files

Histogram

JIT Disasms

Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository


Run Information

Name Value
Architecture x64
OS Windows 10.0.22621
Queue TigerWindows
Baseline 7ff684af49cdf008bebbe10efcc10a2649795d65
Compare 063ae05c63307becef004eefbc8704ccb03c7f04
Diff Diff
Configs CompilationMode:tiered, RunKind:micro

Regressions in PerfLabTests.CastingPerf2.CastingPerf

Benchmark Baseline Test Test/Base Test Quality Edge Detector Baseline IR Compare IR IR Ratio
1.06 ms 1.27 ms 1.19 0.35 False

graph
Test Report

Repro

General Docs link: https://github.com/dotnet/performance/blob/main/docs/benchmarking-workflow-dotnet-runtime.md

git clone https://github.com/dotnet/performance.git
py .\performance\scripts\benchmarks_ci.py -f net8.0 --filter 'PerfLabTests.CastingPerf2.CastingPerf*'

PerfLabTests.CastingPerf2.CastingPerf.ObjrefValueTypeObj

ETL Files

Histogram

JIT Disasms

Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository

@performanceautofiler performanceautofiler bot added arch-x64 os-windows runtime-coreclr specific to the CoreCLR runtime untriaged New issue has not been triaged by the area owner labels Sep 3, 2024
@LoopedBard3 LoopedBard3 transferred this issue from dotnet/perf-autofiling-issues Sep 3, 2024
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-io-compression
See info in area-owners.md if you want to be subscribed.

@ericstj ericstj removed the untriaged New issue has not been triaged by the area owner label Sep 3, 2024
@ericstj ericstj added this to the 9.0.0 milestone Sep 3, 2024
@ericstj
Copy link
Member

ericstj commented Sep 3, 2024

@jkoritzinsky @carlossanlop - it seems these regressions are due to the brotli update. What's your take on this, do we resolve as by-design or should someone take a closer look and share the findings upstream?

@carlossanlop
Copy link
Member

Related discussion here: dotnet/perf-autofiling-issues#40587

@ericstj ericstj added the tracking-external-issue The issue is caused by external problem (e.g. OS) - nothing we can do to fix it directly label Sep 16, 2024
@carlossanlop
Copy link
Member

I opened an issue in google/brotli with our results to try to understand the regressions: google/brotli#1202

@carlossanlop
Copy link
Member

Since we are taking this brotli update anyway, I think we can close this issue. The discussion can continue in the issue I opened in the google/brotli repo.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-x64 area-System.IO.Compression os-windows runtime-coreclr specific to the CoreCLR runtime tracking-external-issue The issue is caused by external problem (e.g. OS) - nothing we can do to fix it directly
Projects
None yet
Development

No branches or pull requests

3 participants