Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display dma_fence events in a single row #755

Open
gramound opened this issue Apr 5, 2024 · 2 comments
Open

Display dma_fence events in a single row #755

gramound opened this issue Apr 5, 2024 · 2 comments

Comments

@gramound
Copy link

gramound commented Apr 5, 2024

The "dma_fence" events currently show-up 1 per row (format "%s-%s%llu-%lld", driver_name, timeline_name, context, seqno).
I'm looking at a 25s trace with over 3000 rows under "Misc Global Tracks" named like driver-timelinex-1000, driver-timelinex-1001, driver-timelinex-1002, driver-timelinex-1003, etc. Each row containing only 1 "fence" slice.

It would be nice if all the seqno of the same "driver_name,timeline_name,context" could be presented in the same row (with the seqno in the slice name).

Is this possible to do within the viewer, or does it require the Linux driver to present its events differently?

@LalitMaganti
Copy link
Collaborator

LalitMaganti commented Apr 5, 2024

Yes it's possible to make the change within trace processor: you just need to change the code at https://cs.android.com/android/platform/superproject/main/+/main:external/perfetto/src/trace_processor/importers/ftrace/drm_tracker.cc;bpv=1

Patches to fix this welcome (contribution guide is available at https://perfetto.dev/docs/contributing/getting-started) but it's likely not something we will get to ourselves soon.

@gramound
Copy link
Author

gramound commented Apr 5, 2024

Thanks for the code link. I think I misunderstood the track name format in the issue description. According to GetFenceTimelineByContext, what I assumed to be seqno (number at the end of the track name) were actually contexts. It's probably something specific to the driver I'm investigating to increment contexts for each fence. I'll study this a bit more (might take a while).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants