jty2.merging.2025-05.postfix #87
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Rather than letting the compiler fail due to the Jansson header file missing, check if it is installed, and, if it is not, provide an informative error message for installing it or building without it. Change-Id: I6bf6195e9c27faa6096a3ca41223c41a4da8c6c5 Signed-off-by: jty2 <46021128+jty2@users.noreply.github.com>
rdtscp also writes to the ECX register. Fix clobber list so that register allocation is done correctly by the compiler. Change-Id: Ie3c6a5ab8fbeae46605642fc7971b0e1504dd7c2 Signed-off-by: jty2 <46021128+jty2@users.noreply.github.com>
The TSC frequency is not easy to obtain. The methods that this patch replace do not always work. However, the TSC frequency is used for blackhole and test durations, so an accurate number is important. This patch improves the TSC frequency measurement routine to support sub-second samples. On every run of the program, the frequency is measured for 3 samples of 0.1 seconds, dropping the highest and lowest. Change-Id: Ifc62210b8da586fbaebf0a3f22d72bc2eebda92b Signed-off-by: jty2 <46021128+jty2@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Always measure TSC frequency on x86
This patch corrects duration calculations and timings on x86 involving the TSC by measuring its frequency for 3x 0.1 seconds on every run of the program. This fixes #86 so that
--hwtimer-freqis not needed to get a correct measurement.Other patches included:
Makefile: improve Jansson-is-missing error handling
include/perf_timer.h: fix rdtscp register clobber list