Skip to content

bsp: optee: extend pkcs11 ta heap size#1065

Merged
ricardosalveti merged 1 commit intofoundriesio:mainfrom
igoropaniuk:pkcs11_ta_heap
Mar 14, 2023
Merged

bsp: optee: extend pkcs11 ta heap size#1065
ricardosalveti merged 1 commit intofoundriesio:mainfrom
igoropaniuk:pkcs11_ta_heap

Conversation

@igoropaniuk
Copy link
Copy Markdown
Contributor

Extend default PKCS11 TA heap size to 64Kb, as on some platforms the PKCS11 TA runs out of memory when generating the RSA keys [1] Set 256Kb PKCS11 TA heap size when SE05x support is enabled. Before it was set only for imx8mm-lpddr4-evk MACHINE.

[1] OP-TEE/optee_os#5022
Signed-off-by: Igor Opaniuk [email protected]

@igoropaniuk igoropaniuk force-pushed the pkcs11_ta_heap branch 3 times, most recently from c61ecfb to 0d504d1 Compare March 12, 2023 09:47
Comment thread meta-lmp-bsp/recipes-security/optee/optee-os-fio-bsp.inc Outdated
@MrCry0 MrCry0 self-requested a review March 14, 2023 09:04
@igoropaniuk igoropaniuk force-pushed the pkcs11_ta_heap branch 2 times, most recently from 294a58a to 7004e5c Compare March 14, 2023 09:24
* Extend default PKCS11 TA heap size to 64KB, as on some platforms
the PKCS11 TA runs out of memory when generating the RSA keys [1]
* Set 256KB PKCS11 TA heap size when SE05x support is enabled.
Before it was set only for imx8mm-lpddr4-evk MACHINE.

[1] OP-TEE/optee_os#5022
Signed-off-by: Igor Opaniuk <[email protected]>
Copy link
Copy Markdown
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM

@ricardosalveti ricardosalveti merged commit d2d75f4 into foundriesio:main Mar 14, 2023
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.

4 participants