Skip to content

Sdk11.1 cyclictest and misc cleanups #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.

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

Jump to bottom
Merged
merged 8 commits into from
Oct 16, 2025
Merged

Sdk11.1 cyclictest and misc cleanups #352

merged 8 commits into from
Oct 16, 2025

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

cshilwant
cshilwant reviewed Jul 9, 2025
View reviewed changes
r-vignesh
r-vignesh reviewed Jul 10, 2025
View reviewed changes
@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?

praneethbajjuri
praneethbajjuri reviewed Jul 10, 2025
View reviewed changes
praneethbajjuri
praneethbajjuri requested changes Jul 11, 2025
View reviewed changes
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)

cshilwant
cshilwant requested changes Aug 1, 2025
View reviewed changes
@bryanbrattlof bryanbrattlof marked this pull request as ready for review October 2, 2025 13:35
@github-actions GitHub Actions
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
Copy link
Collaborator

@dao-qiu please review

praneethbajjuri
praneethbajjuri previously approved these changes Oct 2, 2025
View reviewed changes
@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

https://github.com/TexasInstruments/processor-sdk-doc/pull/352/files/b72f3fb58404e5e18a43712b21d10b6f30d5a17d#r2247290922

@praneethbajjuri praneethbajjuri changed the title Sdk11.1 cyclictest Sdk11.1 cyclictest and misc cleanups Oct 13, 2025
StaticRocket
StaticRocket reviewed Oct 13, 2025
View reviewed changes
Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

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

Those rstcheck errors are valid. Seems like the template needs to be adjusted.

@bryanbrattlof
chore: use proper sections
b79f38f 
The lack of sections in this page makes cross linking difficult. Convert
the bullet points and rubrics to proper sections.

Signed-off-by: Bryan Brattlof <[email protected]>
@github-actions GitHub Actions
Copy link

New warnings found with rstcheck:

source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:598: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:610: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:622: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:655: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:668: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:683: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:697: (SEVERE/4) Title level inconsistent:

StaticRocket
StaticRocket previously approved these changes Oct 14, 2025
View reviewed changes
StaticRocket
StaticRocket reviewed Oct 14, 2025
View reviewed changes
@dao-qiu
Copy link

dao-qiu commented Oct 15, 2025 •
edited
Loading

@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 )

@bryanbrattlof Is there any feedback on the exact RCs used for the cyclictest results?

From Bryan: I updated all the graphs to include which release candidate was used by each board

@bryanbrattlof
feat(linux): add section for building OPTEE with PRNG drivers
ffa5d94 
In certain highly specific use-cases the TRNG drivers in OPTEE could
have unwanted affects to the overall system if the application cares
more about deterministic latency over entropy.

Provide a section to document how to easily switch from the true RNG
drivers to pseudo RNG drivers.

Signed-off-by: Bryan Brattlof <[email protected]>
@bryanbrattlof
feat(linux): Add OP-TEE section to AM62LX builds
7da73b7 
Newer silicon revisions can now integrate OP-TEE into their builds. Add
the OP-TEE foundation component back into the AM62LX build

Signed-off-by: Bryan Brattlof <[email protected]>
@bryanbrattlof
perf(linux): AM62AX: Update PREEMPT_RT perf numbers for 11.1
db50b76 
Update the real time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <[email protected]>
@bryanbrattlof
perf(linux): AM62LX: Update PREEMPT_RT perf numbers for 11.1
c0be088 
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <[email protected]>
@bryanbrattlof
perf(linux): AM62PX: Update PREEMPT_RT perf numbers for 11.1
46cd1d0 
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <[email protected]>
@bryanbrattlof
perf(linux): AM62X: Update PREEMPT_RT perf numbers for 11.1
b04f6d8 
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <[email protected]>
@bryanbrattlof
perf(linux): AM64X: Update PREEMPT_RT perf numbers for 11.1
903f3b8 
Update the real-time Linux latency numbers for SDK 11.1

Signed-off-by: Bryan Brattlof <[email protected]>
@github-actions GitHub Actions
Copy link

New warnings found with rstcheck:

source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:598: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:610: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:622: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:655: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:668: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:683: (SEVERE/4) Title level inconsistent:
source/devices/AM64X/linux/RT_Linux_Performance_Guide.rst:697: (SEVERE/4) Title level inconsistent:

@bryanbrattlof
Copy link
Collaborator Author

@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 )

@bryanbrattlof Is there any feedback on the exact RCs used for the cyclictest results?

Ah sorry (I didn't see a reponse dialog and skipped it) yep I've updated the graphs to indicate which release candidate was used for each

@cshilwant cshilwant requested a review from r-vignesh October 16, 2025 15:57
@praneethbajjuri praneethbajjuri merged commit ae7ac37 into TexasInstruments:master Oct 16, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@praneethbajjuri praneethbajjuri praneethbajjuri approved these changes

@StaticRocket StaticRocket StaticRocket approved these changes

@cshilwant cshilwant cshilwant approved these changes

@uditkumarti uditkumarti Awaiting requested review from uditkumarti uditkumarti is a code owner

@aniket-l aniket-l Awaiting requested review from aniket-l aniket-l is a code owner

@VeeruPrudhvi VeeruPrudhvi Awaiting requested review from VeeruPrudhvi VeeruPrudhvi is a code owner

@r-vignesh r-vignesh Awaiting requested review from r-vignesh

+2 more reviewers

@DhruvaG2000 DhruvaG2000 DhruvaG2000 approved these changes

@dao-qiu dao-qiu dao-qiu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@r-vignesh r-vignesh

@bryanbrattlof bryanbrattlof

@DhruvaG2000 DhruvaG2000

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@bryanbrattlof @praneethbajjuri @StaticRocket @cshilwant @dao-qiu @r-vignesh @DhruvaG2000