Bug 1687709 - enable the CPU utilization feature by default for startup profiling, r=gerald.

Differential Revision: https://phabricator.services.mozilla.com/D102451
This commit is contained in:
Florian Quèze 2021-01-21 11:05:25 +00:00
Родитель 70b524ecf0
Коммит a350b3e6d3
2 изменённых файлов: 6 добавлений и 3 удалений

Просмотреть файл

@ -235,9 +235,11 @@ static uint32_t DefaultFeatures() {
// Extra default features when MOZ_PROFILER_STARTUP is set (even if not
// available).
static uint32_t StartupExtraDefaultFeatures() {
// Enable CPUUtilization by default for startup profiles as it is useful to
// see when startup alternates between CPU intensive tasks and being blocked.
// Enable mainthreadio by default for startup profiles as startup is heavy on
// I/O operations, and main thread I/O is really important to see there.
return ProfilerFeature::MainThreadIO;
return ProfilerFeature::CPUUtilization | ProfilerFeature::MainThreadIO;
}
class MOZ_RAII PSAutoTryLock;

Просмотреть файл

@ -276,10 +276,11 @@ static uint32_t DefaultFeatures() {
// Extra default features when MOZ_PROFILER_STARTUP is set (even if not
// available).
static uint32_t StartupExtraDefaultFeatures() {
// Enable CPUUtilization by default for startup profiles as it is useful to
// see when startup alternates between CPU intensive tasks and being blocked.
// Enable file I/Os by default for startup profiles as startup is heavy on
// I/O operations.
return ProfilerFeature::MainThreadIO | ProfilerFeature::FileIO |
ProfilerFeature::FileIOAll;
return ProfilerFeature::CPUUtilization | ProfilerFeature::FileIOAll;
}
// The class is a thin shell around mozglue PlatformMutex. It does not preserve