Closed bigzachattack closed 1 year ago
This pull request was exported from Phabricator. Differential Revision: D42173995
This pull request was exported from Phabricator. Differential Revision: D42173995
This pull request was exported from Phabricator. Differential Revision: D42173995
This pull request was exported from Phabricator. Differential Revision: D42173995
This pull request was exported from Phabricator. Differential Revision: D42173995
This pull request has been merged in facebookincubator/dynolog@4edaf89847f111a87bda4601760c0e0952977549.
Summary: Many of the tests require running as root, so these test definitions have been updated to use
sudo_cpp_unittests
to allow for normal testing withbuck2 test
.BuiltinMetricsTest -- Init
was updated to account for some counters have a default sampling period of 0.BuiltinMetricsTest, PerfEventAttrPreciseJson
was updated to account for that newer Intel Processors do not have "UOPS_RETIRED.ALL" and when they do not "UOPS_RETIRED.RETIRE_SLOTS" lacks thepebs
feature and therefore not precise."CpuEventsGroup.h:getCount()" was modified check for time_running == 0 as there were occasionaly failures from "BPerfCountReader --SmokeTest" due to USAN on a nan being converted to unit. (nan was from dividing by zero.)
Differential Revision: D42173995