-
Notifications
You must be signed in to change notification settings - Fork 68
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
base: master
Are you sure you want to change the base?
Sdk11.1 cyclictest #352
Conversation
Since there are multiple platform variants for AM62x. |
"Average (us)","5","6","6","5" | ||
"Maximum (us)","68","31","34","28" | ||
"Average (us)","6","6","6","6" | ||
"Maximum (us)","45","30","39","64" |
There was a problem hiding this comment.
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
?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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" |
There was a problem hiding this comment.
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" |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
0c611af
to
f8839d9
Compare
Yeah I agree @praneethbajjuri . I've updated that sentence to be a little more specific. |
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" |
There was a problem hiding this comment.
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.
There was a problem hiding this 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.
moved it to draft based on #352 (review) |
There was a problem hiding this comment.
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>
f8839d9
to
b72f3fb
Compare
New warnings found with rstcheck:
|
@dao-qiu please review |
@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. |
test comment |
"Average (usec)","8","8" | ||
"Maximum (usec)","77","105" | ||
"Average (usec)","10","8" | ||
"Maximum (usec)","85","64" |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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?
No description provided.