Closed gleocadie closed 1 month ago
Branch report: gleocadie/fix-dliteratephdr-tests
Commit report: b165d41
Test service: dd-trace-dotnet
:white_check_mark: 0 Failed, 364230 Passed, 2340 Skipped, 15h 37m 11.21s Total Time :hourglass: 1 Performance Regression
Profiler_allocation_cpu_walltime
- scenarios
2.16s (+36.82ms, +2%) - DetailsExecution-time results for samples comparing the following branches/commits:
Execution-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 shown 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).
Benchmarks for #5963 compared to master:
The following thresholds were used for comparing the benchmark speeds:
Allocation changes below 0.5% are ignored.
Summary of changes
Fix
LinuxDlIteratePhdrDeadlock
test.Reason for change
The test is randomly failing and not because there is a deadlock, but only because it's not correct. It uses the
dlopen
anddlclose
fromlibdl.so
library, instead of the ones we wrapped in theDatadog.Linux.ApiWrapper.x64.so
file.Implementation details
NativeLibrary.SetDllImportResolver
to make sure we control how we want to resolve for ourApiWrapper
library.Test coverage
Other details