[Debugger] Reduce noisy Info logs to debug in native code#8618
Conversation
BenchmarksBenchmark execution time: 2026-05-12 19:51:57 Comparing candidate commit 122c5d2 in PR branch Some scenarios are present only in baseline or only in candidate runs. If you didn't create or remove some scenarios in your branch, this maybe a sign of crashed benchmarks 💥💥💥 Scenarios present only in baseline:
Found 1 performance improvements and 3 performance regressions! Performance is the same for 49 metrics, 19 unstable metrics, 89 known flaky benchmarks, 37 flaky benchmarks without significant changes.
|
Execution-Time Benchmarks Report ⏱️Execution-time results for samples comparing This PR (8618) and master. ✅ No regressions detected - check the details below Full Metrics ComparisonFakeDbCommand
HttpMessageHandler
Comparison explanationExecution-time benchmarks measure the whole time it takes to execute a program, and are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are highlighted in **red**. The following thresholds were used for comparing the execution times:
Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard. Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph). Duration chartsFakeDbCommand (.NET Framework 4.8)gantt
title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (74ms) : 70, 78
master - mean (74ms) : 70, 77
section Bailout
This PR (8618) - mean (78ms) : 76, 79
master - mean (78ms) : 75, 80
section CallTarget+Inlining+NGEN
This PR (8618) - mean (1,098ms) : 1050, 1147
master - mean (1,100ms) : 1038, 1163
FakeDbCommand (.NET Core 3.1)gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (114ms) : 110, 119
master - mean (114ms) : 111, 118
section Bailout
This PR (8618) - mean (114ms) : 111, 116
master - mean (117ms) : 114, 119
section CallTarget+Inlining+NGEN
This PR (8618) - mean (795ms) : 774, 816
master - mean (787ms) : 764, 810
FakeDbCommand (.NET 6)gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (99ms) : 96, 102
master - mean (106ms) : 100, 112
section Bailout
This PR (8618) - mean (104ms) : 98, 110
master - mean (103ms) : 98, 107
section CallTarget+Inlining+NGEN
This PR (8618) - mean (945ms) : 910, 981
master - mean (944ms) : 906, 982
FakeDbCommand (.NET 8)gantt
title Execution time (ms) FakeDbCommand (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (102ms) : 97, 108
master - mean (101ms) : 96, 107
section Bailout
This PR (8618) - mean (102ms) : 100, 104
master - mean (106ms) : 100, 111
section CallTarget+Inlining+NGEN
This PR (8618) - mean (831ms) : 794, 867
master - mean (828ms) : 790, 866
HttpMessageHandler (.NET Framework 4.8)gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (200ms) : 194, 205
master - mean (192ms) : 189, 195
section Bailout
This PR (8618) - mean (202ms) : 197, 207
master - mean (195ms) : 193, 197
section CallTarget+Inlining+NGEN
This PR (8618) - mean (1,206ms) : 1154, 1258
master - mean (1,166ms) : 1130, 1202
HttpMessageHandler (.NET Core 3.1)gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (290ms) : 283, 296
master - mean (274ms) : 271, 277
section Bailout
This PR (8618) - mean (289ms) : 284, 294
master - mean (276ms) : 267, 285
section CallTarget+Inlining+NGEN
This PR (8618) - mean (962ms) : 940, 985
master - mean (939ms) : 910, 967
HttpMessageHandler (.NET 6)gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (284ms) : 276, 291
master - mean (269ms) : 263, 275
section Bailout
This PR (8618) - mean (285ms) : 278, 291
master - mean (276ms) : 268, 284
section CallTarget+Inlining+NGEN
This PR (8618) - mean (1,158ms) : 1117, 1200
master - mean (1,129ms) : 1074, 1183
HttpMessageHandler (.NET 8)gantt
title Execution time (ms) HttpMessageHandler (.NET 8)
dateFormat x
axisFormat %Q
todayMarker off
section Baseline
This PR (8618) - mean (283ms) : 277, 290
master - mean (270ms) : 259, 280
section Bailout
This PR (8618) - mean (284ms) : 275, 293
master - mean (274ms) : 258, 290
section CallTarget+Inlining+NGEN
This PR (8618) - mean (1,046ms) : 996, 1096
master - mean (1,025ms) : 978, 1072
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Summary of changes
Lower native log calls from
InfotoDebugin the debugger / dynamic instrumentation native code (debugger_probes_instrumentation_requester.cpp,debugger_rejit_preprocessor.cpp,debugger_method_rewriter.cpp).Reason for change
These messages fire per-method, per-loop-iteration, or per-polled-call and were noisy at default verbosity without adding information beyond the batch-summary
Infologs that already stay atInfo.Implementation details
Level-only change. No strings modified, no behavior change. No
IsDebugEnabled()guards needed.Test coverage
All existing tests.