Skip to content

fix(linux): Clarify Dhrystone is single-core scoring #719

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

Open
sathish-8 wants to merge 1 commit into
base: master
Choose a base branch
from
+8 −4
Open

fix(linux): Clarify Dhrystone is single-core scoring #719

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions source/devices/AM62X/linux/Linux_Performance_Guide.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -192,11 +192,15 @@
=========

Dhrystone is a core only benchmark that runs from warm L1 caches in all
modern processors. It scales linearly with clock speed.
modern processors. It scales linearly with clock speed. This is a single
core score depending on the current cpu clock frequency. The total DMIPS

Check warning on line 196 in source/devices/AM62X/linux/Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.Definitions] Define acronyms and abbreviations (such as 'DMIPS') on first occurrence if they're likely to be unfamiliar.

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'DMIPS') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62X/linux/Linux_Performance_Guide.rst", "range": {"start": {"line": 196, "column": 68}}}, "severity": "INFO"}
for all cores is at best the number of cores multiplied by the single core
score.

Please take note, different run may produce different slightly results.
This is advised to run this test multiple times in order to get maximum
performance numbers.

Please take note, different runs may produce slightly different results.

Check warning on line 201 in source/devices/AM62X/linux/Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.TermsWarnings] Consider using 'might' or 'can' rather than 'may' unless updating existing content that uses the term.

Raw Output:
{"message": "[RedHat.TermsWarnings] Consider using 'might' or 'can' rather than 'may' unless updating existing content that uses the term.", "location": {"path": "source/devices/AM62X/linux/Linux_Performance_Guide.rst", "range": {"start": {"line": 201, "column": 34}}}, "severity": "WARNING"}
It is advised to run this test multiple times in order to get maximum

Check warning on line 202 in source/devices/AM62X/linux/Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.TermsSuggestions] Depending on the context, consider using 'to' rather than 'in order to'.

Raw Output:
{"message": "[RedHat.TermsSuggestions] Depending on the context, consider using 'to' rather than 'in order to'.", "location": {"path": "source/devices/AM62X/linux/Linux_Performance_Guide.rst", "range": {"start": {"line": 202, "column": 47}}}, "severity": "INFO"}

Check warning on line 202 in source/devices/AM62X/linux/Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.TermsWarnings] Consider using 'to' rather than 'in order to' unless updating existing content that uses the term.

Raw Output:
{"message": "[RedHat.TermsWarnings] Consider using 'to' rather than 'in order to' unless updating existing content that uses the term.", "location": {"path": "source/devices/AM62X/linux/Linux_Performance_Guide.rst", "range": {"start": {"line": 202, "column": 47}}}, "severity": "WARNING"}

Check warning on line 202 in source/devices/AM62X/linux/Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.SimpleWords] Use simple language. Consider using 'many' rather than 'multiple'.

Raw Output:
{"message": "[RedHat.SimpleWords] Use simple language. Consider using 'many' rather than 'multiple'.", "location": {"path": "source/devices/AM62X/linux/Linux_Performance_Guide.rst", "range": {"start": {"line": 202, "column": 32}}}, "severity": "INFO"}

Check warning on line 202 in source/devices/AM62X/linux/Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.PassiveVoice] 'is advised' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.

Raw Output:
{"message": "[RedHat.PassiveVoice] 'is advised' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.", "location": {"path": "source/devices/AM62X/linux/Linux_Performance_Guide.rst", "range": {"start": {"line": 202, "column": 4}}}, "severity": "INFO"}
performance numbers or to average out the results from many trials.

Execute the benchmark with the following:

Expand Down
Loading