diff options
| author | Primiano Tucci <primiano@google.com> | 2021-06-22 21:34:48 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-06-22 21:34:48 +0000 |
| commit | e51f43aa9d45aada2f7821511c2c4bc33fe47cee (patch) | |
| tree | bd5f7cceb352df8c782403fb067cd969519dfe41 | |
| parent | e49dacf5aebfff996828e6e61bebbbf63bc41d26 (diff) | |
| parent | f3f948836e49963a93f9348369adb6aceaaa1638 (diff) | |
| download | perfetto-e51f43aa9d45aada2f7821511c2c4bc33fe47cee.tar.gz | |
tracing service: improve logging for --save-for-bugreport am: f3f948836e
Original change: https://googleplex-android-review.googlesource.com/c/platform/external/perfetto/+/15059758
Change-Id: Ic0e8fdd4a2a88f54f8e6bc21c18a873c2d4d7a1c
| -rw-r--r-- | src/tracing/core/tracing_service_impl.cc | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/src/tracing/core/tracing_service_impl.cc b/src/tracing/core/tracing_service_impl.cc index 7084dec91..472bb319d 100644 --- a/src/tracing/core/tracing_service_impl.cc +++ b/src/tracing/core/tracing_service_impl.cc @@ -1808,8 +1808,9 @@ void TracingServiceImpl::FlushAndDisableTracing(TracingSessionID tsid) { PERFETTO_DLOG("Triggering final flush for %" PRIu64, tsid); auto weak_this = weak_ptr_factory_.GetWeakPtr(); Flush(tsid, 0, [weak_this, tsid](bool success) { - PERFETTO_DLOG("Flush done (success: %d), disabling trace session %" PRIu64, - success, tsid); + // This was a DLOG up to Jun 2021 (v16, Android S). + PERFETTO_LOG("FlushAndDisableTracing(%" PRIu64 ") done, success=%d", tsid, + success); if (!weak_this) return; TracingSession* session = weak_this->GetTracingSession(tsid); @@ -3173,6 +3174,12 @@ bool TracingServiceImpl::MaybeSaveTraceForBugreport( if (!max_session) return false; + PERFETTO_LOG("Seizing trace for bugreport. tsid:%" PRIu64 + " state:%d wf:%d score:%d name:\"%s\"", + max_tsid, max_session->state, !!max_session->write_into_file, + max_session->config.bugreport_score(), + max_session->config.unique_session_name().c_str()); + auto br_fd = CreateTraceFile(GetBugreportTmpPath(), /*overwrite=*/true); if (!br_fd) return false; |