diff options
| author | 2020-07-16 17:37:39 +0800 | |
|---|---|---|
| committer | 2020-08-17 18:25:30 +0800 | |
| commit | 5377d797fff94e0e070fdbbadc89288d90cc5f2c (patch) | |
| tree | 3736850e2b9455052de0997991db40087db29997 /cmds/dumpstate/DumpPool.cpp | |
| parent | 141959209922a15acd27ee66e1408525f541a9dc (diff) | |
Faster bugreports (2/n)
- Having a system property to disable parallel run function.
- Post 'DumpTraces' to the thread pool. 3% is
improved compared with single thread run.
Bug: 136262402
Test: atest dumpstate_test
Test: atest dumpstate_smoke_test
Test: Manual enable and disable parallel run
Test: Manual trigger report using hardware key
Test: Manual trigger using bugreport shortcut
Change-Id: I2185cc3e2f429a150605d6268324c52d137db385
Diffstat (limited to 'cmds/dumpstate/DumpPool.cpp')
| -rw-r--r-- | cmds/dumpstate/DumpPool.cpp | 23 |
1 files changed, 22 insertions, 1 deletions
diff --git a/cmds/dumpstate/DumpPool.cpp b/cmds/dumpstate/DumpPool.cpp index 7324ead7c6..e174c8eff3 100644 --- a/cmds/dumpstate/DumpPool.cpp +++ b/cmds/dumpstate/DumpPool.cpp @@ -33,7 +33,8 @@ namespace dumpstate { const std::string DumpPool::PREFIX_TMPFILE_NAME = "dump-tmp."; -DumpPool::DumpPool(const std::string& tmp_root) : tmp_root_(tmp_root), shutdown_(false) { +DumpPool::DumpPool(const std::string& tmp_root) : tmp_root_(tmp_root), shutdown_(false), + log_duration_(true) { assert(!tmp_root.empty()); deleteTempFiles(tmp_root_); } @@ -99,6 +100,26 @@ void DumpPool::waitForTask(const std::string& task_name, const std::string& titl } } +void DumpPool::setLogDuration(bool log_duration) { + log_duration_ = log_duration; +} + +template <> +void DumpPool::invokeTask<std::function<void()>>(std::function<void()> dump_func, + const std::string& duration_title, int out_fd) { + DurationReporter duration_reporter(duration_title, /*logcat_only =*/!log_duration_, + /*verbose =*/false, out_fd); + std::invoke(dump_func); +} + +template <> +void DumpPool::invokeTask<std::function<void(int)>>(std::function<void(int)> dump_func, + const std::string& duration_title, int out_fd) { + DurationReporter duration_reporter(duration_title, /*logcat_only =*/!log_duration_, + /*verbose =*/false, out_fd); + std::invoke(dump_func, out_fd); +} + std::unique_ptr<DumpPool::TmpFile> DumpPool::createTempFile() { auto tmp_file_ptr = std::make_unique<TmpFile>(); std::string file_name_format = "%s/" + PREFIX_TMPFILE_NAME + "XXXXXX"; |