-
Notifications
You must be signed in to change notification settings - Fork 3k
Update mbed TLS to version 2.6.0 #4987
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
Conversation
@yanesca @sbutcher-arm: Could you please take a look at this PR? Once this is done, I will remove the DRAFT label from the title. Please note that I have added a new line to the |
Enable the compile-time option MBEDTLS_AES_ROM_TABLES in the mbed TLS main config.h file in mbed OS. This option has the effect of labelling the AES tables as 'const' so they are placed in ROM, which saves some RAM space.
Looks good to me. |
/morph test |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
@andresag01 Is this ready? There's still DRAFT in the title though. please keep us updated. Req for 5.6? |
@studavekar I believe this one should go with morph test nightly? (#4987 (comment)) |
/morph test-nightly |
@0xc0170: Currently waiting for approval from mbed TLS developers. |
This PR is approved by mbed TLS. Just needs CI as far as I can see. |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
@studavekar Please can you look at the failure, this is the second test I noticed today with arch pro failures (not related for any of these 2 runs). |
OK, I think I know what is the problem, after looking at the log closer, it's out of heap memory. going to look at the change that could introduce this |
Depends on #5037 |
I just merged #5037, so testing should be unblocked now. /morph test-nightly |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputExample Build failed! |
We will investigate the failures (it's CI failure not this PR). I restarted on another PR a regular test to find out if this was one time or not. |
/morph test-nightly |
1 similar comment
/morph test-nightly |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
/morph test-nightly |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Description
Update mbed OS to use the mbed TLS 2.6.0, the latest released version. For more information on bug fixes, improvements, new features, etc please refer to the mbed TLS ChangeLog.
Status
READY
Migrations
YES
Minor API changes. In particular, two new functions were added to mbed TLS to allow hooks for initialising and freeing system-wide resources before any accelerators, etc are used.