Skip to content

Conversation

bryanbrattlof
Copy link
Collaborator

No description provided.

@praneethbajjuri
Copy link
Collaborator

Since there are multiple platform variants for AM62x.
Please document clearly on which board the performance numbers are captured on.
Under
"The latencies observed with this SDK are summarized below:"
may be update as
"The latencies observed with this default yocto SDK image on https://www.ti.com/tool/SK-AM62B-P1 and are summarized below:"

"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 ?

"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
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 ?

"Average (usec)","6","6","5","6"
"Maximum (usec)","30","33","31","35"
"Average (usec)","6","6","6","6"
"Maximum (usec)","34","74","35","39"
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

@bryanbrattlof
Copy link
Collaborator Author

Yeah I agree @praneethbajjuri . I've updated that sentence to be a little more specific.

@StaticRocket
Copy link
Member

Could you address the vale comments?

"Average (usec)","6","6","5","6"
"Maximum (usec)","30","33","31","35"
"Average (usec)","6","6","6","6"
"Maximum (usec)","34","74","35","39"
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.

Copy link
Collaborator

@praneethbajjuri praneethbajjuri left a comment

Choose a reason for hiding this comment

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

lets hold off on integration of this . Till the investigation completes on why latency numbers changed.
Might need to expand this PR to include the rootcause and workaround details to get better perf numbers.

@cshilwant cshilwant marked this pull request as draft July 11, 2025 16:19
@cshilwant
Copy link
Member

moved it to draft based on #352 (review)

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

Update the real time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <bb@ti.com>
@bryanbrattlof bryanbrattlof marked this pull request as ready for review October 2, 2025 13:35
Copy link

github-actions bot commented Oct 2, 2025

New warnings found with rstcheck:

source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:584: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:595: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:607: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:619: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:652: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:665: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:680: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:694: (SEVERE/4) Title level inconsistent:

@praneethbajjuri praneethbajjuri self-requested a review October 2, 2025 18:29
@praneethbajjuri
Copy link
Collaborator

@dao-qiu please review

@praneethbajjuri
Copy link
Collaborator

@bryanbrattlof can you update in commitmsg and histogram as well on what is the exact tag used for a given device to get this numbers.
Example:
AM62A 11.1 is RC7
AM62P/62/64/ 11.1 is RC5
AM62L 11.1 and AM62P 11.1.1 is RC16 ( Wait till this PR is merged #459 )

@StaticRocket
Copy link
Member

@dao-qiu
Copy link

dao-qiu commented Oct 3, 2025

test comment

"Average (usec)","8","8"
"Maximum (usec)","77","105"
"Average (usec)","10","8"
"Maximum (usec)","85","64"
Copy link

Choose a reason for hiding this comment

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

What is the targeted max threshold for AM64x that we are choosing? I'm hearing no more than AM62x < 75us and AM62a/am62p <50us, what about for AM64x and AM62Lx?

.. note::

A known issue in this SDK release is affecting this benchmark.
Applying `this patch`_ on Yocto's meta-ti layer will restore the
Copy link

Choose a reason for hiding this comment

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

Not all customers are likely going to have ability to rebuild Yocto to restore the switch latencies. Could there be instructions added that could be an alternative to having to rebuild Yocto for those who don't have the hardware/ability to do so?

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.

7 participants