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

Add GpuTimeSync event #644

Merged
merged 3 commits into from
Oct 23, 2023
Merged

Add GpuTimeSync event #644

merged 3 commits into from
Oct 23, 2023

Conversation

YaLTeR
Copy link
Contributor

@YaLTeR YaLTeR commented Oct 22, 2023

Allows to resynchronise GPU and CPU timestamps during profiling. I tested this with my Mutter setup, and it seems to work very well, even with the default power profile.

Do I need to bump the file format version and/or meson version, and to what number?

Allows to resynchronise GPU and CPU timestamps during profiling.
@wolfpld
Copy link
Owner

wolfpld commented Oct 22, 2023

Do I need to bump the file format version and/or meson version, and to what number?

No, these are fine as they are. But you have altered the network protocol and thus you must bump its revision. See public/common/TracyProtocol.hpp.

@wolfpld
Copy link
Owner

wolfpld commented Oct 22, 2023

Oh, and you should also document this in the manual.

@YaLTeR
Copy link
Contributor Author

YaLTeR commented Oct 23, 2023

Did both.

@wolfpld wolfpld merged commit 4c8944b into wolfpld:master Oct 23, 2023
5 checks passed
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

Successfully merging this pull request may close these issues.

2 participants