Enable runtime-async for System.IO.Compression, System.IO.Compression.Brotli, and System.Net.WebSockets#124397
Enable runtime-async for System.IO.Compression, System.IO.Compression.Brotli, and System.Net.WebSockets#124397
Conversation
…ion.Brotli Co-authored-by: agocke <515774+agocke@users.noreply.github.com>
|
/benchmark |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
/benchmark micro |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
/benchmark micro libs |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
Crank Pull Request Bot
Benchmarks:
Profiles:
Components:
Arguments: any additional arguments to pass through to crank, e.g. |
|
/benchmark micro aspnet-perf-lin libs |
|
Benchmark started for micro on aspnet-perf-lin with libs. Logs: link |
|
An error occurred, please check the logs |
|
@copilot please also enable preview features for System.Net.WebSockets |
…ropagation for all three libraries Co-authored-by: agocke <515774+agocke@users.noreply.github.com>
Done in 06236ed. Also fixed all three libraries to use |
|
/benchmark micro aspnet-perf-lin libs |
|
Benchmark started for micro on aspnet-perf-lin with libs. Logs: link |
|
An error occurred, please check the logs |
|
Hey @agocke , we triaged this inside the team because it got labeled System.IO.Compression. But it seems like it's mislabeled, can you please change it to something more suitable? |
|
@DrewScoggins am I doing something wrong with the benchmarking here? It seems like this should work, but I can’t see the results and these errors don’t make sense |
|
@iremyux this is a draft pr, we can leave it untriaged |
|
Took a look, and the issue is that an environment variable is not being set properly, and we try to restore for older TFMs. Taking a step back, this feature was written by Sebestian, and we don't use it or really maintain it. I am going to spend some time today to look at this feature and get it working again, as well as understand what and how it works. |
|
Tagging subscribers to this area: @agocke |
Description
Enables the
runtime-asynccompiler feature on the System.IO.Compression, System.IO.Compression.Brotli, and System.Net.WebSockets source libraries to allow performance testing of runtime-async with compression and WebSocket workloads.<Features>$(Features);runtime-async=on</Features>withNoWarn CA2252suppression to all three library csproj fileseng/testing/tests.targets: excluded for browser, wasi, and Mono targets where runtime-async is unsupportedNoWarn CA2252instead ofEnablePreviewFeaturesto avoid propagating[assembly: RequiresPreviewFeatures]to downstream type-forwarding shim assemblies (e.g., theSystemshim)All existing tests pass (1400 System.IO.Compression, 328 Brotli, 570 System.Net.WebSockets). Full
clr+libsbuild succeeds with 0 warnings and 0 errors.💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.