Skip to content

Mbed TLS v4.1.0 update fixup#29011

Merged
rlubos merged 5 commits into
nrfconnect:mainfrom
tomi-font:mbedtls_update_4.1.0_fixup
May 29, 2026
Merged

Mbed TLS v4.1.0 update fixup#29011
rlubos merged 5 commits into
nrfconnect:mainfrom
tomi-font:mbedtls_update_4.1.0_fixup

Conversation

@tomi-font
Copy link
Copy Markdown
Contributor

Fix CI issues that have appeared with #28496.

@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented May 26, 2026

CI Information

To view the history of this post, click the 'edited' button above
Build number: 11

Inputs:

Sources:

sdk-nrf: PR head: 2693654e3b692ba544e5ed039a21cdc5900e64e4
mbedtls: PR head: 5b7ca76723b3aa77a9ab066e552fe0dca4975f45
oberon-psa-crypto: PR head: ce39726a33b6263b8a006ba404f78b209be1806c
zephyr: PR head: 6086d15e0350f5c3359a9500ef80cea23fc9f8b2

more details

sdk-nrf:

PR head: 2693654e3b692ba544e5ed039a21cdc5900e64e4
merge base: d58dd7466e9bee9dbb39219914b93ad3d775fd87
target head (main): 215de2686917bd9261dd9bbb362f3b2154955aea
Diff

mbedtls:

PR head: 5b7ca76723b3aa77a9ab066e552fe0dca4975f45
merge base: 92d44566209a0d94a571cc7f71a4a7b9262f7602
Diff

oberon-psa-crypto:

PR head: ce39726a33b6263b8a006ba404f78b209be1806c
merge base: 3a9005029dae919e3f6d9d4182465a8e2572cf9f
Diff

zephyr:

PR head: 6086d15e0350f5c3359a9500ef80cea23fc9f8b2
merge base: a9f47784f568c8b4ede9e3b6ea53758e5a391c93
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (10)
modules
│  ├── crypto
│  │  ├── mbedtls
│  │  │  ├── library
│  │  │  │  │ x509_crt.c
│  │  ├── oberon-psa-crypto
│  │  │  ├── extras
│  │  │  │  │ pkparse.c
scripts
│  │ quarantine_zephyr.yaml
west.yml
zephyr
│  ├── MAINTAINERS.yml
│  ├── samples
│  │  ├── net
│  │  │  ├── ocpp
│  │  │  │  │ sample.yaml
│  ├── tests
│  │  ├── crypto
│  │  │  ├── psa
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── prj.conf
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
│  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 19756db709
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:19756db709_5ea73affbf

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-nrf_crypto
    • ✅ test-fw-nrfconnect-tfm
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-apps_nrfdesktop
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@tomi-font tomi-font force-pushed the mbedtls_update_4.1.0_fixup branch from 92ce873 to f094257 Compare May 26, 2026 11:34
@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented May 27, 2026

Memory footprint analysis revealed the following potential issues

applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9352[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 6096[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 13024[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9760[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.mbox[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 7202[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.mbox[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 3958[B] - link (cc: @nrfconnect/ncs-ll-ursus)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-29011/11)

@tomi-font tomi-font force-pushed the mbedtls_update_4.1.0_fixup branch from b8b4938 to e423bf0 Compare May 27, 2026 10:16
@github-actions
Copy link
Copy Markdown

Since quarantine was modified, please make sure you are following the process described in Quarantine Process.

@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented May 27, 2026

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
mbedtls nrfconnect/sdk-mbedtls@92d4456 (ncs-v3.4-branch,ncs-v3.4.0-rc1) nrfconnect/sdk-mbedtls@5b7ca76 (main) nrfconnect/sdk-mbedtls@92d44566..5b7ca767
oberon-psa-crypto nrfconnect/sdk-oberon-psa-crypto@3a90050 (ncs-v3.4-branch,ncs-v3.4.0-rc1) nrfconnect/sdk-oberon-psa-crypto@ce39726 (main) nrfconnect/sdk-oberon-psa-crypto@3a900502..ce39726a
zephyr nrfconnect/sdk-zephyr@a9f4778 nrfconnect/sdk-zephyr@6086d15 (main) nrfconnect/sdk-zephyr@a9f47784..6086d15e

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 27, 2026

You can find the documentation preview for this PR here.

@tomi-font tomi-font force-pushed the mbedtls_update_4.1.0_fixup branch from e423bf0 to 9d3c67c Compare May 28, 2026 09:37
@tomi-font tomi-font marked this pull request as ready for review May 28, 2026 09:39
@tomi-font tomi-font requested review from a team as code owners May 28, 2026 09:39
@tomi-font
Copy link
Copy Markdown
Contributor Author

@nrfconnect/ncs-paladin @nrfconnect/ncs-code-owners please review

@tomi-font tomi-font force-pushed the mbedtls_update_4.1.0_fixup branch from 9d3c67c to 691eae1 Compare May 28, 2026 10:22
Comment thread samples/bluetooth/mesh/ble_peripheral_lbs_coex/boards/nrf52dk_nrf52832.conf Outdated
Copy link
Copy Markdown
Contributor

@alxelax alxelax left a comment

Choose a reason for hiding this comment

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

This is workaround that does not fix or clarify the issue.

@tomi-font tomi-font force-pushed the mbedtls_update_4.1.0_fixup branch from 691eae1 to fd5c4e8 Compare May 29, 2026 07:41
@tomi-font
Copy link
Copy Markdown
Contributor Author

This is workaround that does not fix or clarify the issue.

@alxelax BT Mesh-related changes dropped, you can now remove your nack.

@tomi-font tomi-font requested a review from alxelax May 29, 2026 07:42
@alxelax alxelax dismissed their stale review May 29, 2026 07:44

Mesh related changes have been removed from this PR

@alxelax alxelax removed their request for review May 29, 2026 07:44
@tomi-font
Copy link
Copy Markdown
Contributor Author

@nrfconnect/ncs-code-owners please review

tomi-font added 5 commits May 29, 2026 13:52
To fix some regressions introduced with the update of Mbed TLS to
v4.1.0.

Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
To fix some regressions introduced with the update of Mbed TLS to
v4.1.0.

Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
This reverts commit 6b185fa.

Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
This reverts commit 002829a.

Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
The test case was renamed in Zephyr.

Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
@tomi-font tomi-font force-pushed the mbedtls_update_4.1.0_fixup branch from fd5c4e8 to 2693654 Compare May 29, 2026 10:52
@NordicBuilder NordicBuilder removed the DNM label May 29, 2026
@rlubos rlubos merged commit 7cc064d into nrfconnect:main May 29, 2026
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants