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.

Sign up for GitHub

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

Jump to bottom

Exterimental: Performance: Artificially synchronized TSC #16618

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Exterimental: Performance: Artificially synchronized TSC #16618

wants to merge 1 commit into from
+29 −4

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Jan 25, 2025

If we cannot trust the hardware, make the hardware trust us.

Test with #16545 and #16604

@elad335 elad335 added the Performance Performance related, not directly an optimization. label Jan 25, 2025
@elad335 elad335 mentioned this pull request Jan 25, 2025
Open
@kd-11
Copy link
Contributor

kd-11 commented Jan 25, 2025

I don't think TSC variance is the issue. It's the calculated frequency that usually has problems, at least since the late 2000s.

@daviDarthemis
Copy link

In arm environment the tsc is calculated with 19hz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Performance Performance related, not directly an optimization.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@elad335 @kd-11 @daviDarthemis