Skip to content

Conversation

@PavelVPV
Copy link
Contributor

@PavelVPV PavelVPV commented Aug 7, 2025

The bt_crypto_aes_cmac function from bt_crypto library uses psa_import_key function which requires memory for importing keys. The memory is allocated in the heap, therefore MBEDTLS_ENABLE_HEAP should be enabled.

This allows to build tests.bluetooth.bt_crypto.psa upstream tests.

@PavelVPV PavelVPV requested a review from a team as a code owner August 7, 2025 11:50
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 7, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 8, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 12

Inputs:

Sources:

sdk-nrf: PR head: 4f78df58f8c97a5d45bc279c03cc654f4111d0f6
zephyr: PR head: 319b359ae7669cd64ba3adec65034a4a4e4a5535

more details

sdk-nrf:

PR head: 4f78df58f8c97a5d45bc279c03cc654f4111d0f6
merge base: 2cf084fde86bcda7c6939c4186e4d1a6b1b028ae
target head (main): 2cf084fde86bcda7c6939c4186e4d1a6b1b028ae
Diff

zephyr:

PR head: 319b359ae7669cd64ba3adec65034a4a4e4a5535
merge base: 804a1d59154a63351e29d6cf294e3b462220d5cd
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 (3)
Kconfig.nrf
west.yml
zephyr
│  ├── tests
│  │  ├── bluetooth
│  │  │  ├── bt_crypto
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf

Outputs:

Toolchain

Version: c5be9c56c7
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:c5be9c56c7_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-zephyr test count: 4
  • ✅ Integration tests
    • ✅ test-sdk-find-my
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • 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_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 11, 2025

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@804a1d5 nrfconnect/sdk-zephyr@319b359 (main) nrfconnect/[email protected]

All manifest checks OK

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

@github-actions
Copy link

You can find the documentation preview for this PR here.

@PavelVPV
Copy link
Contributor Author

Passed here: https://jenkins-ncs.nordicsemi.no/job/latest/job/test-fw-twister-ncs-zephyr/job/customizable/job/main/751/console

14:59:34  INFO    - 1/2 nrf52840dk/nrf52840       tests/bluetooth/bt_crypto/bluetooth.bt_crypto.psa  PASSED (device: 1050272802, 4.459s <zephyr>)
14:59:38  INFO    - 2/2 nrf5340dk/nrf5340/cpuapp/ns tests/bluetooth/bt_crypto/bluetooth.bt_crypto.psa PASSED (device: 1050073141, 6.675s <zephyr>)

The bt_crypto_aes_cmac function from bt_crypto library uses
psa_import_key function which requires memory for importing keys.
The memory is allocated in the heap, therefore MBEDTLS_ENABLE_HEAP
should be enabled.

This allows to build tests.bluetooth.bt_crypto.psa upstream tests.

Signed-off-by: Pavel Vasilyev <[email protected]>
Pull in changes for nRF5340 bt_crypto zephyr test.

Signed-off-by: Pavel Vasilyev <[email protected]>
@PavelVPV PavelVPV force-pushed the fix_bt_crypto_upstream_tests branch from ed83753 to 4f78df5 Compare August 12, 2025 07:53
@NordicBuilder NordicBuilder removed the DNM label Aug 12, 2025
@nordic-piks nordic-piks merged commit a9891c4 into nrfconnect:main Aug 12, 2025
14 of 15 checks passed
@PavelVPV PavelVPV deleted the fix_bt_crypto_upstream_tests branch August 12, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants