diff --git a/device.mk b/device.mk index f0d77499..0ef95c07 100644 --- a/device.mk +++ b/device.mk @@ -22,6 +22,7 @@ include device/google/gs-common/modem/modem.mk include device/google/gs-common/aoc/aoc.mk include device/google/gs-common/thermal/thermal.mk include device/google/gs-common/pixel_metrics/pixel_metrics.mk +include device/google/gs-common/performance/perf.mk TARGET_BOARD_PLATFORM := gs201 AB_OTA_POSTINSTALL_CONFIG += \ diff --git a/dumpstate/Dumpstate.cpp b/dumpstate/Dumpstate.cpp index d595bd6f..efa9e254 100644 --- a/dumpstate/Dumpstate.cpp +++ b/dumpstate/Dumpstate.cpp @@ -211,17 +211,6 @@ void endSection(int fd, const std::string §ionName, timepoint_t startTime) { "\n", fd); } -// If you are adding a single RunCommandToFd() or DumpFileToFd() call, please -// add it to dumpMiscSection(). But if you are adding multiple items that are -// related to each other - for instance, for a Foo peripheral - please add them -// to a new dump function and include it in this table so it can be accessed from the -// command line, e.g.: -// dumpsys android.hardware.dumpstate.IDumpstateDevice/default foo -// -// However, if your addition generates attachments and/or binary data for the -// bugreport (i.e. if it requires two file descriptors to execute), it must not be -// added to this table and should instead be added to dumpstateBoard() below. - Dumpstate::Dumpstate() : mTextSections{ { "wlan", [this](int fd) { dumpWlanSection(fd); } }, @@ -229,7 +218,6 @@ Dumpstate::Dumpstate() { "Devfreq", [this](int fd) { dumpDevfreqSection(fd); } }, { "power", [this](int fd) { dumpPowerSection(fd); } }, { "display", [this](int fd) { dumpDisplaySection(fd); } }, - { "misc", [this](int fd) { dumpMiscSection(fd); } }, { "led", [this](int fd) { dumpLEDSection(fd); } }, { "pixel-trace", [this](int fd) { dumpPixelTraceSection(fd); } }, }, @@ -286,6 +274,7 @@ void Dumpstate::dumpTextSection(int fd, const std::string §ionName) { } if (dumpAll) { + RunCommandToFd(fd, "VENDOR PROPERTIES", {"/vendor/bin/getprop"}); return; } @@ -596,12 +585,6 @@ void Dumpstate::dumpDisplaySection(int fd) { } } -// Dump items that don't fit well into any other section -void Dumpstate::dumpMiscSection(int fd) { - RunCommandToFd(fd, "VENDOR PROPERTIES", {"/vendor/bin/getprop"}); - DumpFileToFd(fd, "VENDOR PROC DUMP", "/proc/vendor_sched/dump_task"); -} - // Dump items related to LED void Dumpstate::dumpLEDSection(int fd) { struct stat buffer; diff --git a/dumpstate/Dumpstate.h b/dumpstate/Dumpstate.h index f615a452..75079e31 100644 --- a/dumpstate/Dumpstate.h +++ b/dumpstate/Dumpstate.h @@ -57,7 +57,6 @@ class Dumpstate : public BnDumpstateDevice { void dumpDevfreqSection(int fd); void dumpMemorySection(int fd); void dumpDisplaySection(int fd); - void dumpMiscSection(int fd); void dumpLEDSection(int fd); void dumpPixelTraceSection(int fd);