TraceProcessor: fix trace_bounds computation

The computation code was not taking into account the
duration of the last slice. So in a trace like this:

[   slice1   ][    slice 2   ][  slice 3  ]
|                             |
   Bounds before this CL.

This problem showed up only when analyzing ninja
logs, where the last event went missing.
Ironically this problem never shows up in most
ftrace-based traces, because after every slice
there is almost always a sched_wakeup event that
extends the bounds to the right values.

Bug: 149483774
Test: TraceProcessorIntegrationTest.TraceBounds
Change-Id: I202d1c9ce105f20e40a4eb3424e76f2a773615fa
2 files changed
tree: 50cdffa4f2fb5a275c9e9ce41a948e7467b62cd3
  1. bazel/
  2. build_overrides/
  3. buildtools/
  4. debian/
  5. docs/
  6. gn/
  7. include/
  8. infra/
  9. protos/
  10. src/
  11. test/
  12. tools/
  13. ui/
  14. .clang-format
  15. .gitignore
  16. .gn
  17. .style.yapf
  18. Android.bp
  19. Android.bp.extras
  20. BUILD
  21. BUILD.extras
  22. BUILD.gn
  23. codereview.settings
  24. heapprofd.rc
  25. LICENSE
  26. MODULE_LICENSE_APACHE2
  27. NOTICE
  28. OWNERS
  29. perfetto.rc
  30. PRESUBMIT.py
  31. README.chromium
  32. README.md
  33. TEST_MAPPING
  34. traced_perf.rc
  35. WORKSPACE
README.md

Perfetto - Performance instrumentation and tracing

Perfetto is an open-source project for performance instrumentation and tracing of Linux/Android/Chrome platforms and user-space apps.

See www.perfetto.dev for docs.

Contributing

See /docs/contributing.md for instructions.

The source-of-truth repo is Android's Gerrit. The GitHub repo is a read-only mirror.

Bugs

  • For bugs affecting Android or the tracing internals use the internal bug tracker (go/perfetto-bugs).
  • For bugs affecting Chrome use http://crbug.com, Component:Speed>Tracing label:Perfetto.

Community

You can reach us on our Discord channel. If you prefer using IRC we have an experimental Discord <> IRC bridge synced with #perfetto-dev on Freenode.