Skip to content

Sdk11.1 cyclictest #352

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

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
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
6 changes: 3 additions & 3 deletions source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At line no. 2,

RT-linux 11.01 Performance Guide

Original file line number Diff line number Diff line change
Expand Up @@ -57,14 +57,14 @@
stress-ng --cpu-method=all -c 4 &
cyclictest -m -Sp80 -D6h -h400 -i200 -M -q

The latencies where observed with this SDK are summarized below:
The latencies observed with the default yocto SDK are summarized below:

Check warning on line 60 in source/devices/AM62AX/linux/RT_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 'after', 'later', or 'following' rather than 'below'. Raw Output: {"message": "[RedHat.TermsSuggestions] Depending on the context, consider using 'after', 'later', or 'following' rather than 'below'.", "location": {"path": "source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 60, "column": 66}}}, "severity": "INFO"}

Check warning on line 60 in source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice. Raw Output: {"message": "[RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.", "location": {"path": "source/devices/AM62AX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 60, "column": 51}}}, "severity": "INFO"}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rewording as
"The latencies observed with the default yocto SDK is in the following table:" could fix the vale comment. Check by updating if this fixes across the devices.


.. csv-table::
:header: "Latencies","CPU 0","CPU 1","CPU 2","CPU 3"

"Minimum (us)","5","5","5","5"
"Average (us)","5","6","6","5"
"Maximum (us)","68","31","34","28"
"Average (us)","6","6","6","6"
"Maximum (us)","45","30","39","64"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@praneethbajjuri AM62A coreSDK RC for 11.1 would still be +2 weeks from now.
Most likely nothing in vendor kernel should impact the RT Latency numbers but any chances of
stable update from branch linux-6.12.y of git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux into ti-linux-6.12.y-cicd ?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cshilwant this numbers are ok for now. Will re-do AM62A and AM62L RT test closer to final week of release again and update the numbers if there is a delta.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, ACK from my end once #352 (comment) is fixed

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bryanbrattlof @praneethbajjuri our target for 62A is to keep below 50us. 64us seems a regression ?


.. image:: img/rt-cpu-method-all-latency-histogram.png

Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
6 changes: 3 additions & 3 deletions source/devices/AM62LX/linux/RT_Linux_Performance_Guide.rst
Original file line number Diff line number Diff line change
Expand Up @@ -56,13 +56,13 @@
stress-ng --cpu-method=all -c 4 &
cyclictest -m -Sp80 -D6h -h400 -i200 -M -q

The latencies observed with this SDK are summarized below:
The latencies observed with the default yocto SDK are summarized below:

Check warning on line 59 in source/devices/AM62LX/linux/RT_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 'after', 'later', or 'following' rather than 'below'. Raw Output: {"message": "[RedHat.TermsSuggestions] Depending on the context, consider using 'after', 'later', or 'following' rather than 'below'.", "location": {"path": "source/devices/AM62LX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 59, "column": 66}}}, "severity": "INFO"}

Check warning on line 59 in source/devices/AM62LX/linux/RT_Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice. Raw Output: {"message": "[RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.", "location": {"path": "source/devices/AM62LX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 59, "column": 51}}}, "severity": "INFO"}

.. csv-table::
:header: "Latencies","CPU 0","CPU 1"

"Minimum (us)","5","5"
"Average (us)","6","8"
"Maximum (us)","47","53"
"Average (us)","7","7"
"Maximum (us)","61","60"

.. image:: img/rt-cpu-method-all-latency-histogram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
6 changes: 3 additions & 3 deletions source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,13 @@
stress-ng --cpu-method=all -c 4 &
cyclictest -m -Sp80 -D6h -h400 -i200 -M -q

The latencies observed with this SDK are summarized below:
The latencies observed with the default yocto SDK are summarized below:

Check warning on line 60 in source/devices/AM62PX/linux/RT_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 'after', 'later', or 'following' rather than 'below'. Raw Output: {"message": "[RedHat.TermsSuggestions] Depending on the context, consider using 'after', 'later', or 'following' rather than 'below'.", "location": {"path": "source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 60, "column": 66}}}, "severity": "INFO"}

Check warning on line 60 in source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice. Raw Output: {"message": "[RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.", "location": {"path": "source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 60, "column": 51}}}, "severity": "INFO"}

.. csv-table::
:header: "Latencies","CPU 0","CPU 1","CPU 2","CPU 3"

"Minimum (usec)","5","5","5","5"
"Average (usec)","6","6","5","6"
"Maximum (usec)","30","33","31","35"
"Average (usec)","6","6","6","6"

Check warning on line 66 in source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [RedHat.Spelling] Verify the word 'usec'. It is not in the American English spelling dictionary used by Vale. Raw Output: {"message": "[RedHat.Spelling] Verify the word 'usec'. It is not in the American English spelling dictionary used by Vale.", "location": {"path": "source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 66, "column": 14}}}, "severity": "WARNING"}
"Maximum (usec)","34","74","35","39"

Check warning on line 67 in source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [RedHat.Spelling] Verify the word 'usec'. It is not in the American English spelling dictionary used by Vale. Raw Output: {"message": "[RedHat.Spelling] Verify the word 'usec'. It is not in the American English spelling dictionary used by Vale.", "location": {"path": "source/devices/AM62PX/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 67, "column": 14}}}, "severity": "WARNING"}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

74us needs to be investiagated. This is more than x2 the original number

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah these numbers are all over the place for all platforms. I've opened a ticket internally to follow up on this

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this usec vale warning can be ignored.


.. image:: img/rt-cpu-method-all-latency-histogram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 4 additions & 3 deletions source/devices/AM62X/linux/RT_Linux_Performance_Guide.rst
Original file line number Diff line number Diff line change
Expand Up @@ -61,13 +61,14 @@
stress-ng --cpu-method=all -c 4 &
cyclictest -m -Sp80 -D6h -h400 -i200 -M -q

The latencies observed with this SDK are summarized below:
The latencies observed with the default yocto SDK image on
https://www.ti.com/tool/SK-AM62B-P1 are summarized below:

Check warning on line 65 in source/devices/AM62X/linux/RT_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 'after', 'later', or 'following' rather than 'below'. Raw Output: {"message": "[RedHat.TermsSuggestions] Depending on the context, consider using 'after', 'later', or 'following' rather than 'below'.", "location": {"path": "source/devices/AM62X/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 65, "column": 52}}}, "severity": "INFO"}

Check warning on line 65 in source/devices/AM62X/linux/RT_Linux_Performance_Guide.rst

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice. Raw Output: {"message": "[RedHat.PassiveVoice] 'are summarized' is passive voice. In general, use active voice. Consult the style guide for acceptable use of passive voice.", "location": {"path": "source/devices/AM62X/linux/RT_Linux_Performance_Guide.rst", "range": {"start": {"line": 65, "column": 37}}}, "severity": "INFO"}

.. csv-table::
:header: "Latencies","CPU 0","CPU 1","CPU 2","CPU 3"

"Minimum (usec)","5","5","4","4"
"Average (usec)","7","5","6","6"
"Maximum (usec)","66","41","73","70"
"Average (usec)","6","5","6","6"
"Maximum (usec)","75","77","62","73"

.. image:: img/rt-cpu-method-all-latency-histogram.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,8 @@ The latencies observed with this DK are summarized below:
:header: "Latencies","CPU 0","CPU 1"

"Minimum (usec)","6","6"
"Average (usec)","8","8"
"Maximum (usec)","77","105"
"Average (usec)","10","8"
"Maximum (usec)","85","64"

.. image:: img/rt-cpu-method-all-latency-histogram.png

Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading