Skip to content

feat(crypto): Add crypto perf table for different hardware #652

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

Draft
jsuhaas22 wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

feat(crypto): Add crypto perf table for different hardware #652

Show file tree
Hide file tree
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: 12 additions & 0 deletions source/devices/AM62LX/linux/_Crypto_Perf_AES_128_CBC.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
.. csv-table:: AES-128-CBC Performance

Check warning on line 1 in source/devices/AM62LX/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'CBC') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62LX/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 24}}}, "severity": "INFO"}

Check warning on line 1 in source/devices/AM62LX/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'AES') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62LX/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 16}}}, "severity": "INFO"}
:header: "Size (bytes)", "DTHEv2 Accelerator (MB/s)", "ARM CE (MB/s)", "ARM (MB/s)"
:widths: 30, 30, 30, 30

"16", "0.28", "86.86", "28.02"
"64", "1.16", "271.43", "35.57"
"256", "4.63", "568.25", "38.33"
"1024", "17.15", "800.83", "39.08"
"8192", "82.06", "908.72", "39.31"
"16384", "112.48", "917.23", "39.02"
"CPU Usage %", "18", "99", "99"

Comment on lines +1 to +12
Copy link
Copy Markdown
Member

@StaticRocket StaticRocket Apr 7, 2026

Choose a reason for hiding this comment

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

Maybe I'm just a stickler for this, but what unit is that 0.28 and other measurements reported for the given size? I assume it's some unit of time, but it's not particularly clear.

Copy link
Copy Markdown
Collaborator Author

@jsuhaas22 jsuhaas22 Apr 8, 2026

Choose a reason for hiding this comment

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

Its MB/s. I have mentioned it in column headings now.

11 changes: 11 additions & 0 deletions source/devices/AM62PX/linux/_Crypto_Perf_AES_128_CBC.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
.. csv-table:: AES-128-CBC Performance

Check warning on line 1 in source/devices/AM62PX/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'CBC') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62PX/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 24}}}, "severity": "INFO"}

Check warning on line 1 in source/devices/AM62PX/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'AES') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62PX/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 16}}}, "severity": "INFO"}
:header: "Size (bytes)", "SA2UL Accelerator (MB/s)", "ARM CE (MB/s)", "ARM (MB/s)"
:widths: 30, 30, 30, 30

"16", "0.43", "97.37", "31.46"
"64", "1.88", "304.29", "39.87"
"256", "7.18", "636.54", "42.97"
"1024", "23.80", "897.51", "43.82"
"8192", "71.31", "1018.75", "44.05"
"16384", "83.60", "1028.48", "43.99"
"CPU Usage %", "34%", "99%", "99%"
11 changes: 11 additions & 0 deletions source/devices/AM62X/linux/_Crypto_Perf_AES_128_CBC.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
.. csv-table:: AES-128-CBC Performance

Check warning on line 1 in source/devices/AM62X/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'CBC') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62X/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 24}}}, "severity": "INFO"}

Check warning on line 1 in source/devices/AM62X/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'AES') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM62X/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 16}}}, "severity": "INFO"}
:header: "Size (bytes)", "SA2UL Accelerator (MB/s)", "ARM CE (MB/s)", "ARM (MB/s)"
:widths: 30, 30, 30, 30

"16", "0.38", "95.83", "31.06"
"64", "1.60", "300.27", "39.69"
"256", "6.08", "632.25", "42.89"
"1024", "23.90", "895.17", "43.74"
"8192", "93.02", "1018.16", "44.04"
"16384", "117.17", "1028.05", "44.03"
"CPU Usage %", "38%", "99%", "99%"
11 changes: 11 additions & 0 deletions source/devices/AM64X/linux/_Crypto_Perf_AES_128_CBC.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
.. csv-table:: AES-128-CBC Performance

Check warning on line 1 in source/devices/AM64X/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'CBC') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM64X/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 24}}}, "severity": "INFO"}

Check warning on line 1 in source/devices/AM64X/linux/_Crypto_Perf_AES_128_CBC.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'AES') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/devices/AM64X/linux/_Crypto_Perf_AES_128_CBC.rst", "range": {"start": {"line": 1, "column": 16}}}, "severity": "INFO"}
:header: "Size (bytes)", "SA2UL Accelerator (MB/s)", "ARM CE (MB/s)", "ARM (MB/s)"
:widths: 30, 30, 30, 30

"16", "0.34", "69.43", "22.18"
"64", "1.51", "217.04", "28.34"
"256", "5.99", "454.25", "30.63"
"1024", "21.92", "640.51", "31.25"
"8192", "97.97", "726.61", "31.44"
"16384", "134.93", "727.48", "31.44"
"CPU Usage %", "39%", "99%", "99%"
10 changes: 10 additions & 0 deletions source/linux/Foundational_Components/Kernel/Kernel_Drivers/Crypto/DTHEv2.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,16 @@
Page size (bytes): 4096
Exit status: 0

*******************************
Hardware Performance Comparison

Check warning on line 216 in source/linux/Foundational_Components/Kernel/Kernel_Drivers/Crypto/DTHEv2.rst

View workflow job for this annotation

GitHub Actions / vale 

[vale] reported by reviewdog 🐶
[RedHat.Headings] Use sentence-style capitalization in 'Hardware Performance Comparison'.

Raw Output:
{"message": "[RedHat.Headings] Use sentence-style capitalization in 'Hardware Performance Comparison'.", "location": {"path": "source/linux/Foundational_Components/Kernel/Kernel_Drivers/Crypto/DTHEv2.rst", "range": {"start": {"line": 216, "column": 1}}}, "severity": "INFO"}
*******************************

The following table shows AES-128-CBC throughput measured using ``openssl speed``

Check warning on line 219 in source/linux/Foundational_Components/Kernel/Kernel_Drivers/Crypto/DTHEv2.rst

View workflow job for this annotation

GitHub Actions / vale 

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

Raw Output:
{"message": "[RedHat.Definitions] Define acronyms and abbreviations (such as 'AES') on first occurrence if they're likely to be unfamiliar.", "location": {"path": "source/linux/Foundational_Components/Kernel/Kernel_Drivers/Crypto/DTHEv2.rst", "range": {"start": {"line": 219, "column": 27}}}, "severity": "INFO"}
across the DTHEv2 hardware accelerator, ARM Cryptographic Extension (CE), and
baseline ARM CPU.

.. include:: ../../../../../devices/AM62LX/linux/_Crypto_Perf_AES_128_CBC.rst

******************************************************************
Using the True Random Number Generator (TRNG) Hardware Accelerator
******************************************************************
Expand Down
20 changes: 20 additions & 0 deletions source/linux/Foundational_Components/Kernel/Kernel_Drivers/Crypto/SA2UL_OMAP.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -300,6 +300,26 @@ software only implementation can be compared to the previous test.
Page size (bytes): 4096
Exit status: 0

*******************************
Hardware Performance Comparison
*******************************

The following table shows AES-128-CBC throughput measured using ``openssl speed``
across the SA2UL hardware accelerator, ARM Cryptographic Extension (CE), and
baseline ARM CPU.

.. ifconfig:: CONFIG_part_variant in ('AM62X')

.. include:: ../../../../../devices/AM62X/linux/_Crypto_Perf_AES_128_CBC.rst

.. ifconfig:: CONFIG_part_variant in ('AM62PX')

.. include:: ../../../../../devices/AM62PX/linux/_Crypto_Perf_AES_128_CBC.rst

.. ifconfig:: CONFIG_part_variant in ('AM64X')

.. include:: ../../../../../devices/AM64X/linux/_Crypto_Perf_AES_128_CBC.rst

Comment on lines +303 to +322
Copy link
Copy Markdown
Member

@StaticRocket StaticRocket Apr 7, 2026

Choose a reason for hiding this comment

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

Is there a particular reason this is getting special treatment and not being included in the platform specific performance report that's automatically generated from our test infra?

Copy link
Copy Markdown
Collaborator Author

@jsuhaas22 jsuhaas22 Apr 8, 2026

Choose a reason for hiding this comment

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

We don't have a test ready right now which compares the performance of crypto operations over the 3 hardware options. It's WIP.

***********************************
Using the TRNG Hardware Accelerator
***********************************
Expand Down
Loading