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

Rev'd to 5.1.4 to show results differernce #96

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

Conversation

opsiff
Copy link

@opsiff opsiff commented May 21, 2024

The following commits change results very huge,
but use the same version number in some test result.
More and more people use this version which not
release for likely riscv arch,and post it to ther Internet.
Simplely change the version ,we can compare results
not confused.

commit 81e9de5 ("fstime.c - Seperate r/w files for each parallel") and
commit fb4521c ("Allocate unique fd for each parallel to avoid unexpected syscall test")

The following commits change results very huge,
but use the same version number in some test result.
More and more people use this version which not
release for likely riscv arch,and post it to ther Internet.
Simplely change the version ,we can compare results
 not confused.

commit 81e9de5  ("fstime.c - Seperate r/w files for each parallel")
and commit fb4521c  ("
Allocate unique fd for each parallel to avoid unexpected syscall test")

Signed-off-by: Wentao Guan <[email protected]>
@opsiff
Copy link
Author

opsiff commented May 21, 2024

I have a story of one person who used 'git clone' to download the repository's code for benchmarking, while another person downloaded the repository's release for benchmarking, revealing a significant difference in scores :) .
And this story has happened repeatedly within a year, occurring in the company's internal tests, among video website bloggers, and in online chats.

@gstrauss
Copy link
Collaborator

Interesting. That version appears to have been last updated in Jan 2011 in 7df8999

@opsiff
Copy link
Author

opsiff commented May 21, 2024

Interesting. That version appears to have been last updated in Jan 2011 in 7df8999

Yes.
The benchmark results change so different with #85 and #82 ,bring the big change in FileCopy and Syscall Benchmark.
They are just happened in the last two years.
People share their benchmark score with differnet parallelism program code, and the same unixbench version number:).

@opsiff
Copy link
Author

opsiff commented May 21, 2024

I think that if not a release version, version number are not important, but show user "Oh ,the benchmanrk happened in difference program version.".

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